Update "Changes" Page for 2.16.11, 2.18.5, 2.20.1, and 2.22rc1

RESOLVED FIXED

Status

()

Bugzilla
bugzilla.org
--
blocker
RESOLVED FIXED
13 years ago
13 years ago

People

(Reporter: Max Kanat-Alexander, Assigned: wicked)

Tracking

Details

Attachments

(1 attachment, 3 obsolete attachments)

(Reporter)

Description

13 years ago
I'll probably end up doing this if nobody else does, but I always appreciate a patch! :-)
(Reporter)

Updated

13 years ago
Summary: Update "Changes" Page for 2.16.11, 2.18.5, 2.20.1, and 2.21.2 → Update "Changes" Page for 2.16.11, 2.18.5, 2.20.1, and 2.22rc1
(Assignee)

Comment 1

13 years ago
Created attachment 208195 [details] [diff] [review]
Changes to changes.html.tmpl, V1
Assignee: nobody → wicked
Status: NEW → ASSIGNED
Attachment #208195 - Flags: review?(mkanat)
(Reporter)

Comment 2

13 years ago
Comment on attachment 208195 [details] [diff] [review]
Changes to changes.html.tmpl, V1

Yeah, looks good, but instead of "XX" put "FIXME" -- that's what I grep through the entire site for before launching it.
Attachment #208195 - Flags: review?(mkanat) → review-
(Assignee)

Comment 3

13 years ago
Created attachment 208728 [details] [diff] [review]
Changes to changes.html.tmpl, V1.0.0.0.0.1
Attachment #208195 - Attachment is obsolete: true
Attachment #208728 - Flags: review?(mkanat)
(Reporter)

Comment 4

13 years ago
Comment on attachment 208728 [details] [diff] [review]
Changes to changes.html.tmpl, V1.0.0.0.0.1

We're going to be branching, so you'll need a 2.22 branch now, also.
Attachment #208728 - Flags: review?(mkanat) → review-
(Assignee)

Comment 5

13 years ago
Created attachment 208848 [details] [diff] [review]
Changes to changes.html.tmpl, V2

Here you go. Also closed 2.16 branch which I believe happens when we branch for 2.22rc1.
Attachment #208728 - Attachment is obsolete: true
Attachment #208848 - Flags: review?(mkanat)

Comment 6

13 years ago
(In reply to comment #5)
> Here you go. Also closed 2.16 branch which I believe happens when we branch for
> 2.22rc1.

Is that true? AFAIK, we will close the 2.16 branch when 2.22 is out, not 2.22rc1. justdave?
nope, we've been telling people "when 2.22 comes out".  That means 2.16 doesn't close until 2.22 is final.

Comment 8

13 years ago
just for fun, note that we will soon maintain 5 branches:

2.16, 2.18, 2.20, 2.22 and tip. Happy? :-D

/me doesn't want to be the one who will have to write 5 patch + backports. ;)
(Assignee)

Comment 9

13 years ago
Created attachment 208868 [details] [diff] [review]
Changes to changes.html.tmpl, V2.1

Hmm, maybe I confused killing 2.16 branch to unfreezing trunk when we branch for 2.22 or something. This also fixes a date mismatch in the 2.16.10 release.
Attachment #208848 - Attachment is obsolete: true
Attachment #208868 - Flags: review?(mkanat)
Attachment #208848 - Flags: review?(mkanat)

Comment 10

13 years ago
I think we should remove the 2.14 branch now. Keeping them all doesn't make sense. And some of us now have to scroll horizontally to see all columns.
(Reporter)

Comment 11

13 years ago
OK. Either remove the 2.14 branch or put it in a table that's vertically below the current table. You can call the lower table "Legacy Branches" or something like that.
(Assignee)

Comment 12

13 years ago
(In reply to comment #10)
> I think we should remove the 2.14 branch now. Keeping them all doesn't make
> sense. And some of us now have to scroll horizontally to see all columns.

Umm, I don't see the point. If you don't want to see the 2.14 branch then just don't scroll.

Comment 13

13 years ago
(In reply to comment #12)
> Umm, I don't see the point. If you don't want to see the 2.14 branch then just
> don't scroll.
> 

There is no reason to keep dead branches, except maybe the very last one (in our case: 2.16). After all, we didn't keep 2.12 and older.
(Reporter)

Comment 14

13 years ago
Comment on attachment 208868 [details] [diff] [review]
Changes to changes.html.tmpl, V2.1

Looks good! :-) I reduced the number of arrows in the 2.22 column by one--it was wrapping.
Attachment #208868 - Flags: review?(mkanat) → review+
(Reporter)

Comment 15

13 years ago
Checked-in as part of bug 320306. Thanks for the patch! :-)
Status: ASSIGNED → RESOLVED
Last Resolved: 13 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.