The npm package name is already wrapped in quotes to prevent special characters 
from being execute. However the additional win32 code was further injecting 
quotes around the version causing npm to return a EINVALIDTAGNAME error. The 
win32 code is redundant and seems to not be required.

<!--
Please make sure the checklist boxes are all checked before submitting the PR. 
The checklist
is intended as a quick reference, for complete details please see our 
Contributor Guidelines:

http://cordova.apache.org/contribute/contribute_guidelines.html

Thanks!
-->

### Platforms affected
Windows, CLI

### What does this PR do?
Fix plugin installation on windows platform

### What testing has been done on this change?
Ran it locally and error vanished

### Checklist
- [x] [Reported an issue](http://cordova.apache.org/contribute/issues.html) in 
the JIRA database
- [x] Commit message follows the format: "CB-3232: (android) Fix bug with 
resolving file paths", where CB-xxxx is the JIRA ID & "android" is the platform 
affected.
- [ ] Added automated test coverage as appropriate for this change.


[ Full content available at: https://github.com/apache/cordova-lib/pull/622 ]
This message was relayed via gitbox.apache.org for [email protected]

Reply via email to