Closed Bug 749573 Opened 12 years ago Closed 12 years ago

"make install" doesn't install xpcshell and run-mozilla.sh in the SDK

Categories

(Firefox Build System :: General, defect)

x86_64
Linux
defect
Not set
normal

Tracking

(firefox12 unaffected, firefox13 unaffected, firefox14 fixed)

RESOLVED FIXED
mozilla15
Tracking Status
firefox12 --- unaffected
firefox13 --- unaffected
firefox14 --- fixed

People

(Reporter: glandium, Assigned: glandium)

References

Details

Attachments

(1 file)

bug 743560 made them part of the SDK instead of xulrunner. make make-sdk ships them, but make install doesn't.

As a side note, run-mozilla.sh is not shipped in xulrunner, maybe it shouldn't be shipped in Firefox either.
Attachment #618992 - Flags: review?(ted.mielczarek)
Attachment #618992 - Flags: review?(ted.mielczarek) → review+
https://hg.mozilla.org/mozilla-central/rev/879fd99f199c
Status: NEW → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment on attachment 618992 [details] [diff] [review]
Declare run-mozilla.sh and xpcshell as SDK_BINARY

[Approval Request Comment]
Regression caused by: bug 743560 
User impact if declined: People using make install (mostly, Linux distros) won't have run-mozilla.sh and xpcshell installed with SDK.
Testing completed (on m-c, etc.): Landed on m-c a couple days ago.
Risk to taking this patch (and alternatives if risky): None.
String changes made by this patch: None.
Attachment #618992 - Flags: approval-mozilla-aurora?
Comment on attachment 618992 [details] [diff] [review]
Declare run-mozilla.sh and xpcshell as SDK_BINARY

[Triage Comment]
Low risk fix in support of local builds.
Attachment #618992 - Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Product: Core → Firefox Build System
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: