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

boost 1.68.0 #30914

Closed
wants to merge 41 commits into from
Closed

boost 1.68.0 #30914

wants to merge 41 commits into from

Conversation

lembacon
Copy link
Member

@lembacon lembacon commented Aug 9, 2018

  • Have you followed the guidelines for contributing?
  • Have you checked that there aren't other open pull requests for the same formula update/change?
  • Have you built your formula locally with brew install --build-from-source <formula>, where <formula> is the name of the formula you're submitting?
  • Does your build pass brew audit --strict <formula> (after doing brew install <formula>)?

boost-build has not been updated to 1.68.0 yet. https://github.com/boostorg/build/releases

Edited: boost-build has been updated to 1.68.0.

@eyal0
Copy link

eyal0 commented Aug 9, 2018

@lembacon Hello!

I am a maintainer of pcb2gcode. pcb2gcode uses boost geometry heavily.

Boost geometry 1.67 and up is severely broken: https://svn.boost.org/trac10/ticket/13645 No one has responded to my issue.

boost 1.66 is the latest working boost that I have found. For my OSX users, this is a problem because there are not versions between boost 1.60 and 1.67 available on brew.

I do not have any OSX computers or experience. Can you or someone else add boost 1.66 to homebrew? Or could you point me to someone that can?

My users would appreciate this. Thanks!

@lembacon
Copy link
Member Author

lembacon commented Aug 9, 2018

Since @DomT4 pointed out that

[…], it’s worth splitting anything that breaks into individual PRs and handling those after the main boost update has been pushed through successfully. Trying to get a boost update done in one PR generally results in it taking weeks.

It will indeed take us forever to update boost, so I'm temporarily dropping all commits regarding revision bumps, and will fix test failures due to broken linkages in a separated PR.

[Edited] Actually there should be plenty of separated PRs though. 🤨

@lembacon
Copy link
Member Author

lembacon commented Aug 9, 2018

@eyal0 I'm take a look later. Anyway, a pull request is always welcome though. :-)

@DomT4
Copy link
Member

DomT4 commented Aug 9, 2018

It will indeed take us forever to update boost, so I'm temporarily dropping all commits regarding revision bumps

It's worth running it through once, leaving anything in that works or can be fixed in a trivial way, but pulling out anything that is broken in that initial run into a unique PR. It's possible not everything needs a bump; last time around @commitay only updated ~30 formulae. That number is pretty consistent so I think it's worth starting with that list and see what happens.

boost 1.66 is the latest working boost that I have found. For my OSX users, this is a problem because there are not versions between boost 1.60 and 1.67 available on brew.

It's likely one would need to 💀 if we added a new one. We already have more old boost versions sitting around than we should.

@lembacon
Copy link
Member Author

lembacon commented Aug 9, 2018

Pushed revision bumps for 29 formulae. 🤨

@lembacon lembacon force-pushed the boost-1.68.0 branch 2 times, most recently from 0d97664 to 03d1f37 Compare August 10, 2018 11:39
@lembacon
Copy link
Member Author

lembacon commented Aug 10, 2018

Some updates here after an, unsurprisingly, failed CI run:

@eyal0
Copy link

eyal0 commented Aug 10, 2018

@lembacon No, I think that I figured out what is happening and the problem was mine.

First, bg:simplify on a polygon now changes the start/end point of a ring. That is a new behavior that surprised me but it's correct.

Second, passing a vector of pairs of doubles to bg::simplify used to simplify it like a linestring. That is no longer true. The behavior is strange now. But if my input is a valid input with the right geometry concept, it works fine.

So my problem went away when I fixed my input type. No need for boost 1.66 for me anymore.

Thanks for your hard work!

@lembacon
Copy link
Member Author

@eyal0 So we can drop the vendored boost 1.66.0 when a new release of pcb2gcode is available. 😉

@jbeich
Copy link

jbeich commented Aug 10, 2018

mkvtoolnix and osrm-backend are both still failing due to Undefined symbols ... system_category_instance

Try relplacing -std=c++11 with -std=c++14 per boostorg/system#24
https://github.com/Homebrew/homebrew-core/blob/c7893f954fefda5076144a161014e349336c742a/Formula/boost.rb#L74

@lembacon
Copy link
Member Author

Try relplacing -std=c++11 with -std=c++14 per boostorg/system#24

If we decide to bump the C++ standard used to build Boost from C++11 to C++14, then we may really need to rebuild everything depending on Boost. At least, this is something worthing considering. As of LLVM 6.0.0, which was released on 8 Mar 2018, the default C++ standard is C++14, IIRC. cc @Homebrew/maintainers

@fxcoudert
Copy link
Member

Let's try not to move C++ standard to much… it might break other stuff :(

@DomT4
Copy link
Member

DomT4 commented Aug 11, 2018

@BrewTestBot test this please

@DomT4
Copy link
Member

DomT4 commented Aug 11, 2018

It's fairly quiet right now so we'll try and get a build through. Go ahead and kill it though if CI gets super busy again.

@DomT4 DomT4 added CI-requeued PR has been re-added to the queue and removed CI-requeued PR has been re-added to the queue labels Aug 11, 2018
@lembacon lembacon added the in progress Stale bot should stay away label Aug 13, 2018
@lembacon
Copy link
Member Author

The CI is started again here, feel free to kill it.

@lembacon
Copy link
Member Author

Wish I could add a versioned boost@1.67 for those failed formulae. 🤒

@DomT4
Copy link
Member

DomT4 commented Aug 13, 2018

Hopefully it'll be okay to get through 🤞. It's been a quiet day for CI thus far.

@lembacon lembacon mentioned this pull request Aug 16, 2018
4 tasks
@fxcoudert
Copy link
Member

@BrewTestBot test this please

@JoeLoser
Copy link

Is there a decision on how to move forward with this PR? I'd love to see this get into master so I do not keep maintaining a Boost 1.68 built from source rather than Homebrew.

@claui
Copy link
Contributor

claui commented Sep 25, 2018

@BrewTestBot test this please

@lembacon
Copy link
Member Author

@JoeLoser I didn't find myself some time to handle this as well as #31193, I'm happy to close these PRs in favor of yours, if you do want to take them over. :-)

@lembacon lembacon mentioned this pull request Oct 1, 2018
4 tasks
@lembacon
Copy link
Member Author

lembacon commented Oct 1, 2018

Superseded by #32610.

@lembacon lembacon closed this Oct 1, 2018
@lembacon lembacon deleted the boost-1.68.0 branch October 1, 2018 20:10
@lock lock bot added the outdated PR was locked due to age label Oct 31, 2018
@lock lock bot locked as resolved and limited conversation to collaborators Oct 31, 2018
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
in progress Stale bot should stay away outdated PR was locked due to age
Projects
None yet
Development

Successfully merging this pull request may close these issues.

7 participants