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

Allow decompression of nupkg #23645

Closed
wants to merge 1 commit into from

Conversation

dpvdberg
Copy link
Contributor

In some (legacy) projects, nuget is used to handle C++ packages.
Nuget packages are working fine with WORKSPACE http_archive directives when explicitly specifying type = "zip".

However, when migrating to bzlmod, we ran into issues when the url in source.json is a nupkg file.
This PR resolves that issue and allows consumption of nupkg archives as Bazel modules.

@github-actions github-actions bot added team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file. awaiting-review PR is awaiting review from an assigned reviewer labels Sep 17, 2024
@dpvdberg
Copy link
Contributor Author

Any chance this might still land in 7.4.0?

@bazel-io fork 7.4.0

Copy link
Collaborator

@fmeum fmeum left a comment

Choose a reason for hiding this comment

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

@meteorcloudy meteorcloudy added awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally and removed awaiting-review PR is awaiting review from an assigned reviewer labels Sep 27, 2024
@meteorcloudy
Copy link
Member

@bazel-io fork 7.4.0

@github-actions github-actions bot removed the awaiting-PR-merge PR has been approved by a reviewer and is ready to be merge internally label Sep 30, 2024
bazel-io pushed a commit to bazel-io/bazel that referenced this pull request Sep 30, 2024
In some (legacy) projects, nuget is used to handle C++ packages.
Nuget packages are working fine with WORKSPACE `http_archive` directives when explicitly specifying `type = "zip"`.

However, when migrating to bzlmod, we ran into issues when the url in source.json is a nupkg file.
This PR resolves that issue and allows consumption of nupkg archives as Bazel modules.

Closes bazelbuild#23645.

PiperOrigin-RevId: 680636414
Change-Id: Ia7fd3fc738a2dd68b1063dba0dc847c5b0668401
github-merge-queue bot pushed a commit that referenced this pull request Oct 1, 2024
In some (legacy) projects, nuget is used to handle C++ packages.
Nuget packages are working fine with WORKSPACE `http_archive` directives
when explicitly specifying `type = "zip"`.

However, when migrating to bzlmod, we ran into issues when the url in
source.json is a nupkg file.
This PR resolves that issue and allows consumption of nupkg archives as
Bazel modules.

Closes #23645.

PiperOrigin-RevId: 680636414
Change-Id: Ia7fd3fc738a2dd68b1063dba0dc847c5b0668401

Commit
b387036

Co-authored-by: Dennis van den Berg <[email protected]>
@iancha1992
Copy link
Member

The changes in this PR have been included in Bazel 7.4.0 RC1. Please test out the release candidate and report any issues as soon as possible.
If you're using Bazelisk, you can point to the latest RC by setting USE_BAZEL_VERSION=7.4.0rc1. Thanks!

@dpvdberg dpvdberg deleted the add-nupkg-support branch October 21, 2024 14:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
team-ExternalDeps External dependency handling, remote repositiories, WORKSPACE file.
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants