Skip to content

Introduce libzip bcr.1 with a build fix for RPM-based systems #4934

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

Merged

Conversation

handicraftsman
Copy link
Contributor

@handicraftsman handicraftsman commented Jun 18, 2025

Fixes:

  1. [Bug]: Error when building libzip on RHEL9  #3100
  2. RPM Support & Workflow google/android-cuttlefish#1034 - unblocks this PR from building libzip

Ideally, rules_foreign_cc should handle CMake's lib64 related behavior itself, as it is the module that owns the CMake wrapping logic. Here, however, I am introducing these changes to resolve the existing libzip build issues.

Copy link

google-cla bot commented Jun 18, 2025

Thanks for your pull request! It looks like this may be your first contribution to a Google open source project. Before we can look at your pull request, you'll need to sign a Contributor License Agreement (CLA).

View this failed invocation of the CLA check for more information.

For the most up to date status, view the checks section at the bottom of the pull request.

@bazel-io
Copy link
Member

Hello @bazelbuild/bcr-maintainers, modules without existing maintainers (libzip) have been updated in this PR.
Please review the changes. You can view a diff against the previous version in the "Generate module diff" check.

@handicraftsman handicraftsman force-pushed the handicraftsman/fix-libzip-build branch from db66a39 to 93c1414 Compare June 18, 2025 16:53
@Wyverald Wyverald added the presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval label Jun 18, 2025
@Wyverald Wyverald enabled auto-merge (squash) June 18, 2025 22:43
@Wyverald Wyverald merged commit 1191b7b into bazelbuild:main Jun 18, 2025
20 checks passed
filmil pushed a commit to filmil/bazel-central-registry that referenced this pull request Jun 23, 2025
…uild#4934)

Fixes:
1. bazelbuild#3100
2. google/android-cuttlefish#1034 - unblocks
this PR from building libzip

Ideally, `rules_foreign_cc` should handle CMake's lib64 related behavior
itself, as it is the module that owns the CMake wrapping logic. Here,
however, I am introducing these changes to resolve the existing libzip
build issues.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
presubmit-auto-run Presubmit jobs will be triggered for new changes automatically without reviewer's approval
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants