Set the output of %stdout %stderr and %result from executed tasks #26
Conversation
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
This commit adds the ability to return data from the termux-app TermuxService with a PendingIntent response It also provides a simple implementation of using it to return the process exit status, stdout and stderr outputs of the executed command for Tasker to use
Contributor
Author
|
Oh, actually, this also has the side effect of solving this issue! Perhaps not as a checkbox, but commands will no longer run asynchronously unless you set their timeout to "None", in which case they will revert back to continuing immediately. |
Contributor
Author
|
For the sake of collecting everything relevant, this also appears to be the subject of this issue, and should fix it too. |
Member
|
Thanks a lot! |
Member
|
This has been merged and released in version 0.4 of the Termux:Tasker app - great job! |
This was referenced Dec 16, 2020
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.
This suggestion is invalid because no changes were made to the code.
Suggestions cannot be applied while the pull request is closed.
Suggestions cannot be applied while viewing a subset of changes.
Only one suggestion per line can be applied in a batch.
Add this suggestion to a batch that can be applied as a single commit.
Applying suggestions on deleted lines is not supported.
You must change the existing code in this line in order to create a valid suggestion.
Outdated suggestions cannot be applied.
This suggestion has been applied or marked resolved.
Suggestions cannot be applied from pending reviews.
Suggestions cannot be applied on multi-line comments.
Suggestions cannot be applied while the pull request is queued to merge.
Suggestion cannot be applied right now. Please check back later.
This pull request adds the ability to return data from termux-app using PendingIntents.
It uses them to set the executed process
%result,%stdout, and%stderrfor Tasker to use.(This pull request requires the changes in this pull request in the main app to function properly and solves this issue.)