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

Add FindIpopt.cmake module #720

Merged
merged 6 commits into from
Oct 16, 2024
Merged

Conversation

ManifoldFR
Copy link
Contributor

@ManifoldFR ManifoldFR commented Oct 7, 2024

This PR:

  • adds a FindIpopt.cmake module, which enables the use of find_package(Ipopt)

@ManifoldFR ManifoldFR force-pushed the topic/find-ipopt branch 2 times, most recently from c637376 to 1a95f59 Compare October 7, 2024 12:45
find-external/Ipopt/FindIpopt.cmake Outdated Show resolved Hide resolved
find-external/Ipopt/FindIpopt.cmake Outdated Show resolved Hide resolved
find-external/Ipopt/FindIpopt.cmake Outdated Show resolved Hide resolved
find-external/Ipopt/FindIpopt.cmake Outdated Show resolved Hide resolved
@ManifoldFR ManifoldFR force-pushed the topic/find-ipopt branch 2 times, most recently from 3d5cfee to 8df2047 Compare October 9, 2024 09:38
@ManifoldFR
Copy link
Contributor Author

Hi folks, are we merging this after all ? Do you have feedback @jorisv ?

Copy link
Contributor

@jorisv jorisv left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I will test FindIpopt on Linux and Windows before approving

Since we don't know if this hack is still relevant, we remove it.
Ref: ghorn/debian-coinor-ipopt#2
@jorisv jorisv merged commit d280ff2 into jrl-umi3218:master Oct 16, 2024
2 checks passed
@ManifoldFR ManifoldFR deleted the topic/find-ipopt branch October 16, 2024 09:28
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