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] Update to 1.70.0 #6141

Merged
merged 13 commits into from
May 17, 2019
Merged

Conversation

ras0219-msft
Copy link
Contributor

No description provided.

@ras0219-msft ras0219-msft added the info:internal This PR or Issue was filed by the vcpkg team. label Apr 19, 2019
@BurningEnlightenment
Copy link
Contributor

@ras0219 this PRr doesn't update the boost-build port. Is this intentional?
Compiled boost libraries (like boost-exception) won't compile under VS2019 without the boost build update. This is due to the inability of boost-build 1.69.0 to use/discover VS2019.

@ras0219-msft
Copy link
Contributor Author

Thanks for the comment! This was not intentional -- I need to update boost-build as well.

[fizz][folly] Update and fix for boost 1.70
[libsodium] Use CMake buildsystem replacement to enable non-Windows
@LilyWangL LilyWangL mentioned this pull request Apr 22, 2019
@ras0219-msft
Copy link
Contributor Author

@bitsrules We have a script which generates 99% of the changes located at ports\boost-vcpkg-helpers\generate-ports.ps1.

The status as of b1b9922 was

x64-windows master test notes
wangle Pass Fail Regression
boost-outcome Pass New Port
boost-histogram Pass New Port
x64-windows-static master test notes
wangle Pass Fail Regression
boost-outcome Pass New Port
boost-histogram Pass New Port
pcl Pass Fail Regression
x86-windows master test notes
boost-outcome Pass New Port
boost-histogram Pass New Port
pcl Pass Fail Regression

I believe I've fixed the wangle issue in 09477ff, so the only thing left is pcl.

@nickoe
Copy link

nickoe commented May 5, 2019

@ras0219-msft Hi, are you able to rebase this?

@Rastaban Rastaban self-assigned this May 6, 2019
@philjdf
Copy link
Contributor

philjdf commented May 9, 2019

Hi @ras0219, your Arrow port change removes the following args from vcpkg_configure_cmake

    -DBOOST_ROOT=${CURRENT_INSTALLED_DIR}
    -DBUILD_STATIC=${ARROW_BUILD_STATIC}
    -DBUILD_SHARED=${ARROW_BUILD_SHARED}
    -DPARQUET_ARROW_LINKAGE=${PARQUET_ARROW_LINKAGE}
    -DARROW_BOOST_USE_SHARED=${BOOST_USE_SHARED}
    -DARROW_USE_STATIC_CRT=${USE_STATIC_CRT}

Is this because they were superfluous?

@ras0219-msft
Copy link
Contributor Author

Yes, after reviewing the CMakeLists.txt for arrow I determined that they were all superfluous.

@fwosar
Copy link
Contributor

fwosar commented May 15, 2019

Is there anything that can be done to help move this along? The details of what fails don't seem to be public. The fact that there is no way to use the arguably most important C++ library with Visual Studio 2019 and vcpkg is kind of baffling.

@Sixshaman
Copy link

Sixshaman commented May 15, 2019

Is there anything that can be done to help move this along? The details of what fails don't seem to be public. The fact that there is no way to use the arguably most important C++ library with Visual Studio 2019 and vcpkg is kind of baffling.

I second this, can we do anything to help?

@ras0219-msft ras0219-msft merged commit 9d473a6 into microsoft:master May 17, 2019
@ras0219-msft ras0219-msft deleted the dev/roschuma/boost branch May 17, 2019 01:33
@ras0219-msft
Copy link
Contributor Author

Thanks for the offers!

@fwosar Can you confirm that this resolves all your issues using boost with 2019?

@star-e
Copy link

star-e commented May 17, 2019

When building boost 1.70 with vs2019, generated binaries are renamed to vc141-1_69
I have installed both vs2017 and vs2019. vs2019 has vc141 tool chain.

@OlafvdSpek
Copy link
Contributor

OlafvdSpek commented May 17, 2019

MSVC 16.1.0 Preview 3.0:

notice: found boost-build.jam at C:/vc/b/vcpkg/installed/x86-windows/tools/boost-build/boost-build.jam
notice: loading Boost.Build from C:/vc/b/vcpkg/installed/x86-windows/tools/boost-build/src/kernel
notice: Site configuration files will be ignored due to the
notice: --ignore-site-config command-line option.
notice: Loading explicitly specified user configuration file:
    C:\vc\b\vcpkg\buildtrees\boost-regex\x86-windows-rel\user-config.jam
notice: Searching 'C:\vc\b\vcpkg\buildtrees\boost-regex\x86-windows-rel' for user-config configuration file 'user-config.jam'.
notice: Loading user-config configuration file 'user-config.jam' from 'C:/vc/b/vcpkg/buildtrees/boost-regex/x86-windows-rel'.

warning: Did not find command for MSVC toolset. If you have Visual Studio 2017 installed you will need to specify the full path to the command, set VS150COMNTOOLS for your installation, or build from the 'Visual Studio Command Prompt for VS 2017'.

notice: will use 'cl.exe' for msvc, condition <toolset>msvc
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>', setup: 'x86'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>32', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>32', setup: 'x86'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>', setup: 'x86'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>32', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>32', setup: 'x86'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>64', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>/<address-model>64', setup: 'amd64'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>64', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>x86/<address-model>64', setup: 'amd64'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>ia64/<address-model>', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>ia64/<address-model>', setup: 'x86_ia64'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>ia64/<address-model>64', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>ia64/<address-model>64', setup: 'x86_ia64'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>', setup: 'x86_arm'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>32', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>32', setup: 'x86_arm'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>', setup: 'x86_arm64'
notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>64', setup: 'C:/vc/b/vcpkg/installed/x86-windows/share/boost-build/nothing.bat' notice: [msvc-cfg] condition: '<toolset>msvc/<architecture>arm/<address-model>64', setup: 'x86_arm64'
notice: [cmdline-cfg] toolset msvc-14.1 not previously configured; attempting to auto-configure now
error: msvc initialization: parameter 'version' inconsistent
error: no value was specified in earlier initialization
error: an explicit value is specified now

@chrullrich
Copy link
Contributor

chrullrich commented May 17, 2019

When building boost 1.70 with vs2019, generated binaries are renamed to vc141-1_69
I have installed both vs2017 and vs2019. vs2019 has vc141 tool chain.

Same here. Somewhere during the build, something installs share/boost-build/Jamroot.jam; this file still mentions 1.69 at the top, and changing that to 1.70 appears to fix the library names.

c72578 added a commit to c72578/vcpkg that referenced this pull request Apr 24, 2020
The vcpkg port of websocketpp has been pointing to the upstream commit
1c699ce of a PR, in order to fix build issues of websocketpp 0.8.1 with
boost 1.70 (microsoft#6141). This fix has been merged upstream in the meantime
and is part of the release 0.8.2 now.

- Update to WebSocket++/0.8.2 release
- Remove deprecated include(vcpkg_common_functions)
- Use ${PORT} instead of websocketpp
ras0219-msft pushed a commit that referenced this pull request Apr 24, 2020
The vcpkg port of websocketpp has been pointing to the upstream commit
1c699ce of a PR, in order to fix build issues of websocketpp 0.8.1 with
boost 1.70 (#6141). This fix has been merged upstream in the meantime
and is part of the release 0.8.2 now.

- Update to WebSocket++/0.8.2 release
- Remove deprecated include(vcpkg_common_functions)
- Use ${PORT} instead of websocketpp
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
info:internal This PR or Issue was filed by the vcpkg team.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

10 participants