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

Prebuilt java_tools should have a filegroup that includes zipper #22889

Open
timothyg-stripe opened this issue Jun 25, 2024 · 2 comments
Open
Labels

Comments

@timothyg-stripe
Copy link
Contributor

timothyg-stripe commented Jun 25, 2024

Prebuilt java_tools zip files include the zipper binary:

❯ curl -L https://github.com/bazelbuild/java_tools/releases/download/java_13.6.1/java_tools_linux-v13.6.1.zip 2>/dev/null | tar -t | grep zipper
java_tools/ijar/zipper

This appears to be an intentional choice:

genrule(
name = "ijar_deploy_zip",
srcs = [
":ijar",
":zipper",
],

But the BUILD file that comes with prebuilt java_tools doesn't have a filegroup with the zipper binary, so we can't directly reference it, which is unfortunate.

@fmeum
Copy link
Collaborator

fmeum commented Jun 25, 2024

I think that it lives under @bazel_tools//tools/zip.

@timothyg-stripe
Copy link
Contributor Author

@fmeum @bazel_tools//tools/zip:zipper/zipper is only the current platform's binary. Ideally we'd like access to all platforms.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants