Closed
Bug 481703
Opened 16 years ago
Closed 16 years ago
Use python to create pre release suffix
Categories
(Firefox Build System :: General, defect, P1)
Firefox Build System
General
Tracking
(Not tracked)
RESOLVED
FIXED
mozilla3.5b4
People
(Reporter: robert.strong.bugs, Assigned: robert.strong.bugs)
References
Details
Attachments
(1 file, 4 obsolete files)
(deleted),
patch
|
robert.strong.bugs
:
review+
|
Details | Diff | Splinter Review |
Spinoff of bug 481680.
As bsmedberg stated, python or perl is more reliable / consistent when doing this on all platforms.
Assignee | ||
Comment 1•16 years ago
|
||
The tests are there but they need some additional work / cleanup
Assignee | ||
Updated•16 years ago
|
Attachment #365706 -
Attachment description: patch in progress → patch in progress (bitrotted)
Updated•16 years ago
|
Assignee: nobody → robert.bugzilla
Status: NEW → ASSIGNED
OS: Windows Vista → All
Hardware: x86 → All
Comment 3•16 years ago
|
||
Marking blocking as we'll need this for the next beta; alternative is to unblock on this and remember to file a blocking bug on manually overriding the OSX title change every time.
Flags: blocking-firefox3.1+
Priority: -- → P2
Comment 4•16 years ago
|
||
--> P1, as this bug will require the wider feedback of a beta release or is of sufficient complexity that we should be looking at it sooner, not later.
Priority: P2 → P1
Assignee | ||
Comment 5•16 years ago
|
||
Benjamin, I went with something a little more complex in order to restrict creating the pre-release suffix to version strings that only contain 0-9, periods, and a single 'a' or a single 'b' since version strings can be of the format 1.1aa per
https://developer.mozilla.org/En/Toolkit_version_format
Attachment #365706 -
Attachment is obsolete: true
Attachment #367279 -
Flags: review?(benjamin)
Assignee | ||
Updated•16 years ago
|
Target Milestone: --- → Firefox 3.1b3
Assignee | ||
Comment 6•16 years ago
|
||
Attachment #367279 -
Attachment is obsolete: true
Attachment #367545 -
Flags: review?(benjamin)
Attachment #367279 -
Flags: review?(benjamin)
Updated•16 years ago
|
Whiteboard: [needs review bsmedberg]
Comment 7•16 years ago
|
||
Comment on attachment 367545 [details] [diff] [review]
patch rev2 - removed redundant check and added a couple of tests
>+++ b/config/printprereleasesuffix.py
>+def get_prerelease_suffix(version):
I think this can be simpler and easier to read:
def mfunc(m):
return "%s %s %s" % (m.group('prefix'),
{'a': 'Alpha', 'b': 'Beta'}[m.group('c')],
m.group('suffix'))
result, c = re.subn(r'^(?P<prefix>(\d+\.)*\d+)(?P<c>[ab])(?P<suffix>\d+)$',
mfunc, version)
if c != 1:
return ''
return result
Attachment #367545 -
Flags: review?(benjamin) → review-
Assignee | ||
Comment 8•16 years ago
|
||
Attachment #367545 -
Attachment is obsolete: true
Attachment #368014 -
Flags: review?(benjamin)
Updated•16 years ago
|
Attachment #368014 -
Flags: review?(benjamin) → review+
Comment 9•16 years ago
|
||
Oh, it would be better to do := instead of = in the makefile, so that we only run the python script once.
Assignee | ||
Comment 10•16 years ago
|
||
Attachment #368014 -
Attachment is obsolete: true
Attachment #368310 -
Flags: review+
Assignee | ||
Comment 11•16 years ago
|
||
pushed to mozilla-central
http://hg.mozilla.org/mozilla-central/rev/ae69c2f236cd
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Whiteboard: [needs review bsmedberg]
Assignee | ||
Updated•16 years ago
|
Target Milestone: Firefox 3.1b3 → Firefox 3.5b4
Assignee | ||
Comment 12•16 years ago
|
||
Pushed to mozilla-1.9.1
http://hg.mozilla.org/releases/mozilla-1.9.1/rev/f72fc9fafde7
Keywords: fixed1.9.1
Assignee | ||
Updated•16 years ago
|
Flags: in-testsuite+
Updated•6 years ago
|
Component: Build Config → General
Product: Firefox → Firefox Build System
Updated•6 years ago
|
Keywords: fixed1.9.1
Target Milestone: Firefox 3.5b4 → mozilla3.5b4
You need to log in
before you can comment on or make changes to this bug.
Description
•