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.

Sign up for GitHub

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

Jump to bottom

[BUGFIX] Consider output of stderr in successful cases #206

Open
wants to merge 1 commit into
base: main
Choose a base branch
from
Open

[BUGFIX] Consider output of stderr in successful cases #206

wants to merge 1 commit into from

Conversation

andreaskienast
Copy link

Git writes the output of some commands into stderr instead of stdout (e.g. push). Using the option --porcelain doesn't always help in such cases as other information may be missing. To get the full output, stderr is now fetched in case stdout is empty.

Fixes: #205

@andreaskienast
[BUGFIX] Consider output of stderr in successful cases
c728377 
Git writes the output of some commands into `stderr` instead of
`stdout` (e.g. `push`). Using the option `--porcelain` doesn't always
help in such cases as other information may be missing. To get the full
output, `stderr` is now fetched in case `stdout` is empty.

Fixes: gitonomy#205
andreaskienast added a commit to TYPO3GmbH/site-intercept that referenced this pull request Jul 11, 2023
@andreaskienast
[BUGFIX] Use patched gitonomy/gitlib dependency
035d32d 
The library `gitonomy/gitlib` has an issue with fetching the output of
Git commands, as some output is written to `stderr`. There's a pending
pull request addressing this issue [1], which is now installed instead.

[1] gitonomy/gitlib#206
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Reviewers
No reviews
Assignees
No one assigned
Labels
None yet
Milestone
No milestone
Development

Successfully merging this pull request may close these issues.

Library takes stdout into consideration only
1 participant
@andreaskienast