Closed Bug 727880 Opened 12 years ago Closed 12 years ago

[km] remove accidental landing of language pack files

Categories

(Mozilla Localizations :: km / Khmer, defect)

Product:
Mozilla Localizations
Component:
km / Khmer
Type:
defect
Priority:
Not set
Severity:
normal

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: Pike, Unassigned)

References

Details

The export in attachment 576395 [details] in bug 551886 had a language pack unpacked in the root directory. I didn't see that when doing the review.

We should remove those files, notably:

hg rm chrome chrome.manifest install.rdf
I just removed all those files.
Yep, marking this bug FIXED. You should be able to resolve bugs, too, feel free to do that when you landed fixes. Usually good to also add a comment that refers to the changeset, it this case, that was http://hg.mozilla.org/releases/l10n/mozilla-aurora/km/rev/e58d43e6c278.

In the future, you can check in different parts in different changesets, and then push them all in one go.

So you could have done

hg ci -m"bug 727880, remove language pack files" chrome chrome.manifest install.rdf
hg ci -m"bug 709138, fixing a11y and change local code to km"

hg push

That way, changes are easier to review, but this one's dealt with now.
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.