Skip to content

HTTPS clone URL

Subversion checkout URL

You can clone with
or
.
Download ZIP

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
Collaborator

Thanks.

@jacknagel jacknagel closed this pull request from a commit
@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
@aredridel aredridel referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
@royhodgman royhodgman referenced this pull request from a commit
Commit has since been removed from the repository and is no longer available.
@mneise mneise referenced this pull request from a commit
@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 referenced this pull request from a commit
@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 referenced this pull request from a commit in Sharpie/homebrew
@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 referenced this pull request from a commit in snakeyroc3/homebrew
@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
This page is out of date. Refresh to see the latest.
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.