Skip to content

Loading…

Fixed Clisp formula #10450

Closed
wants to merge 1 commit into from

2 participants

@nesv

As per this suggestion, I added a line to the nested "install" function in the Clisp formula so it would build in OS X Lion (10.7.3).

The formula now compiles, and installs, successfully.

@jacknagel

Thanks.

@jacknagel jacknagel added a commit that closed this pull request
@nesv nesv clisp: turn off optimization
Fixes some build issues on Lion.

Apparently it's enough to remove just the '-O' flags and preserve our
other optimization flags. This may only be needed on Lion, but frankly
clisp tends to have issues on all platforms so it's probably acceptable
to just disable it everywhere.

Closes #10450.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
ee35378
@jacknagel jacknagel closed this in ee35378
@mneise mneise pushed a commit that referenced this pull request
@nesv nesv clisp: turn off optimization
Fixes some build issues on Lion.

Apparently it's enough to remove just the '-O' flags and preserve our
other optimization flags. This may only be needed on Lion, but frankly
clisp tends to have issues on all platforms so it's probably acceptable
to just disable it everywhere.

Closes #10450.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
4ad42b0
@cacoco cacoco added a commit that referenced this pull request
@nesv nesv clisp: turn off optimization
Fixes some build issues on Lion.

Apparently it's enough to remove just the '-O' flags and preserve our
other optimization flags. This may only be needed on Lion, but frankly
clisp tends to have issues on all platforms so it's probably acceptable
to just disable it everywhere.

Closes #10450.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
d11de7f
@Sharpie Sharpie pushed a commit to Sharpie/homebrew that referenced this pull request
@nesv nesv clisp: turn off optimization
Fixes some build issues on Lion.

Apparently it's enough to remove just the '-O' flags and preserve our
other optimization flags. This may only be needed on Lion, but frankly
clisp tends to have issues on all platforms so it's probably acceptable
to just disable it everywhere.

Closes #10450.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
d551150
@snakeyroc3 snakeyroc3 pushed a commit to snakeyroc3/homebrew that referenced this pull request
@nesv nesv clisp: turn off optimization
Fixes some build issues on Lion.

Apparently it's enough to remove just the '-O' flags and preserve our
other optimization flags. This may only be needed on Lion, but frankly
clisp tends to have issues on all platforms so it's probably acceptable
to just disable it everywhere.

Closes #10450.

Signed-off-by: Jack Nagel <jacknagel@gmail.com>
79b228f
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Commits on Feb 24, 2012
  1. @nesv
Showing with 1 addition and 0 deletions.
  1. +1 −0 Library/Formula/clisp.rb
View
1 Library/Formula/clisp.rb
@@ -20,6 +20,7 @@ def patches
def install
ENV.j1 # This build isn't parallel safe.
+ ENV.remove_from_cflags /-O./
# Clisp requires to select word size explicitly this way,
# set it in CFLAGS won't work.
Something went wrong with that request. Please try again.