Closed Bug 1195462 Opened 9 years ago Closed 9 years ago

Remove release-l10n mozconfigs as they are no longer use

Categories

(Thunderbird :: Build Config, defect)

39 Branch
defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED
Thunderbird 43.0

People

(Reporter: Fallen, Assigned: Fallen)

Details

Attachments

(1 file)

Looking at for example http://hg.mozilla.org/comm-central/filelog/ab6c50b76ed3/mail/config/mozconfigs/linux64/release-l10n it seems that we used this file for release repacks.

The current release-repacks use l10n-mozconfig as per http://hg.mozilla.org/build/tools/file/default/scripts/l10n/create-release-repacks.py#l279

So I think we can remove these files.
Attached patch remove-release-l10n.diff β€” β€” Splinter Review
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8648885 - Flags: review?(Pidgeot18)
Comment on attachment 8648885 [details] [diff] [review]
remove-release-l10n.diff

Review of attachment 8648885 [details] [diff] [review]:
-----------------------------------------------------------------

Okay, I've convinced myself that these are no longer used, but it wasn't your notes that did so. The line that added them in the config is still there, but the l10n_mozconfigs reference is no longer present in the beta config file.

For point of reference, these appear to only have ever been used on beta l10n builds, the logs of which I've been unable to track down. :-(
Attachment #8648885 - Flags: review?(Pidgeot18) → review+
Where in the configs is it still used? Then we should get rid of the config too if that is not referenced.

The logs for the last beta are here: http://ftp.mozilla.org/pub/mozilla.org/thunderbird/candidates/40.0b1-candidates/build1/logs/
Flags: needinfo?(Pidgeot18)
No longer blocks: 1191941
Pushed to comm-central changeset 303667f70b87
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 43.0
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: