Closed Bug 415261 Opened 17 years ago Closed 17 years ago

bootstrap updateverify config bump only works with rc1

Categories

(Release Engineering :: General, defect, P1)

Product:
Release Engineering
Component:
General
Type:
defect

Tracking

(Not tracked)

Status:
RESOLVED FIXED

People

(Reporter: rhelmer, Assigned: rhelmer)

References

Details

Attachments

(1 file)

[checked in] remove previous config if present
(deleted), patch
nthomas
: review+
Details | Diff | Splinter Review
The current bootstrap Update->Verify() method only bumps the config file once per version, so it will not catch multiple RCs. When a previous version bump is detected, we should just clobber the version config, instead of skipping.
Assignee: nobody → rhelmer
Priority: -- → P2
Whiteboard: needs patch
Assignee: rhelmer → nobody
Whiteboard: needs patch
Assignee: nobody → rhelmer
Status: NEW → ASSIGNED
Priority: P3 → P1
We should get this in for beta4.
Blocks: 418926
This just removes the top two lines if the previous version string is present in the comment line.
Attachment #305855 - Flags: review?(nrthomas)
Whiteboard: waiting for review
Attachment #305855 - Flags: review?(nrthomas) → review+
Whiteboard: waiting for review
Comment on attachment 305855 [details] [diff] [review] [checked in] remove previous config if present Checking in Bootstrap/Step/Updates.pm; /cvsroot/mozilla/tools/release/Bootstrap/Step/Updates.pm,v <-- Updates.pm new revision: 1.37; previous revision: 1.36 done
Attachment #305855 - Attachment description: remove previous config if present → [checked in] remove previous config if present
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: