Closed Bug 876508 Opened 7 years ago Closed 6 years ago

Remove l10n for Sunbird

Categories

(Calendar :: Build Config, defect)

defect
Not set

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: jcranmer, Assigned: Fallen)

References

Details

Attachments

(2 files)

+++ This bug was initially created as a clone of Bug #866903 +++

(In reply to Joshua Cranmer [:jcranmer] from comment #7)
> I think calendar/locales/en-US/{chrome/sunbird, installer, updater} can be
> removed, but l10n is so far removed from my area of expertise that I'm not
> going to try unless I'm told to.

I also think these can go. Please file a followup bug if you don't take care here. It might not be that bad to do this in the next cycle so that the string "changes" are not required for the upcoming ESR.
This patch takes care of removing the l10n files. I hope I got every bit of it. I've also filed bug 978591 to get MOZ_CHROME_FILE_FORMAT out of the Makefile, then we can remove it altogether.
Assignee: nobody → philipp
Status: NEW → ASSIGNED
Attachment #8384305 - Flags: review?(mschroeder)
This patch removes some unused .dtd entries. Not all of them will be related to Sunbird, but its hard to differ.

I found them using a combination of grep calls. I didn't bother to search through our .properties file entries, because its really a pain and is at least a half day of risky manual work.
Attachment #8384306 - Flags: review?(mschroeder)
Comment on attachment 8384306 [details] [diff] [review]
Remove unused dtd entries  - v1

r=mschroeder
Attachment #8384306 - Flags: review?(mschroeder) → review+
Comment on attachment 8384305 [details] [diff] [review]
Remove l10n files - v1

r=mschroeder, although I can not tell if the l10n Makefiles are okay after the cleanup. Please have another look at the Makefile target around https://mxr.mozilla.org/comm-central/source/calendar/lightning/lightning-packager.mk#157, because you remove the defines.inc in this patch, which seems to be referenced there.
Attachment #8384305 - Flags: review?(mschroeder) → review+
Thanks for the hint. I've made some slight modifications to not process defines.inc.
Pushed to comm-central changeset 0c6c07afee38
Status: ASSIGNED → RESOLVED
Closed: 6 years ago
Resolution: --- → FIXED
Target Milestone: --- → 3.2
You need to log in before you can comment on or make changes to this bug.