Closed
Bug 388078
Opened 17 years ago
Closed 17 years ago
Trunk build broken
Categories
(Core Graveyard :: XForms, defect)
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: aaronr, Unassigned)
References
Details
(Keywords: fixed1.8.1.12)
Attachments
(1 file)
2.92 KB,
patch
|
doronr
:
review+
smaug
:
review+
|
Details | Diff | Splinter Review |
Due to checkin for bug 386903, xforms build is broken on the trunk. We use makejs.pl which that bug removed. It is used in conjunction with install.jst to create an install.js. Looks like to fix this build break we'll have to keep around a static install.js like other extensions do. I think we really only needed it to get our version number and the space required variables. But when this is fixed we'll have to build both firefox and seamonkey and make sure nothing else is broken.
I basically moved the contents of install.jst to install.js. I hardcode the version number and the space required numbers, using the numbers from the install.js that 1.8 branch still generates. These space required values are just a guide, anyhow. I also added the function verifyDiskSpace (copied from the chatzilla install.js). I also think that this change should be made on the 1.8 branch too, just to keep things common between branch and trunk.
Attachment #272542 -
Flags: review?(doronr)
Attachment #272542 -
Flags: review?(Olli.Pettay)
Updated•17 years ago
|
Attachment #272542 -
Flags: review?(doronr) → review+
Updated•17 years ago
|
Attachment #272542 -
Flags: review?(Olli.Pettay) → review+
Fix checked into trunk. Branch is closed, so will have to wait on that until 2.0.0.5 goes out.
Status: NEW → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Whiteboard: xf-to-branch
checked into 1.8 branch via bug 410239.
Keywords: fixed1.8.1.12
Whiteboard: xf-to-branch
note: this was effectively backed out of the 1.8 branch with bug 411960. I shouldn't have checked this into 1.8 in the first place. Doh!
Assignee | ||
Updated•8 years ago
|
Product: Core → Core Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•