-
-
Notifications
You must be signed in to change notification settings - Fork 1.9k
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
Fails at PostgreSQL on OS X Yosemite DP7 #286
Comments
I can confirm this issue. Seems like Perl is definitely missing from DP7, as people are having problem compiling weechat as well in Homebrew/legacy-homebrew#32146. Maybe we should just add |
--no-perl seems fine with me. For a long time, we had --no-python for similar reasons and lack of explicit need.
|
This looks like a major issue when building with Perl and possibly many other Apple-supplied libs on DP7. Several brews are affected, including the following two Laptop brews that fail with linker errors:
Note: It's a significant leap to assume that two linker errors have the same root cause, but it's at least somewhat less of a leap in this case as these errors where introduced at close to the same time. In addition, Qt fails to compile (a failure that's unrelated to the above linker errors):
For PostgreSQL, my team doesn't need Perl support, so But, there's a bigger issue: This workaround for PostgreSQL isn't sufficient to make Laptop run successfully on OS X Yosemite DP7. We'd also need workarounds for Vim and Qt. Those workarounds may not be as trivial, and all the workarounds ought to be reverted as upstream fixes become available. Since these issues apply to pre-release software (that Laptop doesn't claim to support), my inclination is to wait for or contribute to upstream fixes. An upstream fix for Qt is already being worked on. Apple may have acknowledged the bug affecting PostgreSQL and Vim, and a fix is anticipated soon in DP8 (assuming the errors are related). @croaky For the OS X Yosemite DPs, do you think it's okay track Laptop problems but wait for upstream fixes rather than applying Laptop workarounds? In any case, I'll check Laptop for OS X 10.10 Yosemite DP8 when it comes out (probably in a week or so). |
A class of bugs are possible similar to why we had `--no-python` for Postgres previously. #286
That is my preference as well for pre-releases. We can test and potentially put in fixes but for these particular ones, it looks like the Homebrew issues are now closed and DP8 is out. I think we can still add the I'm going to close this, but if I've misread those Homebrew threads or folks feel differently, please feel free to re-open. |
I actually did test this on DP8, but ran into another problem that I think had to do with the Apple CLI Tools package installed by Homebrew not yet having been updated by Apple — I think they were the Xcode 6.0 CLI Tools when DP8 needed Xcode 6.1 CLI Tools, but I didn't pursue it long enough to say so definitively. In any case GM Candidate 1.0 is out now and I updated my VM to it over lunch. I'm testing Laptop now from it's master branch (which at the moment doesn't disable Perl; it's without #293). |
Laptop successfully installs on OS X Yosemite GM Candidate 1.0 — see the laptop.log. Note that the install succeeded without disabling Perl — adding |
Thanks, @gohanlon. |
This appears to be an upstream issue in Homebrew, PostgreSQL, or even OS X:
brew install postgresql
fails on OS X Yosemite Developer Preview 7 (DP7).As of Homebrew/legacy-homebrew@cd33c4b, installing Laptop on a clean OS X Yosemite DP 7 fails as follows:
There is a mega-issue that tracks Homebrew formula status on OS X Yosemite: Homebrew/legacy-homebrew#29988. The issue states that
brew install postgresql
was broken but now works as of OS X Yosemite DP4. I added a comment to the mega-issue with the detailed failure information from my experience on clean DP7.The text was updated successfully, but these errors were encountered: