Skip to content
This repository has been archived by the owner on Apr 29, 2021. It is now read-only.

Separate stdout and stderr in run #180

Open
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

fearphage
Copy link

Introduces new variables:
$output_stdout, $output_stderr, $lines_stdout, $lines_stderr
Existing $output and $lines variables remain.

I found this while looking into #55 and noticed it never made it into a PR.

@duggan Thanks for the excellent contribution. I hope you don't mind.

@ahippo Was previously tagged in the branch as well.

Any thoughts on merging this @sstephenson? There was already a good bit of discussion about the topic in #55.

This is not my code. I'm just making a PR for code that I believe adds the feature that I would like.
I see it cannot be merged cleanly. I wouldn't mind cleaning it up to get it merged. Let me know.

Ross Duggan added 3 commits July 8, 2014 18:47
 - Introduces new variables:
   $output_stdout, $output_stderr, $lines_stdout, $lines_stderr
   Existing `$output` and `$lines` variables remain.
@ahippo
Copy link
Contributor

ahippo commented Sep 21, 2016

I found this while looking into #55 and noticed it never made it into a PR.

@duggan Thanks for the excellent contribution. I hope you don't mind.

@ahippo Was previously tagged in the branch as well.

Any thoughts on merging this @sstephenson? There was already a good bit of discussion about the topic in #55.

This is not my code. I'm just making a PR for code that I believe adds the feature that I would like.
I see it cannot be merged cleanly. I wouldn't mind cleaning it up to get it merged. Let me know

Yeah, it would be great to have this merged!
There is some tricky code though.
I can also help if needed.

@ztombol ztombol mentioned this pull request Dec 13, 2016
18 tasks
yarikoptic pushed a commit to neurodebian/bats that referenced this pull request Aug 6, 2019
Added instructions for installing on Git Bash
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

3 participants