Closed Bug 1334268 Opened 4 years ago Closed 4 years ago

js/src/old-configure reports "grep: invalid option" on Windows

Categories

(Core :: JavaScript Engine, defect)

x86
Windows
defect
Not set
trivial

Tracking

()

RESOLVED FIXED
mozilla54
Tracking Status
firefox54 --- fixed

People

(Reporter: tcampbell, Assigned: tcampbell)

References

Details

Attachments

(1 file, 1 obsolete file)

When building jsshell on Windows using mozilla-build 32-bit environment, the configure process has the following error message:

<<<
checking for _getc_nolock... (cached) yes
checking for sb-conf... no
checking for ve... no
checking for X... (cached) no
grep: invalid option -- /
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
grep: invalid option -- /
Usage: grep [OPTION]... PATTERN [FILE]...
Try `grep --help' for more information.
checking for valid debug flags... yes
checking NSPR selection... source-tree
checking what kind of list files are supported by the linker... list
>>>

This is caused by autoconf quoting problems, and a fix is attached.

Even with the error, the result for a default config is the same. Problems might have occurred if manually specifying -arch flag.
Assignee: nobody → tcampbell
Attachment #8830920 - Flags: review?(mh+mozilla)
Comment on attachment 8830920 [details] [diff] [review]
0001-Bug-1334268-Fix-regex-escaping-in-js-src-old-configu.patch

Review of attachment 8830920 [details] [diff] [review]:
-----------------------------------------------------------------

The same change is needed in top-level old-configure.in.
Attachment #8830920 - Flags: review?(mh+mozilla)
Attachment #8830920 - Attachment is obsolete: true
Attachment #8833174 - Flags: review?(mh+mozilla)
Attachment #8833174 - Flags: review?(mh+mozilla) → review+
Keywords: checkin-needed
Pushed by cbook@mozilla.com:
https://hg.mozilla.org/integration/mozilla-inbound/rev/00f1ed73cf65
Fix regex escaping in js/src/old-configure. r=glandium
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/00f1ed73cf65
Status: NEW → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla54
Duplicate of this bug: 1288849
You need to log in before you can comment on or make changes to this bug.