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

Remove custom FindBoost.cmake #149

Merged
merged 2 commits into from
Aug 20, 2019
Merged

Remove custom FindBoost.cmake #149

merged 2 commits into from
Aug 20, 2019

Conversation

jmjatlanta
Copy link

A custom FindBoost.cmake was added to resolve a v2.x version cmake bug. We now only support cmake 3.2 and above, so the extra steps included are no longer necessary.

@jmjatlanta
Copy link
Author

travis is complaining. It works on my 16.04 and 18.04 Ubuntu boxes. I am attempting to find out what is going on.

@pmconrad
Copy link

pmconrad commented Aug 17, 2019

Getting the same error locally. Apparently -lpthread is missing on the linker command line.

@jmjatlanta
Copy link
Author

Getting the same error locally. Apparently -lpthread is missing on the linker command line.

That stinks. From what I read, they fixed an issue with that back in the 2.x versions of cmake. I guess it crept up again. What version of cmake are you using?

@pmconrad
Copy link

cmake version 3.10.2

With my last commit here it works again. Apparently removing BoostConfig was too much.

Copy link
Member

@oxarbitrage oxarbitrage left a comment

Choose a reason for hiding this comment

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

looks good.

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