-
Notifications
You must be signed in to change notification settings - Fork 7
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
Error! primordials is not defined #357
Comments
DescriptionWhen trying to use
My environmentLinux Mint-Cinnamon 4.2.4
|
+1 |
Initially, I was getting the message
Some Googling quickly led me to test out other Node.js versions and indeed that solved the issue right away. In the npm package page for data-cli, we find the following information:
My environmentLinux debian 4.19.0-8-amd64 Therefore, I believe the documentation should probably be updated to point out this shortcoming as the error message didn't help much in solving the problem. |
@sglavoie can we look for an actual patch for this - it looks like it will be some library not working under node >= 12. Can you see (quickly) if you can identify the source and we then pin the library or similar (e.g. it looks like graceful-fs may be the source). |
@rufuspollock, I didn't have much luck trying to debug this so far. I could of course keep trying, but then it won't be "quickly" anymore. I tried under Node 13.x and 12.x:
Although not a great fix, I have seen this error message pop up with other packages such as Gulp and unzip. Suggestions were in those instances to update the packages or downgrade Node to a version <12. This is clearly not a proper fix, but knowing how to temporarily work around the issue for now can still prove useful until this is debugged further. 🤷♂️ |
@rufuspollock, you were right once again! 😄 SummaryUpdating Using
Result with default data-cli 0.9.6
Result with patched data-cli 0.9.6Addition to package.jsonThe section
Output with patched versionStarting with a cleanly cloned repo with the modified
Reached out to @anuveyatsu to let him know. I can work on a pull request once a proper fix is confirmed. |
@EvgeniiaVak, the above change was merged in the project. It has not been published yet as a new release on npmjs.com, but you could install it by cloning the repo:
Let me know if that works for you! 🙂 |
@sglavoie can you get permissions from @anuveyatsu to publish to npm so that you can get this fix published officially. 🚀 |
@rufuspollock, yes, I'll ping him right now! |
* This is to fix `Error! primordials is not defined` when using Node.js >=12.x. * Stop testing against older versions of Node.js as there were breaking changes with recent versions of `graceful-fs`.
@sglavoie Yes, |
I can confirm that putting
in |
Description
When trying to get data I receive an error:
Publishing data works fine.
My environment
MacOS Catalina
node 12.12.0
npm 12.12.0
data 0.9.6
The text was updated successfully, but these errors were encountered: