Closed
Bug 479131
Opened 16 years ago
Closed 16 years ago
ReleaseBuildFactory and ReleaseRepackFactory need to override MOZ_PKG_VERSION
Categories
(Release Engineering :: General, defect, P2)
Tracking
(Not tracked)
RESOLVED
DUPLICATE
of bug 464014
People
(Reporter: bhearsum, Assigned: bhearsum)
References
Details
Attachments
(1 file)
I just noticed that we don't override MOZ_PKG_VERSION at all in either of these factories. Most of the time this fine, because the version set in release_config.py is the same as the one in browser/config/version.txt, but when we get into RCs these will differ. We simply need to set MOZ_PKG_VERSION in the environment during packaging to fix this.
Comment 1•16 years ago
|
||
You mean use the likes of 3.1rc1 from appVersion in release_config, rather than the 3.1 in version.txt ?
Assignee | ||
Comment 2•16 years ago
|
||
I think version = 3.1rc1 and appVersion = 3.1, actually, but yes. (http://bonsai.mozilla.org/cvsblame.cgi?file=mozilla/tools/release/configs/fx-moz19-bootstrap.cfg&rev=1.27)
Comment 3•16 years ago
|
||
Ah yes, what you said.
Assignee | ||
Comment 4•16 years ago
|
||
Note that this will set MOZ_PKG_VERSION for any step that uses self.env. Since we manually bump version.txt there shouldn't be any way for this to infect that file. I did a quick test by doing:
export MOZ_PKG_VERSION="3.1rc1"
make package
and the package did indeed get named as 3.1rc1, and the internal version was still the version listed in version.txt. (3.1b3pre in this case, because we don't have any plain '3.1' trees).
Attachment #363122 -
Flags: review?(nthomas)
Assignee | ||
Updated•16 years ago
|
Priority: -- → P2
Assignee | ||
Comment 5•16 years ago
|
||
Argh, I just realized that the appVersion patch which is waiting to land does all of this already.
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → DUPLICATE
Assignee | ||
Updated•16 years ago
|
Attachment #363122 -
Flags: review?(nthomas)
Updated•12 years ago
|
Product: mozilla.org → Release Engineering
You need to log in
before you can comment on or make changes to this bug.
Description
•