Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

jsshell testing doesn't work on Windows #16627

Closed
johnniwinther opened this issue Feb 7, 2014 · 3 comments
Closed

jsshell testing doesn't work on Windows #16627

johnniwinther opened this issue Feb 7, 2014 · 3 comments
Assignees
@whesse
Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. closed-invalid Closed as we don't believe the reported issue is generally actionable type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)

Comments

@johnniwinther
Copy link
Member

jsshell testing doesn't work on Windows:

d:\dart-repo\dart>python tools/test.py -mrelease -cdart2js -rjsshell --use-sdk --write-debug-log --write-test-outcome-log -t60 lib/mirrors/mirrors_reader_test
Test configuration: dart2js_jsshell_release_ia32
Process error:
  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js
  Error: ProcessException: The system cannot find the file specified.

  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js

Note that

  Command: \d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js

should have been

  Command: d:\dart-repo\dart\tools\testing\bin\jsshell.bat d:/dart-repo/dart/build/ReleaseIA32/generated_compilations/dart2js-sdk/tests_lib_mirrors_mirrors_reader_test/out.js

That is without the prefixed '/'.
@ricowind
Copy link
Contributor

ricowind commented Feb 7, 2014

Bill -> since you are our path expert

Set owner to @whesse.

@ricowind
Copy link
Contributor

ricowind commented Feb 7, 2014

Removed Area-Test label.
Added Area-Infrastructure label.

@johnniwinther johnniwinther added Type-Defect area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. labels Feb 7, 2014
@kevmoo kevmoo added type-bug Incorrect behavior (everything from a crash to more subtle misbehavior) and removed priority-unassigned labels Feb 29, 2016
@athomas
Copy link
Member

athomas commented Apr 10, 2018

jsshell is no longer a supported configuration.

@athomas athomas closed this as completed Apr 10, 2018
@athomas athomas added the closed-invalid Closed as we don't believe the reported issue is generally actionable label Apr 10, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@whesse whesse

Labels
area-infrastructure Use area-infrastructure for SDK infrastructure issues, like continuous integration bot changes. closed-invalid Closed as we don't believe the reported issue is generally actionable type-bug Incorrect behavior (everything from a crash to more subtle misbehavior)
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
5 participants
@kevmoo @whesse @ricowind @johnniwinther @athomas