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

Upgrade vcpkg dependencies for the missed boost-format dependency #427

Draft
wants to merge 4 commits into
base: main
Choose a base branch
from

Conversation

BewareMyPower
Copy link
Contributor

Motivation

See microsoft/vcpkg#36510

Currently the commit id of the vcpkg submodule is old. With latest dependencies in vcpkg, it requires a boost-format dependency to build.

Modifications

Upgrade the vcpkg submodule to the latest commit at the moment and add the boost-format dependency to fix the build.

### Motivation

See microsoft/vcpkg#36510

Currently the commit id of the vcpkg submodule is old. With latest
dependencies in vcpkg, it requires a `boost-format` dependency to build.

### Modifications

Upgrade the vcpkg submodule to the latest commit at the moment and add
the `boost-format` dependency to fix the build.
@BewareMyPower BewareMyPower added this to the 3.6.0 milestone Jun 11, 2024
@BewareMyPower BewareMyPower self-assigned this Jun 11, 2024
@BewareMyPower
Copy link
Contributor Author

There are some failed tests. I'm going to check if they were introduced by an upgraded dependency

@BewareMyPower BewareMyPower marked this pull request as draft June 11, 2024 15:23
@BewareMyPower
Copy link
Contributor Author

It's an issue with the JSON serialization for key-value schema. Maybe caused by the boost.property_tree dependency.

@BewareMyPower BewareMyPower marked this pull request as ready for review June 12, 2024 03:33
@BewareMyPower
Copy link
Contributor Author

The root cause is that latest's Boost.PropertyTree does not write a line ending in write_json. So I updated the Boost version to 1.85 in dependencies.yaml as well to avoid behavior inconsistency.

@BewareMyPower BewareMyPower marked this pull request as draft June 12, 2024 13:08
@BewareMyPower BewareMyPower marked this pull request as ready for review June 13, 2024 09:22
@BewareMyPower
Copy link
Contributor Author

There is a bug for GCC 4.8 that it does not support std::align, which is a standard of C++11.

I sent an email to drop GCC 4.8 in the next release (see https://lists.apache.org/thread/7o8hpv1gtoffvzx053wm0ss2s9xt0795), so I will open a PR to drop the support for GCC 4.8 first and then go back to this PR.

@BewareMyPower BewareMyPower marked this pull request as draft June 13, 2024 14:16
@shibd shibd modified the milestones: 3.6.0, 3.7.0 Aug 28, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants