Skip to content

App launcher: Allow to pass in stdout and stderr paths#254

Merged
iLLiCiTiT merged 8 commits intodevelopfrom
enhancement/allow-to-pass-stdout-err
Sep 17, 2025
Merged

App launcher: Allow to pass in stdout and stderr paths#254
iLLiCiTiT merged 8 commits intodevelopfrom
enhancement/allow-to-pass-stdout-err

Conversation

@iLLiCiTiT
Copy link
Copy Markdown
Member

@iLLiCiTiT iLLiCiTiT commented Sep 8, 2025

Changelog Description

App laucher can pass stdout and stderr paths from the json file. Or allows to not change stdout and stderr if json explicitly tells to not to.

Testing notes:

Base

  1. App launcher works.

Redirection

  1. Prepare testing json file.
  2. Add "stdout" and "stderr" with paths to custom paths.
  3. Run applauncher with path to the json file.
  4. The stdout and stderr files are used.

@iLLiCiTiT iLLiCiTiT added the type: enhancement Improvement of existing functionality or minor addition label Sep 8, 2025
@iLLiCiTiT iLLiCiTiT self-assigned this Sep 8, 2025
@iLLiCiTiT iLLiCiTiT requested a review from antirotor September 8, 2025 15:27
@antirotor
Copy link
Copy Markdown
Member

Just to add to the description - this is only for linux

antirotor added a commit to ynput/ayon-applications that referenced this pull request Sep 11, 2025
@iLLiCiTiT
Copy link
Copy Markdown
Member Author

Added stdout and stderr handling to python variant. It should also fix issue with the script itself.

@antirotor
Copy link
Copy Markdown
Member

I reverted the commit, at the end, it belongs to its own PR. Sorry

@iLLiCiTiT iLLiCiTiT merged commit 80bdf08 into develop Sep 17, 2025
1 check passed
@iLLiCiTiT iLLiCiTiT deleted the enhancement/allow-to-pass-stdout-err branch September 17, 2025 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

type: enhancement Improvement of existing functionality or minor addition

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants