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

windows tar toolchain gzip errors #968

Open
jbedard opened this issue Oct 18, 2024 · 8 comments
Open

windows tar toolchain gzip errors #968

jbedard opened this issue Oct 18, 2024 · 8 comments
Labels
bug Something isn't working

Comments

@jbedard
Copy link
Collaborator

jbedard commented Oct 18, 2024

See https://github.com/aspect-build/rules_js/actions/runs/11391002935/job/31694034057?pr=1989#step:7:456

(18:41:49) ERROR: D:/a/rules_js/rules_js/e2e/bzlmod/BUILD.bazel:14:22: Extracting npm package [email protected] failed: (Exit 1): tar.exe failed: error executing command (from target //:.aspect_rules_js/node_modules/[email protected]/pkg) external\aspect_bazel_lib~2.9.0~toolchains~bsd_tar_windows_amd64\tar.exe --extract --no-same-owner --no-same-permissions --strip-components 1 --file ... (remaining 3 arguments skipped)
tar.exe: Error opening archive: Can't initialize filter; unable to run program "gzip -d"
@thesayyn
Copy link
Collaborator

Turns out we don't link any of the [de]compression filters for windows builds, they were simply shelling out to system binaries, which did not exist in this case.

This is where we set these for mac and linux

https://github.com/bazelbuild/bazel-central-registry/blob/ef34e6bfad5a6ab54080ddcc83a4d65849855e3a/modules/libarchive/3.7.5/patches/build_with_bazel.patch#L1510-L1522

but not for windows

https://github.com/bazelbuild/bazel-central-registry/blob/ef34e6bfad5a6ab54080ddcc83a4d65849855e3a/modules/libarchive/3.7.5/patches/build_with_bazel.patch#L1548

@thesayyn thesayyn added the bug Something isn't working label Oct 21, 2024
@thesayyn
Copy link
Collaborator

Needs to be fixed in libarchive in BCR before it can be fixed here.

@thesayyn
Copy link
Collaborator

thesayyn commented Nov 1, 2024

Related #848

@thesayyn
Copy link
Collaborator

@jbedard bazelbuild/bazel-central-registry#3225 is going to fix this but there #952 that needs to be fixed before we can roll this

devversion added a commit to devversion/angular-cli that referenced this issue Nov 26, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Nov 26, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Nov 27, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
@devversion
Copy link

Is there any way we can help to unblock this? this prevents adoption of rules_js on our end as we run Windows jobs

devversion added a commit to devversion/angular-cli that referenced this issue Nov 27, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Nov 27, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
@thesayyn
Copy link
Collaborator

@devversion i have fixed this upstream BCR already. I just need to go around and cut some releases to have the fix in prebuilt toolchains.

@devversion
Copy link

@thesayyn thanks for doing that! isn't #952 still a blocker?

@thesayyn
Copy link
Collaborator

Potentially, unfortunately there is no clear repro for me to confirm so it requires some digging to repro and make sure it's fixed.

devversion added a commit to devversion/angular-cli that referenced this issue Nov 29, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
devversion added a commit to devversion/angular-cli that referenced this issue Dec 2, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 3, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 3, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 11, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 11, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 11, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 13, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 13, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
josephperrott pushed a commit to devversion/angular-cli that referenced this issue Dec 16, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
clydin pushed a commit to angular/angular-cli that referenced this issue Dec 16, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
clydin pushed a commit to angular/angular-cli that referenced this issue Dec 16, 2024
Workaround until `rules_js` imports the latest version of
`aspect_bazel_lib`: bazel-contrib/bazel-lib#968
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

No branches or pull requests

3 participants