search

csstools / system-font-css

Use the native system font of the OS running the browser
https://jonathantneal.github.io/system-font-css/
Other
521 stars 33 forks source link

npm install failed at Windows machines #10

Closed benmb1984 closed 6 years ago

benmb1984 commented 6 years ago

Hi Jonathan,

I get an error during the installation of your package. It seems like PowerShell on Windows is using cmd.exe during builds. In your script I see some cp commands. I hope you have some time for this.

More info: https://github.com/ReactiveX/rxjs/issues/1601

Kind regards, Ben

zachleat commented 6 years ago

Wasn’t this fixed in #8?

benmb1984 commented 6 years ago

No, I have test it on multiple Windows machines. Same error everywhere.

image

zachleat commented 6 years ago

Why is it using cp there? #8 changed it to use cpr. Do we need to tag another release or something to update?

benmb1984 commented 6 years ago

cpr is used in the latest release I have tested. Maybe cpr itself (or node.js) is using the copy command.

Maybe the most easy solution is don't copy the file during install. Provide both files already in your repository.

zachleat commented 6 years ago

What’s in your package.json for the install script? Can you run npm run install?

zachleat commented 6 years ago

I just checked and yes, the cpr version was unreleased to npm. Try again with 2.0.2 please!

benmb1984 commented 6 years ago

Hi Zachleat, Problem solved. It works now with classic CMD and PowerShell. Many thanks!