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-filesystem] fix emscripten build #29540

Merged
merged 5 commits into from
Feb 9, 2023

Conversation

Pospelove
Copy link
Contributor

@Pospelove Pospelove commented Feb 8, 2023

fixes #29529

my previous pr #29481 fixed builds of some of non-header-only boost libs

however non-header-only libs that also depend on other non-header-only libs kept failing (like boost-filesystem). in this pr I finally managed to fix that properly. deleted cmake lines that were added in my previous pr as a hack

based on bfgroup/b2#111

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: checked-in files for boost-build have changed but the version was not updated
version: 1.81.0#2
old SHA: 641f76f831109d1c1e6b412dc0be9f0c07f0c1fa
new SHA: 40cd5234db2baf10fb0a2b0ec8b40771910ca045
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***
error: checked-in files for boost-modular-build-helper have changed but the version was not updated
version: 1.81.0#3
old SHA: 51714c0e72628b70c25c15e0a8d494672b955718
new SHA: d47cef023f39ac9d5f3a043d35cba1a74710e1b3
Did you remember to update the version or port version?
Use --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!

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: checked-in files for boost-modular-build-helper have changed but the version was not updated
version: 1.81.0#3
old SHA: 51714c0e72628b70c25c15e0a8d494672b955718
new SHA: d47cef023f39ac9d5f3a043d35cba1a74710e1b3
Did you remember to update the version or port version?
Use --overwrite-version to bypass this check
***No files were updated***

@Pospelove
Copy link
Contributor Author

@tobbi Hey, could you please test this?

@tobbi
Copy link
Contributor

tobbi commented Feb 8, 2023

@tobbi Hey, could you please test this?

Hey, I can confirm that the above changes fix the issue. Thanks for dealing with this so quickly! :)

@Cheney-W Cheney-W added category:port-bug The issue is with a library, which is something the port should already support category:community-triplet A PR or issue related to community triplets not officially validated by the vcpkg team. labels Feb 9, 2023
@Cheney-W Cheney-W added the info:reviewed Pull Request changes follow basic guidelines label Feb 9, 2023
@JavierMatosD JavierMatosD merged commit 0551bc7 into microsoft:master Feb 9, 2023
@Pospelove Pospelove deleted the b2-wasm-fix branch February 9, 2023 18:58
@BillyONeal
Copy link
Member

This forgot to run generate-ports.ps1 :(

BillyONeal added a commit to BillyONeal/vcpkg that referenced this pull request Feb 18, 2023
…rts.ps1.

microsoft#29540 and microsoft#29481 forgot to rerun generate-ports.ps1.

Note that the default port-version was bumped because most everything depends on boost-build that change touched.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:community-triplet A PR or issue related to community triplets not officially validated by the vcpkg team. category:port-bug The issue is with a library, which is something the port should already support info:reviewed Pull Request changes follow basic guidelines
Projects
None yet
Development

Successfully merging this pull request may close these issues.

[boost-filesystem] Build error with emscripten
5 participants