Closed
Bug 1314892
Opened 8 years ago
Closed 8 years ago
Provide allowed duplicate list for SeaMonkey
Categories
(SeaMonkey :: Build Config, defect)
SeaMonkey
Build Config
Tracking
(seamonkey2.48 unaffected, seamonkey2.49esr wontfix, seamonkey2.50 wontfix, seamonkey2.51 fixed, seamonkey2.52 fixed)
RESOLVED
FIXED
seamonkey2.52
Tracking | Status | |
---|---|---|
seamonkey2.48 | --- | unaffected |
seamonkey2.49esr | --- | wontfix |
seamonkey2.50 | --- | wontfix |
seamonkey2.51 | --- | fixed |
seamonkey2.52 | --- | fixed |
People
(Reporter: frg, Assigned: frg)
References
Details
Attachments
(3 files, 6 obsolete files)
(deleted),
patch
|
philip.chee
:
review+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
(deleted),
patch
|
iannbugzilla
:
review+
iannbugzilla
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
c-c compile is currently broken. Provide the list of allowed duplicated for all platforms to make find-dupes.py happy.
Assignee | ||
Updated•8 years ago
|
Blocks: 2.49BulkMalfunctions
Comment 1•8 years ago
|
||
this is the duplicate list for Linux.
Assignee | ||
Comment 2•8 years ago
|
||
Stefan,
can you provide the dupes list for OSX? c-c is currently broken.
Flags: needinfo?(stefanh)
Assignee | ||
Comment 3•8 years ago
|
||
This turns the dupes into a warning till the lists are ready.
Attachment #8807534 -
Flags: review?(iann_bugzilla)
Comment 4•8 years ago
|
||
(In reply to Frank-Rainer Grahl from comment #2)
> Stefan,
>
> can you provide the dupes list for OSX? c-c is currently broken.
What is a "dupes list" and how do I generate one?
Assignee | ||
Comment 5•8 years ago
|
||
make installer or make package should log the duplicates to the console. The one for TB is in mail/installer/allowed-dupes.mn. See bug 1313670. Without it find-dupes.py now errors out per default.
Comment 6•8 years ago
|
||
Comment on attachment 8807534 [details] [diff] [review]
1314892-interim.patch
> + -w -f $(MOZILLA_DIR)/browser/installer/allowed-dupes.mn \
Hmm if you use -w then -f is not needed right? Unless you are trying to reduce the console spam.
Attachment #8807534 -
Flags: review?(iann_bugzilla) → review+
Comment 7•8 years ago
|
||
Here's all the reported dupes. I was a little bit unsure of the format, so I didn't filter the paths.
Updated•8 years ago
|
Flags: needinfo?(stefanh)
Assignee | ||
Comment 8•8 years ago
|
||
https://hg.mozilla.org/comm-central/rev/09eeb66be9226c9e22f91ef1b2bb4b841ca93ce1
> Hmm if you use -w then -f is not needed right? Unless you are trying to reduce the console spam.
Acutally I wasn't sure if it wouldn't need at least one file and wanted to play it safe. Didn't have the patience to play around any longer with it :) Also it should now only report our duplicates.
Severity: blocker → normal
Assignee | ||
Updated•8 years ago
|
Blocks: 2.50BulkMalfunctions
Assignee | ||
Comment 9•8 years ago
|
||
Stefan could you probably apply the patch, run an OSX compile and upload the remaining once here or put them in an updated patch.
The toolkit backend code has a problem here with l10n again. Even after whitelisting the inspector locale files it will report GB and US as duplicate. Personally I would just leave the -w switch in place and not fix it there.
Assignee: nobody → frgrahl
Attachment #8807442 -
Attachment is obsolete: true
Attachment #8807698 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Attachment #8821520 -
Flags: feedback?(stefanh)
Assignee | ||
Comment 10•8 years ago
|
||
>> remaining once
remaining ones of course.
Comment 11•8 years ago
|
||
Comment on attachment 8821520 [details] [diff] [review]
1314892-SeaMonkey-dupes-WIP.patch
Hmm, lots of warnings. I'll attach a log so you can see how it looks.
Attachment #8821520 -
Flags: feedback?(stefanh)
Comment 12•8 years ago
|
||
Log of dupes when doing 'mach package' in suite/installer.
Assignee | ||
Comment 13•8 years ago
|
||
Stefan,
Merry Christmas. Could you try this list. I added the OSX specific files from your log and cleaned it up a bit. I am almost sure the the seamonkey.app prefix is not needed for most of them. This list should only report the DOMi en-GB and en-US l10n files as duplicate.
Attachment #8821520 -
Attachment is obsolete: true
Attachment #8821678 -
Attachment is obsolete: true
Attachment #8821777 -
Flags: feedback?(stefanh)
Comment 14•8 years ago
|
||
Comment on attachment 8821777 [details] [diff] [review]
1314892-SeaMonkey-dupes.patch
If I understand this right, I shouldn't see any warnings at all for files listed in suite/installer/allowed-dupes.mn? I don't really see any difference...
Attachment #8821777 -
Flags: feedback?(stefanh) → feedback-
Comment 15•8 years ago
|
||
Here's the log when packaging with the current patch.
Comment 16•8 years ago
|
||
(In reply to Frank-Rainer Grahl from comment #13)
> Stefan,
>
> Merry Christmas.
Merry Christmas!
Comment 17•8 years ago
|
||
Comment on attachment 8821777 [details] [diff] [review]
1314892-SeaMonkey-dupes.patch
Sorry, I must have missed something when applying the patch. There's 2 lines that needs to be changed, ten it works fine:
+SeaMonkey.app/Contents/MacOS/plugin-container.app/Contents/Resources/English.lproj/InfoPlist.strings
+SeaMonkey.app/Contents/Resources/en.lproj/InfoPlist.strings
Should be:
plugin-container.app/Contents/Resources/English.lproj/InfoPlist.strings
en.lproj/InfoPlist.strings
Attachment #8821777 -
Flags: feedback- → feedback+
Assignee | ||
Comment 18•8 years ago
|
||
-w needs to stay in for now because of the inspector locale files.
Attachment #8821777 -
Attachment is obsolete: true
Attachment #8821813 -
Attachment is obsolete: true
Attachment #8821823 -
Flags: review?(iann_bugzilla)
Attachment #8821823 -
Flags: approval-comm-aurora?
Comment 19•8 years ago
|
||
Comment on attachment 8821823 [details] [diff] [review]
1314892-SeaMonkey-dupes-V1.patch
r/a=me
Attachment #8821823 -
Flags: review?(iann_bugzilla)
Attachment #8821823 -
Flags: review+
Attachment #8821823 -
Flags: approval-comm-aurora?
Attachment #8821823 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 20•8 years ago
|
||
https://hg.mozilla.org/comm-central/rev/b2de81d657b6fe3c5f991e9f29ce0e3076272d0e
https://hg.mozilla.org/releases/comm-aurora/rev/4085badebe0a1745e4ccdd54627104cfbe4ce099
Set leave-open. The patch needs an small update because Lightning is installed under distribution/extentions there.
Keywords: leave-open
Assignee | ||
Comment 21•8 years ago
|
||
Followup for Lightning only.
IanN what to do about DOMi?
- Edit a comment into the GB or US locales so that they are different
- Try to fix it in a followup bug
- Move the locales to the l10n repositories (which would also theoretically help for the l10n build bug).
Attachment #8821923 -
Flags: review?(iann_bugzilla)
Attachment #8821923 -
Flags: approval-comm-aurora?
Comment 22•8 years ago
|
||
(In reply to Frank-Rainer Grahl from comment #21)
> Created attachment 8821923 [details] [diff] [review]
> 1314892-SeaMonkey-dupes-distri.patch
>
> Followup for Lightning only.
>
> IanN what to do about DOMi?
>
> - Edit a comment into the GB or US locales so that they are different
> - Try to fix it in a followup bug
> - Move the locales to the l10n repositories (which would also theoretically
> help for the l10n build bug).
Should Lightning have its own dupes list rather than TB / SM each having one for it?
I already have a patch for moving locales to l10n repos (and doing some flattening), just wanted to check the best way of achieving it with flod or pike.
Assignee | ||
Comment 23•8 years ago
|
||
>> Should Lightning have its own dupes list rather than TB / SM each having one for it?
Was on my mind too and planned to open a bug in calendar next.
Fallen do you concur that Lightning should get its own dupes file which can be included by TB and SeaMonkey then?
FRG
Flags: needinfo?(philipp)
Comment 24•8 years ago
|
||
Comment on attachment 8821923 [details] [diff] [review]
1314892-SeaMonkey-dupes-distri.patch
Review of attachment 8821923 [details] [diff] [review]:
-----------------------------------------------------------------
::: suite/installer/allowed-dupes.mn
@@ +267,5 @@
> chrome/toolkit/skin/classic/global/media/throbber.png
> extensions/modern@themes.mozilla.org/chrome/modern/skin/modern/communicator/brand/throbber-anim.png
> chrome/classic/skin/classic/communicator/brand/throbber-anim.png
> +
> +# Lightning Nighly
nit: Nightly
Assignee | ||
Updated•8 years ago
|
Blocks: 2.51BulkMalfunctions
Assignee | ||
Updated•8 years ago
|
Blocks: 2.52BulkMalfunctions
Comment 25•8 years ago
|
||
Comment on attachment 8821923 [details] [diff] [review]
1314892-SeaMonkey-dupes-distri.patch
r/a=me still needs an answer from Fallen though
Attachment #8821923 -
Flags: review?(iann_bugzilla)
Attachment #8821923 -
Flags: review+
Attachment #8821923 -
Flags: approval-comm-aurora?
Attachment #8821923 -
Flags: approval-comm-aurora+
Comment 26•8 years ago
|
||
(In reply to Frank-Rainer Grahl from comment #23)
> Fallen do you concur that Lightning should get its own dupes file which can
> be included by TB and SeaMonkey then?
Sure, sounds good to me. If it is just the few files as it is right now, I think its not worth too much extra effort through. The goal is to consolidate the themes in a way that we only need one xpi anyway.
Flags: needinfo?(philipp)
Assignee | ||
Comment 27•8 years ago
|
||
Took the liberty and pushed it to beta because aurora merged since the approve:
https://hg.mozilla.org/comm-central/rev/9a8875e3bff1e9f6cd2ba8e1745d722addce4de2
https://hg.mozilla.org/releases/comm-beta/rev/adb2af9a87b6321f1b0fc16320d9996ae0715ff6
If I find some time (no promise) I will look at Bug 1317630 to get the Calendar dupes out of mail and suite.
status-seamonkey2.51:
--- → fixed
status-seamonkey2.52:
--- → fixed
Target Milestone: --- → seamonkey2.52
Assignee | ||
Updated•8 years ago
|
status-seamonkey2.50:
--- → wontfix
Assignee | ||
Updated•8 years ago
|
You need to log in
before you can comment on or make changes to this bug.
Description
•