-
Notifications
You must be signed in to change notification settings - Fork 86
Conversation
snazhmudinov
commented
Dec 2, 2025
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
taken from official Android documentation: https://developer.android.com/tools/apksigner?hl=en#options-sign-key-cert
snazhmudinov
commented
Dec 2, 2025
cc: @meisterT
ahumesky
commented
Dec 2, 2025
I'm not sure that this is good idea because it could appear to encourage checking passwords into source control since this goes into a build file. In general with bazel (and internally with blaze) signing outputs is handled outside of the build system (besides the debug key in the case of android).
snazhmudinov
commented
Dec 3, 2025
I'm not sure that this is good idea because it could appear to encourage checking passwords into source control since this goes into a build file. In general with bazel (and internally with blaze) signing outputs is handled outside of the build system (besides the debug key in the case of android).
Yes, I was having in mind the password for the debug keystore. If that's okay, where would the logic go?
1b7f9c1 to
91fecd5
Compare
snazhmudinov
commented
Dec 10, 2025
@ahumesky any thoughts on the last comment?
Addresses bazelbuild/bazel#10862