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

Support For Prebaked NODE_OPTIONS defaults? #616

Open
EricMCornelius opened this issue Apr 29, 2019 · 4 comments · May be fixed by #949
Open

Support For Prebaked NODE_OPTIONS defaults? #616

EricMCornelius opened this issue Apr 29, 2019 · 4 comments · May be fixed by #949

Comments

@EricMCornelius
Copy link

Is this a BUG or a FEATURE REQUEST?:
Feature Request

What happened:
I am evaluating switching from pkg to nexe, given the former is falling significantly behind in support.

One major feature of use that pkg supports is setting default node command line arguments, which it seems isn't possible today in nexe.

In particular, it would be useful to have the ability to toggle features such as --experimental-modules, --http-parser, --tls-min-v1.3, etc.

I do see that setting v8 options is possible during compilation, but it seems that doesn't extend to default node-only options. If I understand properly, it looks like this is accomplished by injecting command line arguments into node_main.cc in pkg here: https://github.com/zeit/pkg-fetch/blob/master/patches/node.v10.15.3.cpp.patch#L535

What you expected to happen:
It seems it would be useful to be able to inject default node command line arguments.

How to reproduce it (as minimally and precisely as possible):
Unsupported feature

Anything else we need to know?:
Nope

Environment

  • Platform(OS/Version):
    all
  • Host Node Version:
    12.0.0
  • Target Node Version:
    12.0.0
  • Nexe version:
    3.2.0
  • Python Version:
    2.7.15

Happy to help pitch in if this is deemed a worthwhile feature to support in nexe core. Otherwise, probably can just handle this myself with a custom patch.

@calebboyd
Copy link
Member

I think it does make sense! I'd be willing to review a PR.

@calebboyd
Copy link
Member

related #592

@Ricardonacif
Copy link

I would like to help getting this to work. Should we patch Node and add the passed options as default when building it? Is this the way?

@pipobscure pipobscure linked a pull request Dec 19, 2021 that will close this issue
@anvaymishra1
Copy link

Any update on the nexe version that enables us to do this ?

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

Successfully merging a pull request may close this issue.

4 participants