Closed Bug 792716 Opened 12 years ago Closed 11 years ago

MPL 2 upgrade: l10n for bs

Categories

(Mozilla Localizations :: bs / Bosnian, defect)

defect
Not set
normal

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: gerv, Assigned: Milos)

References

Details

Attachments

(2 files)

This language needs some or all of its l10n files to be upgraded to the MPL 2 licence. The current status in the aurora repo http://hg.mozilla.org/releases/l10n/mozilla-aurora/ is: Summary of Licenses in Files ============================ Number Percent License ------- -------- ----------- 203 74.91% <none found> 48 17.71% mpl/gpl/lgpl (standard block) 11 4.06% mpl2 9 3.32% <unknown license> ---------------------------- 271 files processed I have a script which can do the relicensing automatically. I will provide a patch here. Once a patch has been provided, please commit it as soon as possible. The aim is to have the relicensing finished by the end of September. Thanks :-) Gerv
Attached patch Patch v.1 (deleted) — Splinter Review
Attachment #662920 - Flags: review?
Assignee: gerv → nobody
Axel, this looks good to me, but it looks like this patch also removes whitespaces after some strings. Feedback? Kerim, please don't commit until Axel reviews this.
Sir, yes, sir!
So you have specific pointers so that I don't have to go through the full patch myself, too?
Yes. In Splinter Review(All files), search for browser/chrome/browser/devtools/webconsole.properties and see how whitespaces at the end of translations are removed. It's not just that file, but that's the pattern for all the changes I'm concerned about.
That seems to be an artifact of the splinter review display, the actual diff doesn't change those lines. Seems it's just highlighting trailing whitespace, judging by classnames.
Ok. Aside from that, looks good. Thanks Axel!
So, should I apply the patch? If yes, which -pnum should I use: -p0 or -p1?
Kerim: whichever setting you usually use in hg, whichever one works, whichever one finds the files. :-) Gerv
Kerim, looking at the raw diff, I'd say you'll want 'patch -p1 < patch.diff', but as Gerv said, try it to make sure.
Kerim: have you been able to apply this patch? Gerv
This is now fixed except for one file: ./dom/chrome/layout/htmlparser.properties . Kerim: can you fix that one by hand? Gerv
This is a patch against mozilla-aurora.
Assignee: nobody → milos
Attachment #753157 - Flags: review?(gerv)
Comment on attachment 753157 [details] [diff] [review] MPL2 upgrade to htmlparser.properties r=gerv. Gerv
Attachment #753157 - Flags: review?(gerv) → review+
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Attachment #662920 - Flags: review? → review+
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: