Closed
Bug 394707
Opened 17 years ago
Closed 17 years ago
`make package` broken for cross-compiles (not using host xpt_link)
Categories
(Toolkit Graveyard :: Build Config, defect)
Toolkit Graveyard
Build Config
Tracking
(Not tracked)
RESOLVED
FIXED
People
(Reporter: Mook, Assigned: Mook)
References
()
Details
Attachments
(1 file, 1 obsolete file)
2.15 KB,
patch
|
Details | Diff | Splinter Review |
See URL. The script hard-codes the copy of xpt_link in dist/bin, which means that `make package` fails when cross-compiling (because either xpt_link is not found, or it's for the wrong architecture).
Using $(XPIDL_LINK) seems happier for me, but that pulls in $(CYGWIN_WRAPPER) and I have no idea if that breaks cygwin builds. Please test with one of those?
Attachment #279410 -
Flags: review?(benjamin)
Comment 1•17 years ago
|
||
Comment on attachment 279410 [details] [diff] [review]
use $(XPIDL_LINK)
it's likely that the additional CYGWIN_WRAPPER will be correct in the case where we're using cygwin perl. Feel free to check this in, but expect to backout if some of the tinderboxes (seamonkey especially) go red.
Attachment #279410 -
Flags: review?(benjamin)
Attachment #279410 -
Flags: review+
Attachment #279410 -
Flags: approval1.9+
Presumably r+ and a+ carries over, because the only changes were to the crap before the actual diff :( patch doesn't want to apply, apparently...
Assignee: nobody → mook.moz+mozbz
Attachment #279410 -
Attachment is obsolete: true
Status: NEW → ASSIGNED
Comment 3•17 years ago
|
||
Checking in toolkit/mozapps/installer/packager.mk;
/cvsroot/mozilla/toolkit/mozapps/installer/packager.mk,v <-- packager.mk
new revision: 1.52; previous revision: 1.51
done
Thanks for the checkin, mcsmurf!
Status: ASSIGNED → RESOLVED
Closed: 17 years ago
Resolution: --- → FIXED
Updated•6 years ago
|
Product: Toolkit → Toolkit Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•