Closed Bug 493773 Opened 13 years ago Closed 13 years ago

ignore module.ver for version bumps in release automation

Categories

(Release Engineering :: General, defect, P2)

x86
Windows Server 2003
defect

Tracking

(Not tracked)

RESOLVED FIXED

People

(Reporter: kairo, Assigned: bhearsum)

References

Details

Attachments

(1 file)

Bug 316387 made module.ver get version automatically from version.txt on both trunk and 1.9.1, so we can make release automation forget any logic for bumping version numbers there.
This has to get done before we spin 3.5rc1 or else the automation will fall down.
Assignee: nobody → bhearsum
Priority: -- → P2
The immediate thing that should be done is removing module.ver from the bumpFiles in release_master.py

We have a comment in buildbotcustom's process/factory.py that cites it, that's an easy fix.

Ultimately, in http://hg.mozilla.org/build/tools/file/34afe709615b/release/version-bump.pl#l75 we'll be able to remove all the code that knows how to deal with module.ver.

Overall, this is all pretty easy code removal. Do we run into an error right now if we can't replace a version number? I thought we would just not change it and therefore be relatively safe.
(In reply to comment #2)
> The immediate thing that should be done is removing module.ver from the
> bumpFiles in release_master.py
> 
> We have a comment in buildbotcustom's process/factory.py that cites it, that's
> an easy fix.
> 
> Ultimately, in
> http://hg.mozilla.org/build/tools/file/34afe709615b/release/version-bump.pl#l75
> we'll be able to remove all the code that knows how to deal with module.ver.
> 
> Overall, this is all pretty easy code removal. Do we run into an error right
> now if we can't replace a version number? I thought we would just not change it
> and therefore be relatively safe.

Yeah, we do hit an error. fwiw, I was just stating it to make sure it gets done - I think it's totally simple & safe enough to do this late.
Pretty self explanatory, I think.
Attachment #378411 - Flags: review?(nthomas)
Comment on attachment 378411 [details] [diff] [review]
remove module.ver from bumpFiles

Missed my automation test :-(
Attachment #378411 - Flags: review?(nthomas) → review+
Comment on attachment 378411 [details] [diff] [review]
remove module.ver from bumpFiles

changeset:   1150:d1e83fe8fab1
Attachment #378411 - Flags: checked‑in+
Status: NEW → RESOLVED
Closed: 13 years ago
Resolution: --- → FIXED
Should I file followups for process/factory.py and version-bump.pl changes?
If you don't mind filing a follow-up for version-bump.pl that would be great. I don't want to remove support for it until we do a couple releases with the new module.ver, though.

I'll get rid of the comment in factory.py
Blocks: 493955
Product: mozilla.org → Release Engineering
You need to log in before you can comment on or make changes to this bug.