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

Fix Windows compiler invocation to not create new console windows #16

Closed
luser opened this issue Jul 8, 2016 · 7 comments
Closed

Fix Windows compiler invocation to not create new console windows #16

luser opened this issue Jul 8, 2016 · 7 comments

Comments

@luser
Copy link
Contributor

luser commented Jul 8, 2016

235239a changed how we spawn the sccache server process on Windows to make it detached, but in the process it made it so that when the server spawns a compiler process it creates a new console window. This is noticeable if you run cargo test test_sccache_command on Windows, you'll see several console windows pop up.

@luser luser added the bug label Jul 8, 2016
@luser
Copy link
Contributor Author

luser commented Nov 30, 2016

The original Python code was spawning the process with the DETACHED_PROCESS flag:

creationflags=8, # DETACHED_PROCESS

the Rust code is also passing CREATE_NEW_PROCESS_GROUP:

CREATE_UNICODE_ENVIRONMENT | DETACHED_PROCESS | CREATE_NEW_PROCESS_GROUP,

I'll try removing that and see if it makes a difference. If not it might be some other difference in the Rust process spawning code vs. Python's subprocess.

@luser
Copy link
Contributor Author

luser commented Nov 30, 2016

...that's not it.

1 similar comment
@luser
Copy link
Contributor Author

luser commented Dec 1, 2016

...that's not it.

@luser
Copy link
Contributor Author

luser commented Dec 9, 2016

Oh, hah! This bug existed in the Python implementation as well, and @glandium fixed it in this commit by passing CREATE_NO_WINDOW in the CreateProcess flags:
glandium@b259421

@RReverser
Copy link

@luser @alexcrichton I recently started using sccache, and am seeing a very similar issue (new console windows occasionally flickering with sccache). I know this one was fixed a long time ago, but I wonder if it was reintroduced again?

@Systemcluster
Copy link
Contributor

Using sccache 0.2.9, tested on Windows 10 with PowerShell and CMD, this is (still or again) occurring currently. @luser would you mind re-opening this issue?

@luser
Copy link
Contributor Author

luser commented Jul 1, 2019

Can you open a new issue on this? I think that'd be less confusing.

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

3 participants