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

global_compile.dart broken file paths on Windows #27451

Closed
nshahan opened this issue Sep 28, 2016 · 2 comments
Closed

global_compile.dart broken file paths on Windows #27451

nshahan opened this issue Sep 28, 2016 · 2 comments
Assignees
Milestone

Comments

@nshahan
Copy link
Contributor

nshahan commented Sep 28, 2016

There is a problem parsing the URI when running in the windows command line. The file path gets truncated down to just c:

Unhandled exception:
FileSystemException: Cannot open file, path = 'c:' (OS Error: Access is denied.
, errno = 5)
#0      _File.throwIfError (dart:io/file_impl.dart:562)
#1      _File.openSync (dart:io/file_impl.dart:413)
#2      _File.readAsBytesSync (dart:io/file_impl.dart:473)
#3      _File.readAsStringSync (dart:io/file_impl.dart:507)
#4      _loadFile (file:///c:/Users/nshahan/code/dart-sdk/sdk/pkg/dev_compiler/tool/global_compile.dart:267:24)
#5      transitiveFiles (file:///c:/Users/nshahan/code/dart-sdk/sdk/pkg/dev_compiler/tool/global_compile.dart:275:18)
#6      main (file:///c:/Users/nshahan/code/dart-sdk/sdk/pkg/dev_compiler/tool/global_compile.dart:79:3)
#7      _startIsolate.<anonymous closure> (dart:isolate-patch/isolate_patch.dart:259)
#8      _RawReceivePortImpl._handleMessage (dart:isolate-patch/isolate_patch.dart:148)�
@Sfshaza Sfshaza assigned vsmenon and nshahan and unassigned nshahan and vsmenon Oct 5, 2016
@Sfshaza Sfshaza added this to the 1.21 milestone Oct 5, 2016
@mit-mit
Copy link
Member

mit-mit commented Nov 28, 2016

@nshahan was this fixed in 8f5f1a0 ?

@nshahan
Copy link
Contributor Author

nshahan commented Nov 29, 2016

I was seeing this issue when running on a windows machine. The branch you mentioned worked for me on the project I was working on but was not thoroughly tested on other platforms and I don't want to merge those changes in. I will cleanup.

@nshahan nshahan closed this as completed Nov 29, 2016
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

4 participants