configure crashes when /bin/sh == /bin/dash and config.cache is present
VERIFIED
FIXED
in mozilla1.9.1a2
Status
()
People
(Reporter: dbaron, Assigned: dbaron)
Tracking
({fixed1.9.0.2})
Firefox Tracking Flags
(Not tracked)
Details
Attachments
(1 attachment)
|
961 bytes,
patch
|
Pike
:
review+
ted
:
superreview+
Samuel Sidler (old account; do not CC)
:
approval1.9.0.2+
|
Details | Diff | Splinter Review |
As a result of the changes to configure.in in bug 442279, configure crashes on Ubuntu systems where /bin/sh is dash rather than bash, and when a config.cache is present from a previous run of configure. In other words, this crash is a result of the change: http://hg.mozilla.org/mozilla-central/index.cgi/diff/9871082391f5/configure.in
|
(Assignee) | |
Comment 1•10 years ago
|
||
Created attachment 331343 [details] [diff] [review] patch
Assignee: nobody → dbaron
Status: NEW → ASSIGNED
Attachment #331343 -
Flags: superreview?(ted.mielczarek)
Attachment #331343 -
Flags: review?(l10n)
|
||
Comment 2•10 years ago
|
||
Comment on attachment 331343 [details] [diff] [review] patch I don't get why it crashes, but ok!
Attachment #331343 -
Flags: superreview?(ted.mielczarek) → superreview+
|
||
Updated•10 years ago
|
||
Attachment #331343 -
Flags: review?(l10n) → review+
|
||
Comment 3•10 years ago
|
||
FWIW, I filed this to Ubuntu yesterday: https://bugs.launchpad.net/ubuntu/+source/dash/+bug/251889
|
|
(Assignee) | |
Comment 4•10 years ago
|
||
Ah, I didn't file because the obvious simplified testcase didn't exhibit the problem, and I wasn't in the mood to simplify configure down to a testcase (though it may be worth doing). In any case, fixed: http://hg.mozilla.org/mozilla-central/index.cgi/rev/589f091246af
Status: ASSIGNED → RESOLVED
Last Resolved: 10 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla1.9.1a2
|
||
Comment 5•10 years ago
|
||
Comment on attachment 331343 [details] [diff] [review] patch Approved for 1.9.0.2. Please land in CVS. a=ss (Taking per bug 442279 comment 9.)
Attachment #331343 -
Flags: approval1.9.0.2+
See Also: → https://launchpad.net/bugs/251889
You need to log in
before you can comment on or make changes to this bug.
Description
•