Closed
Bug 760792
Opened 13 years ago
Closed 12 years ago
Port |Bug 756325 - channel-prefs.js appears in new 'preferences' location on fresh installation but remains in 'pref' for updated existing installation| to SeaMonkey
Categories
(SeaMonkey :: Build Config, defect, P3)
SeaMonkey
Build Config
Tracking
(seamonkey2.11 verified, seamonkey2.12 verified)
VERIFIED
FIXED
seamonkey2.13
People
(Reporter: sgautherie, Assigned: sgautherie)
References
Details
Attachments
(1 file, 1 obsolete file)
(deleted),
patch
|
Callek
:
review+
Callek
:
approval-comm-aurora+
Callek
:
approval-comm-beta+
|
Details | Diff | Splinter Review |
No description provided.
Flags: in-testsuite-
Assignee | ||
Comment 1•13 years ago
|
||
*Ftr, bug 756325 says 'removed-files.in' needs no update :-|
*Fwiw, FF and TB has (all) this in their /app/ dir,
yet I left it as is, in /suite/browser/.
*(Untested).
Attachment #629465 -
Flags: review?(bugspam.Callek)
Attachment #629465 -
Flags: approval-comm-aurora?
Assignee | ||
Updated•12 years ago
|
status-seamonkey2.12:
--- → affected
Target Milestone: seamonkey2.12 → seamonkey2.13
Assignee | ||
Updated•12 years ago
|
Attachment #629465 -
Flags: approval-comm-beta?
Updated•12 years ago
|
Attachment #629465 -
Flags: review?(bugspam.Callek)
Attachment #629465 -
Flags: review+
Attachment #629465 -
Flags: approval-comm-beta?
Attachment #629465 -
Flags: approval-comm-beta+
Attachment #629465 -
Flags: approval-comm-aurora?
Attachment #629465 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 2•12 years ago
|
||
Av1, plus bug 762588 port.
(I'll assume c-* approvals are still valid.)
NB:
I preferred to separate rather than use 'else', especially in 'Makefile.in' (wrt 'rules.mk').
Attachment #629465 -
Attachment is obsolete: true
Attachment #637064 -
Flags: review?(bugspam.Callek)
Assignee | ||
Comment 3•12 years ago
|
||
Ftr, patch Av1 succeeded as
https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=b820a0a17dab
Iiuc the logs (and .zip), channel-prefs.js is packaged at the correct place whether patch Av1 is applied or not.
Then, I wonder whether this file would be "already" (as in double) packaged by some other (shared?) code.
Anyway, let's just sync' (Av2) code ftb :-|
Comment 4•12 years ago
|
||
Comment on attachment 637064 [details] [diff] [review]
(Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 9 & 6]
Review of attachment 637064 [details] [diff] [review]:
-----------------------------------------------------------------
Nit: I do prefer the way Firefox did it http://mxr.mozilla.org/comm-central/source/mozilla/browser/app/Makefile.in#140 (with the else) but I won't block on it
Attachment #637064 -
Flags: review?(bugspam.Callek)
Attachment #637064 -
Flags: review+
Attachment #637064 -
Flags: approval-comm-beta+
Attachment #637064 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 5•12 years ago
|
||
Comment on attachment 637064 [details] [diff] [review]
(Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 9 & 6]
https://tbpl.mozilla.org/?tree=Thunderbird-Try&rev=37e2a400b4d9
[Mozilla/5.0 (Windows NT 5.1; rv:16.0) Gecko/16.0 Firefox/16.0 SeaMonkey/2.13a1 // Build identifier: 20120627041654] (MsVc8, sgautherie.bz@free.fr-67dbc42bd72d/try-comm-central-win32)
channel-prefs.js is still(!) packaged outside omni.ja only :-)
Comment 6•12 years ago
|
||
tracking-seamonkey2.11:
? → ---
Assignee | ||
Comment 7•12 years ago
|
||
Comment on attachment 637064 [details] [diff] [review]
(Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 9 & 6]
http://hg.mozilla.org/comm-central/rev/6d4fa81cc115
Attachment #637064 -
Attachment description: (Av2) Port bug 756325 and bug 762588 → (Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 6]
Assignee | ||
Updated•12 years ago
|
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: checkin-needed
Resolution: --- → FIXED
Whiteboard: [c-n: 6d4fa81cc115 to c-a]
Assignee | ||
Comment 8•12 years ago
|
||
/pub/seamonkey/candidates/2.11b5-candidates/build1/win32/en-US
seamonkey-2.11b5.zip
seamonkey2.11: verified.
Comment 9•12 years ago
|
||
Keywords: checkin-needed
Whiteboard: [c-n: 6d4fa81cc115 to c-a]
Assignee | ||
Updated•12 years ago
|
Attachment #637064 -
Attachment description: (Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 6] → (Av2) Port bug 756325 and bug 762588
[Checked in: Comment 7 & 9 & 6]
Assignee | ||
Comment 10•12 years ago
|
||
http://tbpl.drapostles.org/?rev=6d4fa81cc115
sgautherie.bz@free.fr – Sat Jul 7 13:52:03 2012
/pub/seamonkey/tinderbox-builds/comm-central-trunk-win32/1341661942
seamonkey-2.13a1.en-US.win32.zip
V.Fixed
http://tbpl.drapostles.org/?tree=SeaMonkey-Aurora&rev=9331aaae3574
ryanvm@gmail.com – Sat Jul 7 15:16:12 2012
/pub/seamonkey/tinderbox-builds/comm-aurora-win32/1341666982
seamonkey-2.12a2.en-US.win32.zip
seamonkey2.12: verified.
Status: RESOLVED → VERIFIED
You need to log in
before you can comment on or make changes to this bug.
Description
•