Closed Bug 545171 Opened 14 years ago Closed 14 years ago

android test proxy script reverse the order of vm arguments

Categories

(Tamarin Graveyard :: Tools, defect, P3)

x86
macOS
defect

Tracking

(Not tracked)

VERIFIED FIXED
flash10.1

People

(Reporter: dschaffe, Assigned: dschaffe)

Details

Attachments

(1 file, 1 obsolete file)

4.08 KB, application/octet-stream
rwinchel
: review+
Details
the android_shell.sh script reverse the arguments
- improve android script to copy all abc files and clean up after test finishes
- mark android known failures in testconfig.txt and exclude the slow running tests
Attached patch patch (obsolete) — Splinter Review
Assignee: nobody → dschaffe
Status: NEW → ASSIGNED
Attachment #426023 - Flags: review?(brbaker)
Attachment #426023 - Flags: review?(rwinchel)
Comment on attachment 426023 [details] [diff] [review]
patch

Need to be able to maintain application arguments that are passed after the .abc file

$AVM test.abc -- foobar

This gets converted to:
$AVM -- foobar test.abc
Attachment #426023 - Flags: review?(brbaker) → review-
yeah good catch.  I'll work on a version that preserves the order for all args.
Assignee: dschaffe → nobody
Component: Virtual Machine → Tools
Flags: flashplayer-qrb+
Priority: -- → P3
QA Contact: vm → tools
Target Milestone: --- → flash10.1
Assignee: nobody → dschaffe
Attached file patch
updated patch for adding android configuration to acceptance tests.  copy all abc files to the device, run the abc, cleanup abc on device.
Attachment #426023 - Attachment is obsolete: true
Attachment #427810 - Flags: review?(rwinchel)
Attachment #426023 - Flags: review?(rwinchel)
Attachment #427810 - Flags: review?(rwinchel) → review+
pushed in 3864:f687ea8698c0, http://hg.mozilla.org/tamarin-redux/rev/f687ea8698c0
Status: ASSIGNED → RESOLVED
Closed: 14 years ago
Resolution: --- → FIXED
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: