732963 - xulrunner shouldn't use pretty name for checksums files

Status: RESOLVED FIXED

(Firefox Build System :: General, defect, P2)

Description

[Rail Aliiev [:rail]]

Attachment: xulrunner checksums, no pretty names

We upload xulrunner deliverables for all platforms in one directory. Checksums files use pretty name schema and doesn't contain platform name in the file name. Checksums file name should have the same basename as PKG_BASENAME.

Comment 1

[Rail Aliiev [:rail]]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

http://hg.mozilla.org/integration/mozilla-inbound/rev/5774e4736140

Comment 2

[Marco Bonardo [:mak]]

https://hg.mozilla.org/mozilla-central/rev/5774e4736140

Comment 3

[Rail Aliiev [:rail]]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

This patch can be reproduced only in release builds and doesn't affect any CI builds. Worked fine in m-c and staging release builds.

Regression caused by (bug #): the current bug
Testing completed (on m-c, etc.): tested in dev environment by running staging releases 
Risk to taking this patch (and alternatives if risky): very low
String changes made by this patch: none

Comment 4

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

[Triage comment]
Approved low-risk, non-CI affecting, dev tested.

Comment 5

[Rail Aliiev [:rail]]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

http://hg.mozilla.org/releases/mozilla-beta/rev/0457b542fba7

Comment 6

[Rail Aliiev [:rail]]

Worked fine in 12.0b1 build2

Comment 7

[Rail Aliiev [:rail]]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

I would like to land this patch on m-r and esr10 branches. 12.0b1 release passed without any problems. The risk is very low since patch contains only cosmetic but needed by release automation changes.

Comment 8

[Lukas Blakk [:lsblakk] use ?needinfo]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

[Triage Comment]
sounds good, low risk, please go ahead and land.

Comment 9

[Rail Aliiev [:rail]]

Comment on attachment 602897 [details] [diff] [review]
xulrunner checksums, no pretty names

http://hg.mozilla.org/releases/mozilla-release/rev/3990fcb06900
http://hg.mozilla.org/releases/mozilla-esr10/rev/2b9b6e9e8af7