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

Turn off node_modules deps resolving or the warning about missing deps #2551

Closed
wojtkowiak opened this issue Feb 5, 2018 · 6 comments · May be fixed by qcif/data-curator#563
Closed

Turn off node_modules deps resolving or the warning about missing deps #2551

wojtkowiak opened this issue Feb 5, 2018 · 6 comments · May be fixed by qcif/data-curator#563
Labels

Comments

@wojtkowiak
Copy link
Contributor

Hey,

I am in the proces of updating [email protected] to the newest one in meteor-desktop.
I did mention in the past 🙂 (#982 (comment)) that I am packing my node_modules into asar before building installer. Now electron-builder is complaining about unresolved deps as node_modules is not present at the time it processes the app.
I know that I can set the ELECTRON_BUILDER_ALLOW_UNRESOLVED_DEPENDENCIES but is still gives a warning (#982 (comment))

Would you add an option to disable this mechanism or at least disable this warning in any way?

Of course I can prepare the PR but just let me know how would you prefer this to be realised?

@develar
Copy link
Member

develar commented Feb 6, 2018

Please see Packager.productionDeps getter. I guess here we should check is node_modules dir exist if beforeBuild is already set. And do nothing if no dir and beforeBuild is set.

@wojtkowiak
Copy link
Contributor Author

Sth like this wojtkowiak@bbacd3e ?

But maybe better to condition this also with and additional env var?

@develar
Copy link
Member

develar commented Feb 6, 2018

Yep (except that I think no need to exists if beforeBuild not set).

maybe better to condition this also with and additional env var?

No, no pleasure for users to guess such env instead of just "build my app".

@develar develar added the feature label Feb 6, 2018
@wojtkowiak
Copy link
Contributor Author

Yep (except that I think no need to exists if beforeBuild not set).

Totally agree: wojtkowiak@17d33ea

Should I add a note in the docs about it somewhere? Maybe in the beforeBuild desc?

@develar
Copy link
Member

develar commented Feb 6, 2018

Maybe in the beforeBuild desc?

👍

@UberMouse
Copy link

I can't find any mention of this option in the current version. Is there some other way to not get electron-builder to check for missing deps? Our CI log ends up with nearly 600,000 lines because of all the unresolved deps spam in the logs, so looking for a solution to suppress it.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging a pull request may close this issue.

3 participants