-
-
Notifications
You must be signed in to change notification settings - Fork 13k
boost 1.69.0 #35030
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
boost 1.69.0 #35030
Conversation
Well, starting Boost 1.69, the https://github.com/boostorg/boost/blob/boost-1.69.0/boostcpp.jam#L156-L157 The library name will look like |
e5ee5b6
to
151597a
Compare
I think we should keep the layout as-is for now, so I've changed |
Rev-bumping needed for |
a38ba87
to
54eea6f
Compare
osquery is missing some boost headers, so it's failing before the no-weak-import issue. I submitted a patch upstream in osquery/osquery#5325, but I don't think it will apply to 3.3.0, and the build is failing anyway. |
The boost signals library is removed in 1.69: I maintain one package that uses that; I need to see if we can easily switch to signals2 or if we'll need |
Turns out we aren't even using boost signals in the code; it was just searching for the component. I think it will be an easy fix for us; no |
54eea6f
to
dc7e846
Compare
I tried resolving conflicts through the gui, which created a merge commit and broke CI, so I rebased instead. |
…#5325) Summary: boost 1.69 is in a pull request in homebrew-core at Homebrew/homebrew-core#35030 and `osquery` is failing to compile due to using `boost::noncopyable` in a couple header files without including `boost/noncopyable.hpp`: * [jenkins log of build failure](https://jenkins.brew.sh/job/Homebrew%20Core%20Pull%20Requests/34941/version=high_sierra/testReport/junit/brew-test-bot/high_sierra/install_osquery/) The build still fails on homebrew due to #5284. Pull Request resolved: #5325 Differential Revision: D13464741 Pulled By: fmanco fbshipit-source-id: bdaf573b180f8cdfd08dc719db4829911249caaf
A patch for osquery has been accepted: osquery/osquery@41b785b A patch for uhd is in the works: EttusResearch/uhd#241 (comment) |
The libbitcoin repository was renamed at some point to libbitcoin-system, so the tarball sha256 has changed. Perhaps it should just point to the new repo name with the new sha256.
|
14f79e2
to
839c931
Compare
I've rebased and pushed two fixes:
|
that last build took 12 hours; here's a discussion of some (not all) of the failures
|
Ok, oddly, the
|
I've added a few commits to my fork while we wait for |
36bdd32
to
8f4bc2e
Compare
trying again with more patches... |
the build took 11 hours this time; it looks like two of my patches (690046f for graph-tool on 10.13 and 8c74c5a for osquery) didn't work. The other failures are:
|
the osquery patch should apply after osquery 3.3.2 #35941 |
I'm testing the folly build locally and then I will push if it's ok |
Then we should proceed. |
I just pushed 🤞 |
latest build:
seems close enough to me |
Given that a |
How hard will Boost 1.70 be to add? Really looking forward to Boost:::Histogram! |
brew install --build-from-source <formula>
, where<formula>
is the name of the formula you're submitting?brew audit --strict <formula>
(after doingbrew install <formula>
)?