Last Comment Bug 546487 - Remove 1.9.2 ifdefs from suite/ on comm-central
: Remove 1.9.2 ifdefs from suite/ on comm-central
Status: RESOLVED FIXED
:
Product: SeaMonkey
Classification: Client Software
Component: Build Config (show other bugs)
: Trunk
: All All
: -- trivial (vote)
: seamonkey2.1a1
Assigned To: Serge Gautherie (:sgautherie)
:
Mentors:
http://mxr.mozilla.org/comm-central/s...
Depends on: 521624 545534 NoC192SM
Blocks:
  Show dependency treegraph
 
Reported: 2010-02-16 12:45 PST by Robert Kaiser
Modified: 2010-02-19 22:09 PST (History)
1 user (show)
bugzillamozillaorg_serge_20140323: in‑testsuite-
See Also:
Crash Signature:
(edit)
QA Whiteboard:
Iteration: ---
Points: ---


Attachments
(Av1) The non-installer part [Checkin: Comment 5] (2.77 KB, patch)
2010-02-17 16:47 PST, Serge Gautherie (:sgautherie)
neil: review+
neil: superreview+
Details | Diff | Splinter Review
(Bv1) The installer part [Checkin: Comment 7] (6.40 KB, patch)
2010-02-19 20:17 PST, Serge Gautherie (:sgautherie)
bugspam.Callek: review+
Details | Diff | Splinter Review

Description Robert Kaiser 2010-02-16 12:45:46 PST
As we de-support 1.9.2 for SeaMonkey builds on comm-central, we should remove the ifdefs for that from the suite/ directory once bug 546484 has been solved (not before, so that we don't give people broken builds).
Comment 1 Serge Gautherie (:sgautherie) 2010-02-16 18:15:26 PST
Not too bad:
"Found 52 matching lines in 6 files"
which are 95% in /installer/.
Comment 2 Robert Kaiser 2010-02-17 05:33:14 PST
And a few of those are going away in bug 545534 actually, as Walter needed to touch a number of them in any case.
Comment 3 Serge Gautherie (:sgautherie) 2010-02-17 16:47:16 PST
Created attachment 427493 [details] [diff] [review]
(Av1) The non-installer part
[Checkin: Comment 5]
Comment 4 neil@parkwaycc.co.uk 2010-02-19 13:26:20 PST
Comment on attachment 427493 [details] [diff] [review]
(Av1) The non-installer part
[Checkin: Comment 5]

>- * MOZILLA_1_9_2_BRANCH
Nice of Ratty to make this easily discoverable ;-)
Comment 5 Serge Gautherie (:sgautherie) 2010-02-19 16:09:30 PST
Comment on attachment 427493 [details] [diff] [review]
(Av1) The non-installer part
[Checkin: Comment 5]


http://hg.mozilla.org/comm-central/rev/6eca7164d6b0
Comment 6 Serge Gautherie (:sgautherie) 2010-02-19 20:17:50 PST
Created attachment 427904 [details] [diff] [review]
(Bv1) The installer part
[Checkin: Comment 7]
Comment 7 Serge Gautherie (:sgautherie) 2010-02-19 22:09:09 PST
Comment on attachment 427904 [details] [diff] [review]
(Bv1) The installer part
[Checkin: Comment 7]


http://hg.mozilla.org/comm-central/rev/26f10666fd13

Note You need to log in before you can comment on or make changes to this bug.