[go: nahoru, domu]
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.

Sign up for GitHub

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

Jump to bottom

coin-utils: add v2.11.11, apply autoreconf, use OpenBLAS, add GLPK support #23532

Open
wants to merge 9 commits into
base: master
Choose a base branch
from
Open

coin-utils: add v2.11.11, apply autoreconf, use OpenBLAS, add GLPK support #23532

wants to merge 9 commits into from
+57 −43

Conversation

valgur
Copy link
Contributor
@valgur valgur commented Apr 15, 2024

Also sets --enable-coinutils-threads on systems where it is supported.

I did not make BLAS and LAPACK support optional as using COIN-OR libraries without them barely makes sense, really. Can be added later, if necessary.

GLPK is used in most of the COIN-OR packages, but is GPL-licensed, so I included an option to disable it.

Requires

@conan-center-bot

This comment has been minimized.

Sign in to view
@prince-chrismc prince-chrismc mentioned this pull request Apr 15, 2024
Open
This was referenced Apr 15, 2024
Open
Open
Open
Open
@ericLemanissier ericLemanissier mentioned this pull request Apr 15, 2024
Closed
This was referenced Apr 15, 2024
Open
Open
@conan-center-bot
Copy link
Collaborator

Conan v1 pipeline ❌

Failure in build 2 (3b13c2feb837b75553f58136c8a52c36123073e8):

  • coin-utils/2.11.9:
    Error running command conan info coin-utils/2.11.9@#0b464a07065b71fdd89c3afd9a373bbd --json {jsonName} -pr {profileName}:

    [settings]
arch=x86_64
build_type=Release
compiler=gcc
compiler.libcxx=libstdc++11
compiler.version=5
os=Linux
[options]
coin-utils:shared=False

...
ERROR: Failed requirement 'glpk/4.48' from 'coin-utils/2.11.9'
ERROR: Unable to find 'glpk/4.48' in remotes

  • coin-utils/2.11.11:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.6:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.10:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.4:
    Didn't run or was cancelled before finishing

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

Conan v2 pipeline ❌

Note: Conan v2 builds are now mandatory. Please read our discussion about it.

The v2 pipeline failed. Please, review the errors and note this is required for pull requests to be merged. In case this recipe is still not ported to Conan 2.x, please, ping @conan-io/barbarians on the PR and we will help you.

See details:

Failure in build 2 (3b13c2feb837b75553f58136c8a52c36123073e8):

  • coin-utils/2.11.11:
    Error running command conan graph info --requires coin-utils/2.11.11@#a764e3aeefef42885965cb346244241c -f json -pr:h /home/conan/workspace/prod-v2/bsr/23512/beecb/profile_linux_11_libstdcpp11_17_gcc_release_64.-shared-False.txt -pr:b /home/conan/workspace/prod-v2/bsr/23512/beecb/profile_linux_11_libstdcpp11_17_gcc_release_64..txt:

    ======== Computing dependency graph ========
bzip2/1.0.8: Not found in local cache, looking in remotes...
bzip2/1.0.8: Checking remote: conan-center
bzip2/1.0.8: Downloaded recipe revision 457c272f7da34cb9c67456dd217d36c4
zlib/1.3.1: Not found in local cache, looking in remotes...
zlib/1.3.1: Checking remote: conan-center
zlib/1.3.1: Downloaded recipe revision f52e03ae3d251dec704634230cd806a2
openblas/0.3.26: Not found in local cache, looking in remotes...
openblas/0.3.26: Checking remote: conan-center
openblas/0.3.26: Downloaded recipe revision efb7501e3059ca4f35cafc365a199c29
glpk/4.48: Not found in local cache, looking in remotes...
glpk/4.48: Checking remote: conan-center
glpk/4.48: Checking remote: c3i_PR-v2-23532
Graph root
    cli
Requirements
    bzip2/1.0.8#457c272f7da34cb9c67456dd217d36c4 - Downloaded (conan-center)
    coin-utils/2.11.11#a764e3aeefef42885965cb346244241c - Cache
    openblas/0.3.26#efb7501e3059ca4f35cafc365a199c29 - Downloaded (conan-center)
    zlib/1.3.1#f52e03ae3d251dec704634230cd806a2 - Downloaded (conan-center)
Resolved version ranges
    zlib/[>=1.2.11 <2]: zlib/1.3.1
Graph error
    Package 'glpk/4.48' not resolved: Unable to find 'glpk/4.48' in remotes.
ERROR: Package 'glpk/4.48' not resolved: Unable to find 'glpk/4.48' in remotes.

  • coin-utils/2.11.9:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.6:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.4:
    Didn't run or was cancelled before finishing

  • coin-utils/2.11.10:
    Didn't run or was cancelled before finishing

Note: To save resources, CI tries to finish as soon as an error is found. For this reason you might find that not all the references have been launched or not all the configurations for a given reference. Also, take into account that we cannot guarantee the order of execution as it depends on CI workload and workers availability.

EstebanDugueperoux2
EstebanDugueperoux2 reviewed Jul 25, 2024
View reviewed changes
recipes/coin-utils/all/conanfile.py
# should be packaged and added to build requirements.
if hasattr(self, "settings_build") and cross_building(self) and self.options.shared:
raise ConanInvalidConfiguration("coin-utils shared not supported yet when cross-building")
self.requires("openblas/0.3.26")
Copy link
Contributor

Choose a reason for hiding this comment

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

Suggested change
self.requires("openblas/0.3.26")
self.requires("openblas/0.3.27")

EstebanDugueperoux2
EstebanDugueperoux2 suggested changes Jul 25, 2024
View reviewed changes
Copy link
Contributor
@EstebanDugueperoux2 EstebanDugueperoux2 left a comment

Choose a reason for hiding this comment

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

Just a requirement bump

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers

@EstebanDugueperoux2 EstebanDugueperoux2 EstebanDugueperoux2 requested changes

At least 2 approving reviews are required to merge this pull request.

Assignees
No one assigned
Labels
Failed
Projects
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

None yet
3 participants
@valgur @conan-center-bot @EstebanDugueperoux2