Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Hang if subprocess spawns dependent subprocess #630

Open
harrisonliew opened this issue Jan 25, 2022 · 1 comment
Open

Hang if subprocess spawns dependent subprocess #630

harrisonliew opened this issue Jan 25, 2022 · 1 comment
Labels
advanced PR bug low priority Easy workaround exists, regular QoL issue usability

Comments

@harrisonliew
Copy link
Contributor

harrisonliew commented Jan 25, 2022

In this scenario, Hammer will hang:

  1. Hammer spawns a subprocess running a tool (e.g. a fill deck)
  2. The tool's script spawns some other process that waits for the tool to finish before running (e.g. merging fill with input database, like Hammer GDS merging #287)
  3. As a result, there is a bidirectional wait dependency and so the tool never exits.

It appears the subprocess module can't support this scenario? At the moment, these kinds of scripts need to be run outside of Hammer.

@harrisonliew harrisonliew added bug low priority Easy workaround exists, regular QoL issue usability advanced PR labels Jan 25, 2022
@harrisonliew
Copy link
Contributor Author

Possibly related: when VCS runs the compiled simulator, that output is not captured by the Hammer logs either.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
advanced PR bug low priority Easy workaround exists, regular QoL issue usability
Projects
None yet
Development

No branches or pull requests

1 participant