355 bytes, patch
|Details | Diff | Splinter Review|
Linux build 2000101608-MN6. 1. Open up one of the installed scripts generated by build automation (e.g. install.js in xpcom.xpi). 2. Check addDirectory and addFile commands. 3. Look at the version string passed to them, particularly the build component (major.minor.release.build). Result: 5.0.0.0000000000. no build ID in 3rd component. Expected: 220.127.116.110101608. Question: Both Linux and Mac use version 5 instead of 6. Don't we want to use 6 for the major component?
punt to samir. I have no idea how the install.js scripts are being generated other than a vague thought that it's in deliver.pl.
Assignee: granrose → sgehani
Pong. Back to granrose. You supply the build id to deliver.pl. http://lxr.mozilla.org/seamonkey/source/xpinstall/packager/unix/deliver.pl#26
Assignee: sgehani → granrose
doh! looking at the automation, it's looking at mozilla/build/build_number which apparently is no longer being used (and should probably be removed from the tree). pointed automation to use mozilla/config/build_number which does have the correct build id in it. one line change to the automation to look at the correct file. I'll attach the diff. cls - can you review? leaf - can you approve?
Status: NEW → ASSIGNED
Whiteboard: [rtm need info] fix in hand, waiting for r= a=
Created attachment 17282 [details] [diff] [review] get buildid from config/build_number, not build/build_number
oh, and this patch is just to include the buildid. I'm not touching the 5.0 vs 6.0 thing with a 10' pole. That's for marketing and the installer team (and whoever else wants to jump into the fray) to work out.
r=cls on the patch. build/build_number is going away as soon as we can land bug 26798.
not sure i'm officially on the `a' list yet, but a=leaf
PDT: how bout that rtm++?
Whiteboard: [rtm need info] fix in hand, waiting for r= a= → [rtm+] fix in hand, r=cls, a=leaf
added pp to keyword (platform specific)
Build ID is also 0000000000 for the registry version in initInstall().
PDT says rtm++, please land on branch ASAP.
Whiteboard: [rtm+] fix in hand, r=cls, a=leaf → [rtm++] fix in hand, r=cls, a=leaf
fix checked in.
Status: ASSIGNED → RESOLVED
Last Resolved: 19 years ago
Resolution: --- → FIXED
build id should start showing up correctly tomorrow.
verified on Linux branch build 2000-10-19-09-MN6. On to the trunk build.
Verified on Linux trunk build 2000-10-19-08-MTrunk.
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.