Closed
Bug 669504
Opened 13 years ago
Closed 13 years ago
Port bug 649373 - remove the PRE_RELEASE_SUFFIX so that nightly/aurora builds don't have Alpha in their window titles
Categories
(Thunderbird :: Mail Window Front End, defect)
Thunderbird
Mail Window Front End
Tracking
(thunderbird6+ fixed, thunderbird7+ fixed)
RESOLVED
FIXED
Thunderbird 8.0
People
(Reporter: standard8, Assigned: standard8)
Details
Attachments
(1 file)
1.77 KB,
patch
|
Bienvenu
:
review+
standard8
:
approval-comm-aurora+
|
Details | Diff | Splinter Review |
From bug 649373: (In reply to comment #0) > Aurora blank windows have "Alpha 2" in the title. Aurora will always have > [version]a2 as its version. We should remove the "Alpha 2" (In reply to comment #3) > My understanding from #developers is that the actual summary should be "Drop > the whole PRE_RELEASE_SUFFIX thing." > > mozilla-central will always be n.0a1, never pre, and never released or > branded, so a screenshot with "Nightly" in the titlebar can't be confused > with anything; mozilla-aurora will always be n.0a2, never pre, never > branded, and we're fine with having no way to tell the difference between a > screenshot of a shipped Aurora and a tinderbox-build Aurora; mozilla-beta > will always be n.0, never pre, always branded, and we're fine with having no > way to tell the difference between a screenshot of the very first > tinderbox-build from mozilla-beta and the final release of Firefox n.0. I think it is reasonable that we do the same for Thunderbird - it is a bit strange that it says Alpha 1/Alpha 2 when it is always that according to the branch, and the branding will tell us the difference between branches anyway.
Assignee | ||
Comment 1•13 years ago
|
||
This blanks the pre-release suffix like Firefox did. The removals in jquery land are just because jquery doesn't get preprocessed at all, so it is pointless putting additional pre-process variables in there.
Attachment #544190 -
Flags: review?(bwinton)
Assignee | ||
Comment 2•13 years ago
|
||
Comment on attachment 544190 [details] [diff] [review] The fix Blake said he's happy with the idea, and I'd like to get this hitting the branches fairly soon, especially as we can't do first beta until we get it.
Attachment #544190 -
Flags: review?(bwinton) → review?(dbienvenu)
Updated•13 years ago
|
Attachment #544190 -
Flags: review?(dbienvenu) → review+
Assignee | ||
Comment 3•13 years ago
|
||
Checked in: http://hg.mozilla.org/comm-central/rev/3ac94faeeb55
Status: NEW → RESOLVED
Closed: 13 years ago
Flags: in-testsuite-
Resolution: --- → FIXED
Target Milestone: --- → Thunderbird 8.0
Assignee | ||
Updated•13 years ago
|
Attachment #544190 -
Flags: approval-comm-beta+
Attachment #544190 -
Flags: approval-comm-aurora+
Assignee | ||
Comment 4•13 years ago
|
||
This got checked into aurora and beta a while ago: http://hg.mozilla.org/releases/comm-aurora/rev/b3603e475d44 http://hg.mozilla.org/releases/comm-beta/rev/c8aad432778a
status-thunderbird6:
--- → fixed
status-thunderbird7:
--- → fixed
You need to log in
before you can comment on or make changes to this bug.
Description
•