Skip to content

(windows) fix window flicker when forwarding CLI args to running inst…#12

Merged
ArthurHeitmann merged 4 commits intoArthurHeitmann:masterfrom
Vluurie:master
Dec 14, 2025
Merged

(windows) fix window flicker when forwarding CLI args to running inst…#12
ArthurHeitmann merged 4 commits intoArthurHeitmann:masterfrom
Vluurie:master

Conversation

@Vluurie
Copy link
Contributor

@Vluurie Vluurie commented Nov 27, 2025

…ance

@ArthurHeitmann
Copy link
Owner

It seems like the built updater no longer works. After clicking the update button, a new window flashes for a short moment, and then nothing happens.

I think it silently fails at this line https://github.com/ArthurHeitmann/F-SERVO/blob/master/lib/version/installRelease.dart#L99

However the updater cli doesn't output any logs. Not sure why.

image

@Vluurie
Copy link
Contributor Author

Vluurie commented Dec 7, 2025

I added an check if the args contain flags, then we will not go into headless arg forwarding.

@ArthurHeitmann
Copy link
Owner

Still the same behavior

The updater was started via shell command string which broke when
the installation path contained spaces. Now passes arguments as
array directly to Process.start, which handles quoting correctly.

Since shell redirection (> logfile) no longer works without runInShell,
the updater now writes its own log via --log-file parameter.
@ArthurHeitmann ArthurHeitmann merged commit 4a1ffbe into ArthurHeitmann:master Dec 14, 2025
1 check passed
@ArthurHeitmann
Copy link
Owner

✅ Works
✅ Cool feature
✅ Fixes old bug

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