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

cmake: explicit boost components discovery (required for boost 1.71) #306

Merged
merged 1 commit into from
Sep 25, 2019
Merged

cmake: explicit boost components discovery (required for boost 1.71) #306

merged 1 commit into from
Sep 25, 2019

Conversation

anthraxx
Copy link
Contributor

Currently using boost 1.71 compiling the test executable results in
cmake errors for not using find_package for the required boost
components for proper discovery.

@puppetlabs-jenkins
Copy link
Contributor

Can one of the admins verify this patch?

@GabrielNagy
Copy link
Contributor

Hi @anthraxx,

Thanks for this! We noticed the issue after boost was bumped to 1.71 but didn't have time to get to it. I filed the following ticket for this: LTH-167.

Can you prepend the ticket name enclosed in brackets to the commit message? This helps us when writing changelogs on release. (e.g. (LTH-167) cmake: explicit boost components discovery)

Thank you

@puppetcla
Copy link

Waiting for CLA signature by @anthraxx

@anthraxx - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

Currently using boost 1.71 compiling the test executable results in
cmake errors for not using find_package for the required boost
components for proper discovery.
@anthraxx
Copy link
Contributor Author

Waiting for CLA signature by @anthraxx

@anthraxx - We require a Contributor License Agreement (CLA) for people who contribute to Puppet, but we have an easy click-through license with instructions, which is available at https://cla.puppet.com/

Note: if your contribution is trivial and you think it may be exempt from the CLA, please post a short reply to this comment with details. http://docs.puppet.com/community/trivial_patch_exemption.html

I don't think this single like cmake discovery really requires a CLA, it's hardly any meaningful contribution in terms of code but just a tiny toolchain adjustment.

PS: that trivial patch exemption link leads to a 404

@GabrielNagy
Copy link
Contributor

That's fine, this is the correct link for trivial patches: https://puppet.com/community/trivial-patch-exemption-policy

Copy link
Contributor

@GabrielNagy GabrielNagy left a comment

Choose a reason for hiding this comment

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

👍

@GabrielNagy GabrielNagy merged commit 68a00ef into puppetlabs:master Sep 25, 2019
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.

4 participants