Closed
Bug 1268882
Opened 8 years ago
Closed 8 years ago
unportable test operator in run-all-loop-tests.sh
Categories
(Hello (Loop) :: Client, defect)
Hello (Loop)
Client
Tracking
(firefox49 fixed)
Tracking | Status | |
---|---|---|
firefox49 | --- | fixed |
People
(Reporter: wiz, Assigned: wiz)
Details
Attachments
(1 file)
382 bytes,
patch
|
standard8
:
review+
|
Details | Diff | Splinter Review |
User Agent: Mozilla/5.0 (X11; NetBSD amd64; rv:46.0) Gecko/20100101 Firefox/46.0 Build ID: 20160428084422 Steps to reproduce: The file browser/extensions/loop/run-all-loop-tests.sh contains: if [ "$1" == "--help" ]; then However, '==' as comparison operator is only supported by bash (not even GNU coreutils). '=' is the standard comparison operator for test(1).
Component: Untriaged → Client
Product: Firefox → Hello (Loop)
Version: 46 Branch → unspecified
Assignee | ||
Updated•8 years ago
|
Attachment #8747062 -
Attachment is patch: true
Attachment #8747062 -
Attachment mime type: application/x-shellscript → text/plain
Attachment #8747062 -
Flags: review?(standard8)
Comment 1•8 years ago
|
||
Comment on attachment 8747062 [details] [diff] [review] Patch against trunk fixing the problem. Thanks for the patch. Looks good r=Standard8. I'll get this landed for you. For future reference, its easier for reviewers to download & apply if you can generate patches in the git style, there's some more information on that here: https://developer.mozilla.org/en-US/docs/Mercurial/Using_Mercurial#How_can_I_generate_a_patch_for_somebody_else_to_check-in_for_me.3F
Attachment #8747062 -
Flags: review?(standard8) → review+
Updated•8 years ago
|
Assignee: nobody → wiz
Updated•8 years ago
|
Status: UNCONFIRMED → NEW
Iteration: --- → 49.2 - May 23
Ever confirmed: true
Comment 3•8 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/5c35538356c0
Status: NEW → RESOLVED
Closed: 8 years ago
status-firefox49:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla49
You need to log in
before you can comment on or make changes to this bug.
Description
•