1517016 - Fix l10n for picking up new SM branding location

Status: RESOLVED FIXED

(SeaMonkey :: Build Config, enhancement)

Description

[Ian Neal]

At the moment l10n does not know where the SM branding is so files are missing from gecko-strings repo.

Comment 1

[Ian Neal]

Attachment: Update l10n files

Add suite/branding/seamonkey to relevant files as per TB version (Bug 1434667)

Comment 2

[Frank-Rainer Grahl (:frg)]

Something is wrong here with the merge. Need to look at it when I have more time.

> suite
>   branding/seamonkey
>     brand.dtd
>         // add and localize this file
>     brand.properties
>         // add and localize this file
>   chrome
>     branding
>       brand.dtd
>           // remove this file
>       brand.properties
>           // remove this file
>     common

Comment 3

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9033782 [details] [diff] [review]
Update l10n files

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

I basically goofed. Forgot about Bug 1451847 Part 3 for the branding and Bug 1378089 for the bookmarks shuffle around. So when the script removed the old files it was technically correct. 

We have imho 2 options:

1.) Wait for localizers to pick up the change (branding was onyl really used a little in de and ru) 
2.) Restore the removed files in a local copy, change the location to the correct one and push them to the l10n repos. 

Bookmarks would need additional changes and branding was mostly untranslated anyway so not sure if we should pursue this.

Comment 4

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9033782 [details] [diff] [review]
Update l10n files

From r- to r+ in a single move. Sorry this is essentially right :)

Also needed for 2.57.

Comment 5

[Pulsebot]

Pushed by [e-mail]:
https://hg.mozilla.org/comm-central/rev/e267cc10e397
Fix l10n for picking up new SeaMonkey branding location. r=frg

Comment 6

[Ian Neal]

Comment on attachment 9033782 [details] [diff] [review]
Update l10n files

a=me

Comment 7

[Frank-Rainer Grahl (:frg)]

Comment on attachment 9033782 [details] [diff] [review]
Update l10n files

https://hg.mozilla.org/releases/comm-esr60/rev/b41b63991f5cfab8797536be81c8ff0970dbbe67