Skip to content

osquery: vendor boost 1.62.0 and depend on openssl#8290

Closed
ilovezfs wants to merge 1 commit intoHomebrew:masterfrom
ilovezfs:osquery-revision-for-boost-1.63.0
Closed

osquery: vendor boost 1.62.0 and depend on openssl#8290
ilovezfs wants to merge 1 commit intoHomebrew:masterfrom
ilovezfs:osquery-revision-for-boost-1.63.0

Conversation

@ilovezfs
Copy link
Copy Markdown
Contributor

@ilovezfs ilovezfs commented Dec 29, 2016

  • vendor Boost 1.62.0 since osquery is not yet compatible with 1.63.0
  • depend on openssl
  • drop libressl dependency (as it wasn't actually being used anyway)

Without the Boost vendoring, this was failing to build due to the Boost 1.63.0 upgrade: https://bot.brew.sh/job/Homebrew%20Testing/1209/version=el_capitan/console

Note that osquery 1.7.3 is several versions behind the current stable
version (2.2.1), which is also be broken with boost 1.63.0.

CC @theopolis

@ilovezfs ilovezfs added the upstream issue An upstream issue report is needed label Dec 29, 2016
@ilovezfs ilovezfs changed the title osquery: revision for boost osquery: vendor boost 1.62.0 and depend on openssl Jan 1, 2017
@ilovezfs ilovezfs force-pushed the osquery-revision-for-boost-1.63.0 branch from aa07624 to a756691 Compare January 1, 2017 01:47
@ilovezfs ilovezfs requested a review from MikeMcQuaid January 1, 2017 01:47
@ilovezfs
Copy link
Copy Markdown
Contributor Author

ilovezfs commented Jan 1, 2017

Boost 1.63.0 breakage reported to osquery upstream here: osquery/osquery#2888.

vendor Boost 1.62.0 since osquery is not yet compatible with 1.63.0
depend on openssl
drop libressl dependency (as it wasn't actually being used anyway)
@ilovezfs ilovezfs force-pushed the osquery-revision-for-boost-1.63.0 branch from a756691 to b55a68c Compare January 1, 2017 02:14
Copy link
Copy Markdown
Member

@MikeMcQuaid MikeMcQuaid left a comment

Choose a reason for hiding this comment

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

Does this build all of Boost or just some of it? Given the vendoring and difficulty building a newer version I'm tempted to say we just boneyard this.

@ilovezfs
Copy link
Copy Markdown
Contributor Author

ilovezfs commented Jan 1, 2017

 +        --with-libraries=filesystem,regex,system 

https://github.com/Homebrew/homebrew-core/pull/8290/files#diff-ef2935cce4dec080563243e3608d91b3R79

It only builds those three libraries, which is fast.

As for boneyarding, analytics shows 1,340 installs in the last 30 days.

@MikeMcQuaid
Copy link
Copy Markdown
Member

Ok, cool, that's fine with me. Kinda regardless of the number of installs it feels a bit 💩 to be sitting on an old version for so long but I'm happy to leave it if you are.

@ilovezfs
Copy link
Copy Markdown
Contributor Author

ilovezfs commented Jan 1, 2017

@MikeMcQuaid I recall that I was able to work around osquery/osquery#2362 by similarly vendoring a static aws, so that should still be a way to get it current again.

@ilovezfs ilovezfs closed this in e7db721 Jan 2, 2017
@Homebrew Homebrew locked and limited conversation to collaborators May 4, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

upstream issue An upstream issue report is needed

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants