Closed Bug 524680 Opened 16 years ago Closed 16 years ago

run-acceptance-generic.sh build script tweaks

Categories

(Tamarin Graveyard :: Build Config, defect)

defect
Not set
normal

Tracking

(Not tracked)

VERIFIED FIXED

People

(Reporter: brbaker, Assigned: brbaker)

Details

Attachments

(1 obsolete file)

1) The vmargs option does not handle passing in multiple values. The value should be added to the runtests.py call with surrounding quotes. --vmargs="${vmargs}" 2) Need to have the ability to pass in <scriptargs> which are arguments that are passed to the runtests.py script.
Flags: flashplayer-triage+
1) easy 2) Could we just change the name/documentation on the <testDir> option to be something along the lines of whatever is set here will be tacked onto the call to runtests.py?
Attached patch Patch (obsolete) — Splinter Review
- patch runtestBase.py to handle multiple vmargs in the config string (need to remove the space) - update acceptance-generic to pass additional args into runtests.py
Attachment #408632 - Flags: review?(cpeyer)
Assignee: nobody → brbaker
Status: NEW → ASSIGNED
Attachment #408632 - Flags: review?(cpeyer) → review+
Pushed to redux: 2907:91aad27d59b3
Status: ASSIGNED → RESOLVED
Closed: 16 years ago
Resolution: --- → FIXED
Comment on attachment 408632 [details] [diff] [review] Patch Pushed to redux: 2907:91aad27d59b3
Attachment #408632 - Attachment is obsolete: true
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: