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

[Bug] v6 can't open file like 'dist/report.html' #16

Closed
wenfangdu opened this issue Nov 27, 2020 · 4 comments
Closed

[Bug] v6 can't open file like 'dist/report.html' #16

wenfangdu opened this issue Nov 27, 2020 · 4 comments

Comments

@wenfangdu
Copy link

Command:

open-cli dist/report.html

Works in v5, breaks in v6

@wenfangdu wenfangdu changed the title [Bug] v6 can't open dist/report.html [Bug] v6 can't open file like 'dist/report.html' Nov 27, 2020
@sindresorhus
Copy link
Owner

Works fine for me on macOS 11. You haven't really included any useful info for me to debug this. Try to look through https://github.com/sindresorhus/open/issues?q=is:issue+is:open+sort:updated-desc and see if your issue is one of those.

@Kreozot
Copy link

Kreozot commented Dec 28, 2020

I confirm this issue on Windows 10 (node 14.15.1)

And yes, it's working pretty well on open-cli 5.0.0. So I suggest considering it as a new Windows-specific bug.

@Kreozot
Copy link

Kreozot commented Dec 28, 2020

I was trying to execute open-cli out/index.html

console.log(command, cliArguments, childProcessOptions); before childProcess.spawn in index.js of "open" package shows this:

on open@6.4.0:

cmd [ '/c', 'start', '""', '/b', 'out/index.html' ] {}

(and this works)

on open@7.3.0:

C:\WINDOWS\System32\WindowsPowerShell\v1.0\powershell [
  '-NoProfile',
  '-NonInteractive',
  '–ExecutionPolicy',
  'Bypass',
  '-EncodedCommand',
  'UwB0AGEAcgB0ACAAIgBgACIAbwB1AHQALwBpAG4AZABlAHgALgBoAHQAbQBsAGAAIgAiAA=='
] { windowsVerbatimArguments: true }

(and this doesn't works)

@kuzalekon
Copy link

#214 must fix this problem in the open library. Then it will be necessary to update this dependency up to the new version.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

4 participants