add missing dependency for ipopt on libscotch #864
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
See conda-forge/ipopt-feedstock#112 (comment)
ipopt linked libscotch as a transitive dependency of mumps without declaring it as a dependency, meaning a mumps rebuild against an updated libscotch would break ipopt, which happened with mumps 5.7.3 build 5.
Since mumps pins libscotch all the way down, the only builds for which this is a problem is when a given version of mumps bumps libscotch after ipopt has been built against that mumps.
Since tracking down the scotch at build time for each ipopt build is tedious and the transitive pin from mumps is ~always right for past builds, only apply this patch to the latest version of ipopt (3.14.16).
All builds against mumps 5.6-5.7.2 only have builds against scotch 7.0.4, and only ipopt 3.14.16 has builds against 5.7.3. So while this added dependency is correct, it's only strictly needed to solve a problem for ipopt 3.14.16 build number 5, the first build against mumps 5.7.3.
diff