Homebrew / homebrew-core

🍻 Default formulae for the missing package manager for macOS (or Linux)
https://brew.sh
BSD 2-Clause "Simplified" License
13.75k stars 12.44k forks source link

Formulae fail to build #342

Closed xu-cheng closed 8 years ago

xu-cheng commented 8 years ago

I have finished the massive bottle build. Below is the list of formulae fail to build.

Without an El Capitan bottle, but not necessarily broken:

ilovezfs commented 8 years ago

sile cannot be bottled presently due to LuaRocks requirements. There is a new version though: https://github.com/Homebrew/homebrew-core/pull/1039

ilovezfs commented 8 years ago

skytools is a sandbox violation:

/bin/sh /usr/local/lib/postgresql/pgxs/src/makefiles/../../config/install-sh -c -d '/usr/local/share/postgresql/contrib'
mkdir: /usr/local/share/postgresql/contrib: Operation not permitted
make[2]: *** [installdirs-old-contrib] Error 1
make[1]: *** [install] Error 2
make: *** [sub-install] Error 2
==> Sandbox log
May 10 03:00:07 sandboxd[137]: mkdir(49045) deny file-write-create /usr/local/share/postgresql/contrib
ilovezfs commented 8 years ago

slony is also a sandbox violation:

/bin/sh /private/tmp/slony-20160510-51357-1nbbfrp/slony1-2.2.4/config/install-sh -c -m 755 slony1_funcs.2.2.4.so /usr/local/lib/postgresql/
cp: /usr/local/lib/postgresql//#inst.52720#: Operation not permitted
make[2]: *** [install] Error 1
make[1]: *** [install] Error 2
make: *** [install] Error 2
==> Sandbox log
May 10 03:02:54 sandboxd[137]: cp(52726) deny file-write-create /usr/local/lib/postgresql/#inst.52720#
ilovezfs commented 8 years ago

This fixes soccerwindow2: https://github.com/Homebrew/homebrew-core/pull/1044

ilovezfs commented 8 years ago

This fixes ssreflect https://github.com/Homebrew/homebrew-core/pull/1166

I figured it might be a good idea to get the ssreflect 1.5/Coq 8.4 series a working formula+bottle before replacing it outright with the mathcomp/ssreflect 1.6/Coq 8.5 new thing. The latter will depend on https://github.com/Homebrew/homebrew-core/pull/1142 being merged first.

MikeMcQuaid commented 8 years ago

Boneyarding a bunch of these in https://github.com/Homebrew/homebrew-core/pull/2194.

ilovezfs commented 8 years ago

@MikeMcQuaid not sure swatchbooker needs to go as @apjanke said "swatchbooker: just needs external dependency pillow," and pillow is vendored in several formulae in core currently so that should be trivial.

MikeMcQuaid commented 8 years ago

@ilovezfs I got it running locally but it has no installations and is a pretty terrible port so I'd rather we just killed it.

ilovezfs commented 8 years ago

@MikeMcQuaid In that case kill it with :fire:

UniqMartin commented 8 years ago

I updated the top comment with a second list of formulae that don't have an El Capitan bottle, but are not necessarily broken. I also included the reason for why they don't have a bottle. Not sure if this should be a separate issue, but I figured since this is all about completing our bottling, it fits. Generated with:

brew for-each-formula 'puts f.full_name if !f.tap? && f.bottle_defined? && !f.bottle_specification.collector.keys.include?(:el_capitan)'
MikeMcQuaid commented 8 years ago

🚢d.

ilovezfs commented 8 years ago

HOMEBREW_SANDBOX=1 for all!

MikeMcQuaid commented 8 years ago

@ilovezfs see https://github.com/Homebrew/brew/issues/713 😉