Skip to content

fix(input): split app launch command arguments - #177

Open
Dexarim wants to merge 1 commit into
mmaher88:masterfrom
Dexarim:fix/launch-app-arguments
Open

Dexarim wants to merge 1 commit into
mmaher88:masterfrom
Dexarim:fix/launch-app-arguments

Conversation

@Dexarim

@Dexarim Dexarim commented Nov 9, 2026

Copy link
Copy Markdown

Summary

  • split an app-launch command into the executable and arguments before calling QProcess::startDetached
  • preserve quoted arguments instead of treating the complete command string as the executable path
  • log a warning when the executable cannot be started
  • add a regression test using an argument with spaces and a literal shell metacharacter

This is the first standalone slice extracted from #163, as requested in review.

Validation

  • build completed successfully
  • full CTest suite: 822/822 passed
  • extractor and package-dependency Python tests: 45/45 passed
  • README device table and Debian dependency drift checks passed
  • local pre-push gate passed

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.

1 participant