If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Include license for Russian Spellchecking dictionary in about:license

RESOLVED FIXED

Status

()

Firefox
General
RESOLVED FIXED
9 years ago
9 years ago

People

(Reporter: Alexander L. Slovesnik, Assigned: Alexander L. Slovesnik)

Tracking

({fixed1.9.0.2})

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

7.46 KB, patch
gerv
: review+
Samuel Sidler (old account; do not CC)
: approval1.9.0.2+
Details | Diff | Splinter Review
(Assignee)

Description

9 years ago
Created attachment 334709 [details] [diff] [review]
Patch

As discussed with Gerv on irc, license for upcoming Russian Spellchecking dictionary from Bug 389071 should be included in about:license.
License is taken from ftp://scon155.phys.msu.su/pub/russian/ispell/LICENSE
Attachment #334709 - Flags: review?(gerv)
Comment on attachment 334709 [details] [diff] [review]
Patch

Looks good to me.

Gerv
Attachment #334709 - Flags: review?(gerv) → review+
(Assignee)

Updated

9 years ago
Attachment #334709 - Flags: approval1.9.0.2?
Summary: Include license for Russian Spellchecking dictionary in about:license → [ru] Include license for Russian Spellchecking dictionary in about:license

Comment 2

9 years ago
Moving this into Firefox for general triage and landing.
Status: NEW → ASSIGNED
Component: ru / Russian → General
Keywords: checkin-needed
Product: Mozilla Localizations → Firefox
Summary: [ru] Include license for Russian Spellchecking dictionary in about:license → Include license for Russian Spellchecking dictionary in about:license
Comment on attachment 334709 [details] [diff] [review]
Patch

Approved for 1.9.0.2. Please land in CVS as soon as possible. a=ss

When you land this for 1.9.0.2, please add the "fixed1.9.0.2" keyword, but leave the bug open until it lands on mozilla-central.
Attachment #334709 - Flags: approval1.9.0.2? → approval1.9.0.2+
(Assignee)

Comment 4

9 years ago
Can someone help with landing on CVS and mozilla-central?
Whiteboard: [needs checkin on m-c and 1.9.0]
QA Contact: unghost → general

Comment 5

9 years ago
Pushed on mozilla-central, though only the toolkit part, as xpfe exists on neither mozilla-central nor comm-central.

http://hg.mozilla.org/mozilla-central/index.cgi/rev/b6045c8449b9
Whiteboard: [needs checkin on m-c and 1.9.0] → [needs checkin 1.9.0]

Comment 6

9 years ago
Landed on cvs trunk, too, both toolkit and xpfe.

Checking in toolkit/content/license.html;
/cvsroot/mozilla/toolkit/content/license.html,v  <--  license.html
new revision: 1.25; previous revision: 1.24
done
Checking in xpfe/global/resources/content/license.html;
/cvsroot/mozilla/xpfe/global/resources/content/license.html,v  <--  license.html
new revision: 1.22; previous revision: 1.21
done
Status: ASSIGNED → RESOLVED
Last Resolved: 9 years ago
Keywords: checkin-needed → fixed1.9.0.2
Resolution: --- → FIXED
Whiteboard: [needs checkin 1.9.0]
You need to log in before you can comment on or make changes to this bug.