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: wait for windows app to exit as well #18

Closed
wants to merge 24 commits into from

Conversation

kiennq
Copy link

@kiennq kiennq commented Nov 2, 2024

Fix #16

kiennq and others added 24 commits March 27, 2020 22:11
Fix 'Could not create process with command' error
* Fix the detection of targets that use the GUI subsystem

* Allocate path on the stack
I've sometimes fiddled with scripts and shims for shortcuts and I really would've appreciated having this error tell me it's the shim telling me that I'm dumb and set up my path wrong, I hope this helps other fiddlers like me in the future
* build: Add zig build system support

Signed-off-by: Chawye Hsu <[email protected]>

* note about recent zig requirement

Signed-off-by: Chawye Hsu <[email protected]>

* add build.ps1

Signed-off-by: Chawye Hsu <[email protected]>

* build: move zig build script to build folder

Signed-off-by: Chawye Hsu <[email protected]>

* build(zig): set subsystem to console

Signed-off-by: Chawye Hsu <[email protected]>

* conditional corecrt_wstdio.h inclusion

Signed-off-by: Chawye Hsu <[email protected]>

* build: fix build.ps1 and support building for x86

Signed-off-by: Chawye Hsu <[email protected]>

* build: update build.ps1

Signed-off-by: Chawye Hsu <[email protected]>

* chore: i386 -> ia32

Signed-off-by: Chawye Hsu <[email protected]>

---------

Signed-off-by: Chawye Hsu <[email protected]>
@kiennq kiennq closed this Nov 2, 2024
@kiennq kiennq deleted the fix/wait-windows-app branch November 2, 2024 20:39
@kiennq kiennq restored the fix/wait-windows-app branch November 2, 2024 20:41
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.

Doesn't detect GUI apps
6 participants