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

windows shell with forever #25

Closed
wants to merge 6 commits into from

Conversation

FrancoisMentec
Copy link

Context:
I made a script that update my wallpaper every 10 minutes, and used forever to run it as a daemon.

Problem:
Every time it set the wallpaper on windows there is a shell that pop for 0.5 sec and it's quite annoying, especially while playing games or watching movies.

Solution:
Use childrenProcess.spawn to set the wallpaper with detached at true and shell at false to prevent the opening of a shell.

FrancoisMentec and others added 5 commits July 13, 2017 22:38
Use childrenProcess.spawn to set the wallpaper with detached at true and
shell at false to prevent the opening of a shell with forever.
@FrancoisMentec
Copy link
Author

I added mate support from  @fa7ad and fixed errors from test (mostly typo)

@sindresorhus
Copy link
Owner

shell: false is moot as it's the default. Can you try setting shell: true instead and not setting detached: true.

@sindresorhus
Copy link
Owner

I added mate support from  @fa7ad and fixed errors from test (mostly typo)

That is completely unrelated to this PR and should not be included here.

@FrancoisMentec
Copy link
Author

shell: false is moot as it's the default. Can you try setting shell: true instead and not setting detached: true.

Tried, doesn't works, but removed shell false, useless

I added mate support from @fa7ad and fixed errors from test (mostly typo)
That is completely unrelated to this PR and should not be included here.

I needed it for a project so it was easier to get all modifications in one repo, probably not the cleanest way to do that.

@sindresorhus
Copy link
Owner

This is better solved in Node.js core: nodejs/node#15380

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 this pull request may close these issues.

2 participants