-
Notifications
You must be signed in to change notification settings - Fork 83
refactor(build): Remove CLP_NEEDS_X flags for marking required dependencies; bump ystdlib to y-scope/ystdlib-cpp@c03806a.
#1107
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
Open
gibber9809
wants to merge
35
commits into
y-scope:main
Choose a base branch
from
gibber9809:remove-needs-flags
base: main
Could not load branches
Branch not found: {{ refName }}
Loading
Could not load tags
Nothing to show
Loading
Are you sure you want to change the base?
Some commits from the old base branch may be removed from the timeline,
and old review comments may become outdated.
Open
Changes from 15 commits
Commits
Show all changes
35 commits
Select commit
Hold shift + click to select a range
19d4901
Encapsulate logic for finding,setting up, and logging found dependenc…
gibber9809 d60ffff
Update cmake scripts to point of just working after removing CLP_NEED…
gibber9809 47cd6a6
Comment out status messages that would repeat on every call to find_X
gibber9809 c98d664
Rename find_X functions to clp_find_X
gibber9809 7d1da0b
Use macros for most clp_find_X functions to avoid having to manually …
gibber9809 d588c3f
Fix docstring
gibber9809 4a39d14
Delete macros that are equivalent to a simple find_package call
gibber9809 4200b50
Remove most messages in utils
gibber9809 c806443
Move cmake/Utils/utils.cmake -> cmake/find_utils.cmake
gibber9809 2973087
Merge branch 'main' into remove-needs-flags
gibber9809 e749db4
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 1afacc5
Minor fixup after merge.
gibber9809 609ea59
Delete files accidentally added back in merge.
gibber9809 b2f917b
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 e5b9b7c
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 d2c24ae
Add missing clp_find_msgpack() calls
gibber9809 4c2bc50
Add one more missing instance of clp_find_msgpack
gibber9809 47c8643
Merge branch 'main' into remove-needs-flags
Bill-hbrhbr 9cd35ee
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 f76a408
Update components/core/cmake/find_utils.cmake
gibber9809 2d4ef0c
Remove unnecessary find_package(yaml-cpp) calls
gibber9809 51d26f6
Address review comment
gibber9809 b1e5d37
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 1251b95
Move remaining find_package calls into macros.
gibber9809 b6aeef6
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 322bfb4
Fix boost warning
gibber9809 02fc9f3
Make boost find_package call silent after first invocation.
gibber9809 0019aa2
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 3808fe2
Delete unnecessary function.
gibber9809 48ab8b2
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 a42fd90
Fix ystdlib-specific issue related to how ystdlib config scripts are …
gibber9809 05836ac
Merge remote-tracking branch 'upstream/main' into remove-needs-flags
gibber9809 735f91e
Add missing clp_find statements to log-converter CMakeLists
gibber9809 cf9f42a
Bump ystdlib to c03806a
gibber9809 3298467
Remove workaround for finding ystdlib
gibber9809 File filter
Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
There are no files selected for viewing
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
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
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
Oops, something went wrong.
Oops, something went wrong.
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.
Uh oh!
There was an error while loading. Please reload this page.