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

v5.11.2 creates broken builds #28

Closed
Mikescops opened this issue Feb 10, 2024 · 10 comments
Closed

v5.11.2 creates broken builds #28

Mikescops opened this issue Feb 10, 2024 · 10 comments

Comments

@Mikescops
Copy link

What version of pkg are you using?

5.11.2

What version of Node.js are you using?

18.19.0

What operating system are you using?

macOS

What CPU architecture are you using?

Apple M1/2/3

What Node versions, OSs and CPU architectures are you building for?

node-18-macos-arm64

Describe the Bug

Since the update to 5.11.2 the builds give the following error when executed:

node:internal/bootstrap/realm:420
  if (!mod) throw new TypeError(`Missing internal module '${id}'`);
            ^

TypeError: Missing internal module 'internal/modules/cjs/helpers'
    at requireBuiltin (node:internal/bootstrap/realm:420:19)
    at pkg/prelude/bootstrap.js:1884:12
    at pkg/prelude/bootstrap.js:1986:3
    at pkg/prelude/bootstrap.js:2259:3
    at readPrelude (node:internal/bootstrap/pkg:36:12)
    at node:internal/bootstrap/pkg:41:18
    at node:internal/bootstrap/pkg:48:4
    at node:internal/bootstrap/pkg:49:2

Node.js v18.19.0

Also the CI here for the latest do not pass the yarn test.

Expected Behavior

Builds should work.

CI should pass tests.

To Reproduce

Bundle code with the version 5.11.2.

@Mikescops
Copy link
Author

@robertsLando thanks for your work here btw.

Link to the reported issue on our builds: Dashlane/dashlane-cli#211

LInk to the failed CI: https://github.com/yao-pkg/pkg/actions/runs/7797115337/job/21263105246

@timokoessler
Copy link

I get the same error when I create binaries for Windows or Linux on my Windows device with Node.js v20 that target node-18-*. If I use node-20-* as the target, the problem does not occur.

@robertsLando
Copy link
Member

There must be something wrong with nodejs 18.19 patches, could someone check them? I will give it a look on Monday

@robertsLando
Copy link
Member

Ok found the issue, release is coming

@robertsLando
Copy link
Member

Try 5.11.3 and let me know

@Mikescops
Copy link
Author

Works for me! Thanks 🎉

@paulish
Copy link

paulish commented Feb 14, 2024

Now it does not work with node:18-alpine docker

@robertsLando
Copy link
Member

@paulish PKG Version and error?

@paulish
Copy link

paulish commented Feb 15, 2024

@robertsLando seems that the problem appeared in earlier version. I described it in a separate issue.

@robertsLando
Copy link
Member

I have a feel your problem isn't related to this issue BTW, it's something else

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