Closed
Bug 792716
Opened 12 years ago
Closed 11 years ago
MPL 2 upgrade: l10n for bs
Categories
(Mozilla Localizations :: bs / Bosnian, defect)
Mozilla Localizations
bs / Bosnian
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: gerv, Assigned: Milos)
References
Details
Attachments
(2 files)
(deleted),
patch
|
Pike
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
gerv
:
review+
|
Details | Diff | Splinter Review |
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
Reporter | ||
Comment 1•12 years ago
|
||
Attachment #662920 -
Flags: review?
Reporter | ||
Updated•12 years ago
|
Assignee: gerv → nobody
Assignee | ||
Comment 2•12 years ago
|
||
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.
Comment 3•12 years ago
|
||
Sir, yes, sir!
Comment 4•12 years ago
|
||
So you have specific pointers so that I don't have to go through the full patch myself, too?
Assignee | ||
Comment 5•12 years ago
|
||
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.
Comment 6•12 years ago
|
||
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.
Assignee | ||
Comment 7•12 years ago
|
||
Ok. Aside from that, looks good.
Thanks Axel!
Comment 8•12 years ago
|
||
So, should I apply the patch? If yes, which -pnum should I use: -p0 or -p1?
Reporter | ||
Comment 9•12 years ago
|
||
Kerim: whichever setting you usually use in hg, whichever one works, whichever one finds the files. :-)
Gerv
Assignee | ||
Comment 10•12 years ago
|
||
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.
Reporter | ||
Comment 11•12 years ago
|
||
Kerim: have you been able to apply this patch?
Gerv
Reporter | ||
Comment 12•11 years ago
|
||
This is now fixed except for one file: ./dom/chrome/layout/htmlparser.properties . Kerim: can you fix that one by hand?
Gerv
Assignee | ||
Comment 13•11 years ago
|
||
This is a patch against mozilla-aurora.
Assignee: nobody → milos
Attachment #753157 -
Flags: review?(gerv)
Reporter | ||
Comment 14•11 years ago
|
||
Comment on attachment 753157 [details] [diff] [review]
MPL2 upgrade to htmlparser.properties
r=gerv.
Gerv
Attachment #753157 -
Flags: review?(gerv) → review+
Assignee | ||
Comment 15•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Updated•11 years ago
|
Attachment #662920 -
Flags: review? → review+
You need to log in
before you can comment on or make changes to this bug.
Description
•