-
-
Notifications
You must be signed in to change notification settings - Fork 183
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
Upgrade Boost to 1.81 #2955
Upgrade Boost to 1.81 #2955
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
@andrjohns, thanks for doing this!
There's a file that was missed in the script that needs to be modified by hand. doxgyen/README.md
. I'm seeing L11 and L76.
Has R's BH package been bumped to 1.81 or is that concern irrelevant now? |
Yep, 1.81 is the current version of |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Thank you!
After this cmdstan's build has a new warning:
It's not immediately clear to me what we can do - it sounds like boost is including its own deprecated header? |
I had a look at the error, looks like it's a Stan header including
I believe that this line just needs to be changed to |
I think that |
Summary
This PR upgrades the bundled boost version to 1.81, using the
lib/upgrade-boost.sh
script. 1.81 is the current version in the RBH
headers.'Tests
N/A current tests should still pass
Side Effects
N/A
Release notes
Updated Boost headers to v1.81
Checklist
Math issue #(issue number)
Copyright holder: Andrew Johnson
The copyright holder is typically you or your assignee, such as a university or company. By submitting this pull request, the copyright holder is agreeing to the license the submitted work under the following licenses:
- Code: BSD 3-clause (https://opensource.org/licenses/BSD-3-Clause)
- Documentation: CC-BY 4.0 (https://creativecommons.org/licenses/by/4.0/)
the basic tests are passing
./runTests.py test/unit
)make test-headers
)make test-math-dependencies
)make doxygen
)make cpplint
)the code is written in idiomatic C++ and changes are documented in the doxygen
the new changes are tested