Closed Bug 415261 Opened 16 years ago Closed 16 years ago

bootstrap updateverify config bump only works with rc1

Categories

(Release Engineering :: General, defect, P1)

defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: rhelmer, Assigned: rhelmer)

References

Details

Attachments

(1 file)

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: 16 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.