-
Notifications
You must be signed in to change notification settings - Fork 9
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
Fix versions from upstream modules #12
base: master
Are you sure you want to change the base?
Conversation
Attempted to try to find an old version but no luck. Do you have an older version that does not error with 'ERR_REQUIRE_ESM'? |
Hi Hugo, yes, well if you use the version that I specify in package.json it will pull in the correct versions. You have to install the version on my temp branch: npm install https://github.com/remkolodder/vmw-sdk I use them at work regularly without issues :-) |
I'm not sure if I'm doing this incorrectly. Need an idiots guide.
|
Hi, I am sorry, the git part is appended: npm install https://github.com/remkolodder/vmw-sdk.git --global This works for us and uses the working versions of the software :) |
Getting another error. `npm install https://github.com/remkolodder/vmw-sdk.git --global npm ERR! A complete log of this run can be found in: |
Ah but you do not have git available? |
That was it! Forgot to install git onto my new docker image! Thanks for the help, working now. |
I have added a part of the pull request to initialise the storage for the cache. The "init()" was missing from the cache storage, and therefor breaking my usage of the vmw-cli application. After adding this, caching works again and i can read the vmware upstream repositories. Please consider merging these, in addition to the other update (#13) that brings this into a more future proof form. |
The latest versions from Chalk and Got require "ESM" and a different way of using the code. Require becomes import and more changes like that. To fix this, use the old version that is also in the docker image.