Skip to content

Extract proguard logic to new rules#29641

Open
katre wants to merge 1 commit into
bazelbuild:masterfrom
katre:push-syywlltpvxtx
Open

Extract proguard logic to new rules#29641
katre wants to merge 1 commit into
bazelbuild:masterfrom
katre:push-syywlltpvxtx

Conversation

@katre
Copy link
Copy Markdown
Collaborator

@katre katre commented May 25, 2026

  • Timestamp clearing now uses a small hermetic python script instead of relying on non-hermetic unzip and zip.
  • Also split proguard and timestamp update actions for better caching.
  • Moved fastutil proguard spec into third_party where it is used.

RELNOTES: None.

@google-cla
Copy link
Copy Markdown

google-cla Bot commented May 25, 2026

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.

@github-actions github-actions Bot added the awaiting-review PR is awaiting review from an assigned reviewer label May 25, 2026
@katre
Copy link
Copy Markdown
Collaborator Author

katre commented May 25, 2026

The only test failure is an exec format error only on windows/arm64 (windows/x64 passes):

# Execution platform: //:default_host_platform
/usr/bin/bash: line 6: /c/b/4qcz3zc6/execroot/_main/external/rules_java++toolchains+remotejdk25_win_arm64/bin/jar: cannot execute binary file: Exec format error

This is pretty surprising: is there a misconfiguration with the windows/arm64 jdk?

@katre katre force-pushed the push-syywlltpvxtx branch from 96a1324 to 6f2c183 Compare May 25, 2026 14:42
@katre
Copy link
Copy Markdown
Collaborator Author

katre commented May 25, 2026

Hmm, adding a find call shows it really is an arm64 binary:

/usr/bin/bash: line 7: /c/b/k3n7yowv/execroot/_main/external/rules_java++toolchains+remotejdk25_win_arm64/bin/jar: cannot execute binary file: Exec format error
/c/b/k3n7yowv/execroot/_main/external/rules_java++toolchains+remotejdk25_win_arm64/bin/jar: PE32+ executable for MS Windows 6.02 (console), ARM64, 6 sections

@meteorcloudy , any ideas what's wrong here?

@katre katre requested a review from meteorcloudy May 25, 2026 14:46
@katre
Copy link
Copy Markdown
Collaborator Author

katre commented May 25, 2026

I'm also tempted to rewrite this as a python script, which would probably also fix the windows issue. I'll think about it and get back to this later.

- Timestamp clearing now uses a small hermetic python script instead of
  relying on non-hermetic `unzip` and `zip`.
- Also split proguard and timestamp update actions for better caching.
- Moved fastutil proguard spec into third_party where it is used.

RELNOTES: None.
@katre katre force-pushed the push-syywlltpvxtx branch from 6f2c183 to 0b5d43c Compare May 25, 2026 18:09
@katre katre changed the title Update fastutil_tripped to use hermetic jar instead of nonhermetic unzip Extract proguard logic to new rules May 25, 2026
@katre
Copy link
Copy Markdown
Collaborator Author

katre commented May 25, 2026

Went ahead and used the script, and added actual rules, and re-arranged things somewhat. PTAL.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

awaiting-review PR is awaiting review from an assigned reviewer

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant