Skip to content

Platform.executable does not return the '.exe' on Windows #23613

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.

Sign up for GitHub

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
Closed
kevmoo opened this issue Jun 9, 2015 · 1 comment
Closed

Platform.executable does not return the '.exe' on Windows #23613

kevmoo opened this issue Jun 9, 2015 · 1 comment
Assignees
@bkonyi
Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-as-intended Closed as the reported issue is expected behavior library-io

Comments

@kevmoo
Copy link
Member

kevmoo commented Jun 9, 2015

See dart-lang/test#133
and https://codereview.chromium.org/1165313002/
@sethladd sethladd added area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. library-io labels Jun 11, 2015
@sgjesse sgjesse removed their assignment Jun 14, 2016
@bkonyi bkonyi self-assigned this Jun 21, 2018
@bkonyi bkonyi added the closed-as-intended Closed as the reported issue is expected behavior label Jun 29, 2018
@bkonyi
Copy link
Contributor

bkonyi commented Jun 29, 2018

It looks like this just has to do with how the script is invoked:

C:\Users\bkonyi\dart-sdk-2\sdk>out\DebugX64\dart tmp.dart
out\DebugX64\dart

C:\Users\bkonyi\dart-sdk-2\sdk>out\DebugX64\dart.exe tmp.dart
out\DebugX64\dart.exe

Should we be appending .exe to Platform.executable on Windows? From what I can tell, that wouldn't be a good solution since Windows doesn't require that executables have a .exe extension. I'm going to close this as working as intended for now.

cc/ @zanderso for potential input.

@bkonyi bkonyi closed this as completed Jun 29, 2018
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@bkonyi bkonyi

Labels
area-core-library SDK core library issues (core, async, ...); use area-vm or area-web for platform specific libraries. closed-as-intended Closed as the reported issue is expected behavior library-io
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
4 participants
@sethladd @kevmoo @sgjesse @bkonyi