Closed Bug 417529 Opened 12 years ago Closed 12 years ago

Release/News Announcement for 3.0.4, 3.1.4, 2.22.4, and 2.20.6

Categories

(Bugzilla :: bugzilla.org, defect, blocker)

defect
Not set
blocker

Tracking

()

RESOLVED FIXED

People

(Reporter: emmanuel, Assigned: mkanat)

References

Details

Attachments

(2 files, 4 obsolete files)

 
Assignee: website → cedric.corazza
Attached patch Patch for News/announcement (obsolete) — Splinter Review
Assuming that security bug 419188 also apply to 2.20 and 2.22 branches.
Attachment #311248 - Flags: review?(mkanat)
Comment on attachment 311248 [details] [diff] [review]
Patch for  News/announcement

No security release for 2.20 nor 2.22.
Attachment #311248 - Flags: review?(mkanat) → review-
Attached patch Patch for Release/news v2 (obsolete) — Splinter Review
Attachment #311248 - Attachment is obsolete: true
Attachment #311354 - Flags: review?(LpSolit)
Comment on attachment 311354 [details] [diff] [review]
Patch for Release/news v2

A quick thing I've spotted:

>+<p><a href="../releases/3.0.4/">Bugzilla 3.0.4</a> is the last stable release
>+  of the 3.0.x series. In addition to various bug fixes, it also contains a security
>+  fix discovered in 3.0.x and 3.1.x series.</p>


last -> latest as last implies to me that there won't be any more?
Comment on attachment 311354 [details] [diff] [review]
Patch for Release/news v2

>+<p><a href="../releases/3.0.4/">Bugzilla 3.0.4</a> is the last stable release

As Colin said, "last" should be "latest".


>+  of the 3.0.x series. In addition to various bug fixes, it also contains a security
>+  fix discovered in 3.0.x and 3.1.x series.</p>

You should specify that 3.0.4 contains many fixes in email_in.pl (creating and editing bugs per email) and many additions to the documentation.


>+<p><a href="../releases/3.2/">Bugzilla 3.1.4</a> also contains the 
>+  security fix, but just like 3.1.3, it has had massive changes and 
>+  received little or no testing. It should be considered unstable and 
>+  only used in testing environments.</p>

3.1.4 has much less massive changes than 3.1.3, and got more bug fixes in order to prepare 3.2 RC1. It also received some testing and is more stable than previous 3.1.x releases, but it still shouldn't be used in production yet, though. One hot spot is searches based on strings in comments and in bug summaries which should now be much faster. And SMTP authentication is also supported for the very first time.


>+Read the <a href="../security/3.0.4/">Security Advisory</a> for more
>+details.</p>

The URL is incorrect. It will be "../security/3.0.3/".
Attachment #311354 - Flags: review?(LpSolit) → review-
Attached patch Patch for Release/news v3 (obsolete) — Splinter Review
Patch addressing all the comments
Attachment #311354 - Attachment is obsolete: true
Attachment #311855 - Flags: review?(LpSolit)
Comment on attachment 311855 [details] [diff] [review]
Patch for Release/news v3

Looks good. r=LpSolit
Attachment #311855 - Flags: review?(LpSolit) → review+
Attached patch Patch for Release/news v4 (obsolete) — Splinter Review
Fixes to add the 2.20.x and 2.22.x series
Attachment #311855 - Attachment is obsolete: true
Attachment #317367 - Flags: review?(LpSolit)
Comment on attachment 317367 [details] [diff] [review]
Patch for Release/news v4

>+<h3>3.1.4</h3>

>+  contains the security fix for 3.0.x series and another one specific to 3.1.x
>+  series.

This string should be reworded. 3.1.4 has 3 security fixes, 3.0.4 has 2, and 2.20.6 and 2.22.4 both have only one. Also, you should add a section for 2.20.6 and 2.22.4 (a single section for both).
Attachment #317367 - Flags: review?(LpSolit) → review-
This one addresses latest LpSolit comments
Attachment #317367 - Attachment is obsolete: true
Attachment #319032 - Flags: review?(LpSolit)
Summary: Release/News Announcement for 3.0.4 and 3.1.4 → Release/News Announcement for 3.0.4, 3.1.4, 2.22.4, and 2.20.6
Comment on attachment 319032 [details] [diff] [review]
Patch for News v5

I added this one to the website with a few modifications.
Attachment #319032 - Flags: review?(LpSolit) → review+
Attachment #319032 - Attachment description: Patch for Release/news v5 → Patch for News v5
Attached file Release Announcement
Assignee: cedric.corazza → mkanat
Status: NEW → ASSIGNED
Attachment #319456 - Flags: review?(LpSolit)
Comment on attachment 319456 [details]
Release Announcement

>You can read the security advisory describing the fixed issues here:
>
>  http://www.bugzilla.org/security/3.0.4/

This URL is wrong. Must be http://www.bugzilla.org/security/2.20.5/. Otherwise looks good. r=LpSolit with the URL fixed.
Attachment #319456 - Flags: review?(LpSolit) → review+
  Okay, news posted (yesterday) and release announcement sent out (just now).
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.