Closed Bug 443154 Opened 12 years ago Closed 12 years ago

Sisyphus - JavaScript Tests - do not require js tests to be on path

Categories

(Testing Graveyard :: Sisyphus, defect)

x86
Linux
defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: bc, Assigned: bc)

References

Details

(Keywords: verified1.9.0.1)

Attachments

(1 file)

Attached patch patchSplinter Review
js/tests/Makefile calls pattern-expander.pl and pattern-extracter.pl without specifying a path to the script which requires the js tests directory be in PATH. This patch simply calls the two scripts from the current directory.
Flags: in-testsuite-
Flags: in-litmus-
/cvsroot/mozilla/js/tests/Makefile,v  <--  Makefile
new revision: 1.18; previous revision: 1.17

changeset:   15630:826d74c701ef
Blocks: 396028
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Keywords: fixed1.9.0.1
Resolution: --- → FIXED
hi bob, can you verify this is fixed on 3.0.1 builds?   just update the keyword to "verified1.9.0.1", and marked status VERIFIED when complete.  thanks!
Its not part of the build, but I'll rubber stamp it for ya. 
Status: RESOLVED → VERIFIED
Mass move of Sisyphus bugs to Testing : Sisyphus. Filter on SisyphusMassMove to ignore.
Component: Testing → Sisyphus
Flags: in-litmus-
Product: Core → Testing
QA Contact: testing → sisyphus
Version: 1.9.0 Branch → Trunk
Product: Testing → Testing Graveyard
You need to log in before you can comment on or make changes to this bug.