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-config] Fix boost builds for WebAssembly #20551

Merged
merged 1 commit into from
Oct 6, 2021

Conversation

tobbi
Copy link
Contributor

@tobbi tobbi commented Oct 6, 2021

This PR fixes the compilation of boost-config on WASM / emscripten (upstream patch)

all

Yes

  • If you have added/updated a port: Have you run ./vcpkg x-add-version --all and committed the result?

Yes

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for boost-config but no changes to version or port version.
-- Version: 1.77.0
-- Old SHA: 2a93a4c962b961f617c133389c18a994e3c27af1
-- New SHA: 8f2ed64bee882aebc33c08ecee4d43ceb9140a77
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

@tobbi tobbi changed the title Fix boost wasm builds [boost-config] Fix boost builds for WebAssembly Oct 6, 2021
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

PRs must add only one version and must not modify any published versions

When making any changes to a library, the version or port-version in vcpkg.json or CONTROL must be modified.

Error: Local changes detected for boost-config but no changes to version or port version.
-- Version: 1.77.0
-- Old SHA: 2a93a4c962b961f617c133389c18a994e3c27af1
-- New SHA: 8f2ed64bee882aebc33c08ecee4d43ceb9140a77
-- Did you remember to update the version or port version?
-- Pass `--overwrite-version` to bypass this check.
***No files were updated.***

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout ff5afbab34834eaf1aebba808c47fea4fd606276 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/b-/boost-config.json b/versions/b-/boost-config.json
index 6e93f18..0ae80a5 100644
--- a/versions/b-/boost-config.json
+++ b/versions/b-/boost-config.json
@@ -1,5 +1,10 @@
 {
   "versions": [
+    {
+      "git-tree": "d17e033e2768f8e78783191780af9deb6c104276",
+      "version": "1.77.0",
+      "port-version": 2
+    },
     {
       "git-tree": "2a93a4c962b961f617c133389c18a994e3c27af1",
       "version": "1.77.0",
diff --git a/versions/baseline.json b/versions/baseline.json
index 3aa7be4..e606f79 100644
--- a/versions/baseline.json
+++ b/versions/baseline.json
@@ -582,7 +582,7 @@
     },
     "boost-config": {
       "baseline": "1.77.0",
-      "port-version": 0
+      "port-version": 2
     },
     "boost-container": {
       "baseline": "1.77.0",

Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout ff5afbab34834eaf1aebba808c47fea4fd606276 -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/b-/boost-config.json b/versions/b-/boost-config.json
index 752cae5..49b522f 100644
--- a/versions/b-/boost-config.json
+++ b/versions/b-/boost-config.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "d17e033e2768f8e78783191780af9deb6c104276",
+      "git-tree": "8ecb52721b71f17b6f3247812e1a51658c106f74",
       "version": "1.77.0",
       "port-version": 1
     },

@tobbi tobbi marked this pull request as ready for review October 6, 2021 16:44
Copy link

@github-actions github-actions bot left a comment

Choose a reason for hiding this comment

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

This is a new experimental fast check for PR issues. Please let us know if this bot is helpful!

After committing all other changes, the version database must be updated
git add -u && git commit
git checkout e1c8bbd3c6731cd988eb7e27d2fd0de7983e454d -- versions
./vcpkg x-add-version --all
Diff
diff --git a/versions/b-/boost-config.json b/versions/b-/boost-config.json
index 49b522f..920bf11 100644
--- a/versions/b-/boost-config.json
+++ b/versions/b-/boost-config.json
@@ -1,7 +1,7 @@
 {
   "versions": [
     {
-      "git-tree": "8ecb52721b71f17b6f3247812e1a51658c106f74",
+      "git-tree": "6aa6932068c1fbc812d545502f60c3a7cef80dde",
       "version": "1.77.0",
       "port-version": 1
     },

@strega-nil-ms
Copy link
Contributor

LGTM, thanks @tobbi :)

@strega-nil-ms strega-nil-ms merged commit 5033b3e into microsoft:master Oct 6, 2021
yurybura pushed a commit to yurybura/vcpkg that referenced this pull request Oct 7, 2021
BillyONeal added a commit that referenced this pull request Oct 12, 2021
* [scripts/boost] fix issue #20417
[boost-iostreams] replace explicit dependencies with default-features, allow to disable compression filters

* add versions

* [scripts/boost] update port version after #20421

* [boost-odeint] move boost-mpi dependency to the feature

* add version

* [boost-modular-build-helper] rework user-config generation

* [boost-python] move python3 from explicit dependency to default feature

* [boost-modular-build-helper] update version

* add verions

* [boost-modular-build-helper] fix empty flags

* update version

* Fix compilation on Emscripten / WebAssembly (#20551)

* [opencv4,opencv3] Control exported protobuf dependency (#20550)

* Control exported protobuf dependency

* Update versions

* Control exported protobuf dependency

* Update versions

* [OpenMVS] restore deprecated cmake scripts for configure, build & fixup targets (#20422)

* [OpenMVS] restore deprecated cmake scripts

* [OpenMVS] fix references

* switch back to vcpkg_cmake_*

Co-authored-by: nicole mazzuca <mazzucan@outlook.com>

* [libffi] Don't replace string in file that doesn't exist. (#20554)

* Don't replace string in file that doesn't exist.

* Update per bot.

* Update per bot again.

* Address comments.

* [json-dto] Update to 0.2.14 (#20570)

* json-dto updated to v.0.2.14.

* json-dto-0.2.14 added to baseline.

* Change deprecated commands.

* Update baseline for fresh json-dto-0.2.14.

* [sail] Update to 0.9.0-pre17 (#20562)

* [sail] Update to 0.9.0-pre17

* [sail] vcpkg x-add-version --all --overwrite-version

* [sail] Update pre17 hash

* [sail] vcpkg x-add-version --all --overwrite-version

* [freexl] Update to 1.0.6 (#20520)

* Reformat portfile

* Minimize makefiles patch

* Update to 1.0.6

* Fix uwp builds, remove skip from baseline

* Install pc file for windows

* Update versions

Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com>

* [dartsim] Update to 6.11.0 (#20566)

* Update dartsim

* Use newer vcpkg functions

* Update version number

* Update ports/dartsim/portfile.cmake

Co-authored-by: Robert Schumacher <roschuma@microsoft.com>

* Update versions

Co-authored-by: Robert Schumacher <roschuma@microsoft.com>

* [scripts/boost] update Boost version in boost-modular-build.cmake to avoid manual changes

* update version

* [boost-modular-build-helper] fix USER_CONFIG_EXTRA_LINES variable name and rename generated files

* update version

* Resolve differences from rerunning generate-ports.ps1.

Co-authored-by: Billy Robert O'Neal III <bion@microsoft.com>
Co-authored-by: Tobias Markus <tobbi.bugs@googlemail.com>
Co-authored-by: Kai Pastor <dg0yt@darc.de>
Co-authored-by: Stefano Sinigardi <stesinigardi@hotmail.com>
Co-authored-by: nicole mazzuca <mazzucan@outlook.com>
Co-authored-by: Lars Glud <larshg@gmail.com>
Co-authored-by: eao197 <eao197@users.noreply.github.com>
Co-authored-by: Dmitry Baryshev <dmitrymq@gmail.com>
Co-authored-by: Akash <Ace314159@users.noreply.github.com>
Co-authored-by: Robert Schumacher <roschuma@microsoft.com>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[boost-date-time] Build failure on wasm32:emscripten [boost-filesystem] Build failure on wasm32:emscripten
3 participants