-
-
Notifications
You must be signed in to change notification settings - Fork 1.5k
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
resolve-system-dependencies.pl hanging #941
Conversation
Actually, to put it into more technical terms, I believe that the output of |
Here is an attempt, using best of my (non-existent) Perl knowledge. |
We should always read all the output before calling `waitpid` as otherwise the process might overfill the buffer and block.
@domenkozar install bug? |
Looks like this was some weirdness on the side of the OS. After a reboot the issue (and a few others) is gone. But I think, this changes make sense anyway. |
I'm labeling all issues that rise when Nix is being installed or is installing. These should have a priority. |
What's the workaround until this gets merged? How do I apply this patch? This is a big blocker. It's seeming to me that Nix just doesn't work at all on OSX because of it? |
@chris-martin Actually in my case the workaround was to restart the computer. Apparently, such a small buffer is not a normal thing, but for some reason it suddenly became small, probably due to a bug in OS X or whatever. Restarting the OS made the issue go away. |
See #1030 |
fwiw, a reboot did work for us (thanks @kirelagin 👍) |
Closing due to #1030 (comment) |
This is OS X 10.11.5.
The problem is here. It totally seems to me that
otool
is not willing to output anything unless someone is actually reading its output. If I execute that sameotool
command line myself, everything works fine; but if I runresolve-system-dependencies.pl
theotool
process hangs.I’m as far from being a Perl guru as it gets, so I made a quick and dirty adjustment: I made it first read all the output and only then
waitpid
and check the exit status. (Not showing the code here as it is extremely ugly.) This helped.