Repository navigation
[stable/1.8] Add GitHub Actions CI, config.yml, and generate_readme; relicense under EPL 2.0 - #30
Merged
Merged
Conversation
Port of 3d432dc (Bjarni Kristjansson) from master. CbcOrClpRead_mode is defined in both libClpSolver and libCbcSolver of Clp 1.17/Cbc 2.10. Through it, the unit test got linked with libClpSolver ahead of libCbcSolver, and then libCbcSolver's calls of establishParams() and whichParam() were bound to libClpSolver's versions, which don't know Cbc's parameters, so CbcMain0() crashed (when solving P0033). The workaround it was set for (a bug in CbcMain1 of Cbc 2.2) is not needed anymore. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
…er EPL 2.0 As on master (and for the other projects of the COIN-OR Optimization Suite): - .github/workflows: Linux/macOS and Windows (MSYS2 and MSVC) builds and tests, and release builds. - .coin-or/config.yml describes the project and its dependencies (CoinUtils 2.11, Osi 0.108, Clp 1.17, Cgl 0.60, Cbc 2.10, the current stable versions); Dependencies uses the GitHub repositories and matches config.yml. The SVN Externals file is removed. - README.md is generated by .coin-or/generate_readme; the old README (which pointed to CoinMP/README) is removed. - CoinMP/LICENSE: CoinMP is relicensed from the Common Public License 1.0 to the Eclipse Public License 2.0, like the other projects of the Suite, and the comments that mentioned the Common Public License are updated (in configure and the Makefile.in files only these comment lines change). - The Travis and AppVeyor setup is removed. Co-Authored-By: Claude Opus 5.5 <noreply@anthropic.com>
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
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.
This adds the GitHub Actions CI,
.coin-or/config.ymlandgenerate_readmetostable/1.8, as #29 does for master. The build system is unchanged; only master is converted to the new one.stablebranches that still use BuildTools 0.8, including--disable-pkg-configon Windows.config.ymlandDependenciesnow list the current stable versions instead: CoinUtils 2.11, Osi 0.108, Clp 1.17, Cgl 0.60 and Cbc 2.10.Dependenciespoints at the GitHub repositories, and the old SVNExternalsfile is removed.CbcOrClpRead_mode.CbcMain0()when solving P0033.CbcOrClpRead_modeis defined in bothlibClpSolverandlibCbcSolverof Clp 1.17/Cbc 2.10. Through it, the unit test got linked withlibClpSolverahead oflibCbcSolver.libCbcSolver's calls ofestablishParams()andwhichParam()were then bound tolibClpSolver's versions, which don't know Cbc's parameters.README.mdis generated, and the old rootREADMEis removed.CoinMP/LICENSEchanges from the Common Public License 1.0 to the Eclipse Public License 2.0, as on master and like the rest of the Suite. The CPL comments are updated too; inconfigureand theMakefile.ins only those comment lines change.🤖 Generated with Claude Code