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

[libb2] Fix Windows build #42925

Closed
wants to merge 2 commits into from
Closed

Conversation

paperchalice
Copy link

  • Changes comply with the maintainer guide.
  • SHA512s are updated for each updated download.
  • The "supports" clause reflects platforms that may be fixed by this new version.
  • Any fixed CI baseline entries are removed from that file.
  • Any patches that are no longer applied are deleted from the port's directory.
  • The version database is fixed by rerunning ./vcpkg x-add-version --all and committing the result.
  • Only one version is added to each modified port's versions file.

@paperchalice paperchalice force-pushed the libb2 branch 2 times, most recently from 310ae2d to 54a15bc Compare December 25, 2024 12:15
@jimwang118 jimwang118 added the category:port-bug The issue is with a library, which is something the port should already support label Dec 26, 2024
@jimwang118
Copy link
Contributor

It is recommended that the patches be submitted to the upstream at the same time, and the upstream will adapt to Windows when the version is updated later.

Comment on lines +13 to +15
set(pre_run PRERUN_SHELL
"sed -i 's/cross_compiling=yes/cross_compiling=no/g' configure"
)
Copy link
Contributor

Choose a reason for hiding this comment

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

This can be done in the portfile with vcpkg_replace_string, but...
... changing configure before AUTOCONFIG is pointless.

Copy link
Author

Choose a reason for hiding this comment

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

libb2 doesn't ship configure in release tarball... and I coudn't fix error in configure when cross compile from x64 to x86 in CI, although it works fine in may local machine.

Copy link
Contributor

Choose a reason for hiding this comment

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

Enabling Windows without patch in #42952.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-bug The issue is with a library, which is something the port should already support
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants