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

Windows filenames versus URI/URL #312

Closed
peter-ahe-google opened this issue Nov 2, 2011 · 2 comments
Closed

Windows filenames versus URI/URL #312

peter-ahe-google opened this issue Nov 2, 2011 · 2 comments
Assignees
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.

Comments

@peter-ahe-google
Copy link
Contributor

This is not about the error below. This is about the formatting of the stack trace on Windows as illustrated by the error message below.

Consider the first two lines of of the stack trace:

 0. Function: '[email protected]' url: './dart:builtin-lib' line:857 col:7
 1. Function: 'FileTest.testReadWriteSync' url: 'e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart' line:201 col:18

Notice these parts in particular:

url: './dart:builtin-lib'
url: 'e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart'

The first is correct, the second contains a Windows file name that can be represented correctly as a URI using this form:

file:///E:/b/build/slave/vm-win32-release/build/tests/standalone/src/FileTest.dart

See: http://blogs.msdn.com/b/ie/archive/2006/12/06/file-uris-in-windows.aspx

Since Dart lives in a world where URI/URL are important and used all the time, I would consider if the basic abstraction should be URIs, not platform dependent filenames.

The example error message is from:

=== releaseia32 vm standalone/FileTest ===
Path: standalone/FileTest

--- stderr ---
An unhandled exception has been thrown
FileIOException: Cannot open file: runtime/tests/vm/data/fixed_length_file_out
 0. Function: '[email protected]' url: './dart:builtin-lib' line:857 col:7
 1. Function: 'FileTest.testReadWriteSync' url: 'e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart' line:201 col:18
 2. Function: 'FileTest.testMain' url: 'e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart' line:540 col:39
 3. Function: 'main' url: 'e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart' line:555 col:20
Command: e:\b\build\slave\vm-win32-release\build\runtime\Release_ia32\dart_bin --ignore-unrecognized-flags --optimize e:\b\build\slave\vm-win32-release\build\tests\standalone\src\FileTest.dart

@iposva-google
Copy link
Contributor

This is fixed now that the URI handling in imports has been rewritten. Output from a recent test failure on the Windows buildbot:

Unhandled exception:
type 'Null' is not a subtype of type 'bool' of 'boolean expression'.
 0. Function: '::get:_port@6be832b' url: 'dart:isolate' line:374 col:7
 1. Function: '::get:port' url: 'dart:isolate' line:17 col:27
 2. Function: '::timIsolate' url: 'file:///e:/b/build/slave/vm-win32-release/build/dart/tests/isolate/src/APIv2_unresolvedPortsStandaloneTest.dart' line:23 col:3


Set owner to @iposva-google.
Added Fixed label.

@peter-ahe-google
Copy link
Contributor Author

I have filed issue #6 to track handling of //. 

@peter-ahe-google peter-ahe-google added Type-Defect area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends. labels Mar 18, 2012
nex3 pushed a commit that referenced this issue Aug 31, 2016
Closure compiler chokes on super classes that aren't qualified paths.

(example: http://goo.gl/5mHC7S)

BUG=
[email protected]

Review URL: https://codereview.chromium.org/1638533004 .
nex3 pushed a commit that referenced this issue Aug 31, 2016
…or Closure, to avoid ES5->ES6 lowering bug (google/closure-compiler#1460).

This is part of the overall "simple closure" effort (issue #312)

BUG=
[email protected]

Review URL: https://codereview.chromium.org/1630963003 .
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
area-vm Use area-vm for VM related issues, including code coverage, and the AOT and JIT backends.
Projects
None yet
Development

No branches or pull requests

2 participants