make -C [browser|xulrunner]/installer fails

RESOLVED FIXED in mozilla8

Status

()

Core
Build Config
RESOLVED FIXED
6 years ago
6 years ago

People

(Reporter: Jacek Caban, Assigned: glandium)

Tracking

Trunk
mozilla8
x86_64
Linux
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

Attachments

(2 attachments)

(Reporter)

Description

6 years ago
Created attachment 547030 [details] [diff] [review]
fix v1.0

I've found the problem by trying to do |make -C xulrunner/installer| in my cross compiled mingw build on Linux, but after analyzing the problem, it seems to be a problem also for browser builds and other configs. The error message is:

/bin/sh: 1: command not found

That's because GENERATE_CACHE is used as a command in toolkit/mozapps/installer/packager.mk PACK_OMNIJAR rule:

PACK_OMNIJAR    = \
  rm -f omni.jar components/binary.manifest && \
  grep -h '^binary-component' components/*.manifest > binary.manifest ; \
  for m in components/*.manifest; do \
    sed -e 's/^binary-component/\#binary-component/' $$m > tmp.manifest && \
    mv tmp.manifest $$m; \
  done; \
  $(ZIP) -r9m omni.jar $(OMNIJAR_FILES) -x $(NON_OMNIJAR_FILES) && \
  $(GENERATE_CACHE) && \
  $(OPTIMIZE_JARS_CMD) --optimize $(JARLOG_DIR_AB_CD) ./ ./ && \
  mv binary.manifest components && \
  printf "manifest components/binary.manifest\n" > chrome.manifest

but it's set to 1 instead of true in browser/installer/Makefile.in and xulrunner/installer/Makefile.in.
Attachment #547030 - Flags: review?(mh+mozilla)
(Assignee)

Updated

6 years ago
Attachment #547030 - Flags: review?(mh+mozilla) → review-
(Assignee)

Comment 1

6 years ago
Created attachment 547037 [details] [diff] [review]
Properly set GENERATE_CACHE in all cases
Attachment #547037 - Flags: review?(ted.mielczarek)
(Assignee)

Updated

6 years ago
Assignee: jacek → mh+mozilla
Blocks: 655678
Comment on attachment 547037 [details] [diff] [review]
Properly set GENERATE_CACHE in all cases

Review of attachment 547037 [details] [diff] [review]:
-----------------------------------------------------------------
Attachment #547037 - Flags: review?(ted.mielczarek) → review+
(Assignee)

Comment 3

6 years ago
http://hg.mozilla.org/integration/mozilla-inbound/rev/c3906fd66ba0
Whiteboard: [inbound]
(Assignee)

Comment 4

6 years ago
Fixup:
http://hg.mozilla.org/integration/mozilla-inbound/rev/3223495dfc69
http://hg.mozilla.org/mozilla-central/rev/c3906fd66ba0
http://hg.mozilla.org/mozilla-central/rev/3223495dfc69
Status: NEW → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
Whiteboard: [inbound]
Target Milestone: --- → mozilla8
You need to log in before you can comment on or make changes to this bug.