make package fails when $(MOZILLA_DIR) is a relative path

RESOLVED FIXED in mozilla9

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: jorendorff, Assigned: jorendorff)

Tracking

Other Branch
mozilla9
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(1 attachment)

Comment hidden (empty)
(Assignee)

Comment 1

6 years ago
Created attachment 556594 [details] [diff] [review]
fix

Ted suggested this fix and it WFM.
Comment on attachment 556594 [details] [diff] [review]
fix

That looks right, given the number of places we call GENERATE_CACHE and what this is doing.
Attachment #556594 - Flags: review+
(Assignee)

Comment 3

6 years ago
Oh, but you know what? We should find out if xpcshell is actually exiting with a nonzero exit code in this case, and if so, fix the makefile to notice the error. (It is a kind of complicated command with some () and some && and some ; up in there.)

If it's xpcshell's fault, we should fix that, separate bug I guess.
(Assignee)

Comment 4

6 years ago
(In reply to Jason Orendorff [:jorendorff] from comment #3)
> Oh, but you know what? We should find out if xpcshell is actually exiting
> with a nonzero exit code in this case, and if so, fix the makefile to notice
> the error. (It is a kind of complicated command with some () and some && and
> some ; up in there.)

By "we", of course, I mean "y'all".

I checked and it looks like xpcshell's exit code is 3 in this case, so I guess a Makefile fix would be the thing.
(Assignee)

Updated

6 years ago
Attachment #556594 - Flags: superreview?(ted.mielczarek)
Attachment #556594 - Flags: superreview?(ted.mielczarek) → superreview+
(Assignee)

Comment 5

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/7509ecc14e9e
Whiteboard: [inbound]

Comment 6

6 years ago
http://hg.mozilla.org/mozilla-central/rev/7509ecc14e9e
Assignee: nobody → jorendorff
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla9
You need to log in before you can comment on or make changes to this bug.