Skip to content
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

bump Ipopt #10

Merged
merged 1 commit into from
Mar 17, 2014
Merged

bump Ipopt #10

merged 1 commit into from
Mar 17, 2014

Conversation

mlubin
Copy link
Contributor

@mlubin mlubin commented Mar 16, 2014

Bumps and fixes some recent issues with Ipopt and LAPACK on OS X. Requesting an updated bottle as well.

This supersedes #9.
@joehuchette @tkelman

@tkelman
Copy link
Collaborator

tkelman commented Mar 16, 2014

The way Homebrew works, you're always starting in a clean sandbox with just the source, right? Nothing can ever be left over from previous builds? As long as that's the case this looks good, you won't have the reusing libtool problem from jump-dev/Ipopt.jl#1 (comment) so you don't need separate build folders for coinblas and coinlapack as in jump-dev/Ipopt.jl#5.

@mlubin
Copy link
Contributor Author

mlubin commented Mar 16, 2014

I believe that you do start with a clean sandbox with homebrew. Anyway users shouldn't be using this script to build, it's really just for generating the binaries ("bottles").

staticfloat added a commit that referenced this pull request Mar 17, 2014
@staticfloat staticfloat merged commit c5cb7c7 into staticfloat:master Mar 17, 2014
@staticfloat
Copy link
Owner

Bottle has been uploaded. Please test!

@mlubin
Copy link
Contributor Author

mlubin commented Mar 17, 2014

Thanks!

@mlubin
Copy link
Contributor Author

mlubin commented Mar 18, 2014

On my 10.7 machine, Homebrew.jl seems to ignore the bottle and try to compile from source.

@staticfloat
Copy link
Owner

@mlubin can you give me the whole output?

@mlubin
Copy link
Contributor Author

mlubin commented Mar 31, 2014

Seems to be working now, never mind!

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants