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

Specify how to treat a file that contains interleaved stdout/stderr #110

Closed
ghost opened this issue Feb 28, 2018 · 2 comments
Closed

Specify how to treat a file that contains interleaved stdout/stderr #110

ghost opened this issue Feb 28, 2018 · 2 comments
Labels
bug CSD.1 Will be fixed in CSD.1. enhancement resolved-fixed

Comments

@ghost
Copy link

ghost commented Feb 28, 2018

This issue was approved for resolution in CSD.1 in the TC meeting of 2/28/2018.

In #92, we introduced properties on the invocation object to specify the locations of files containing the contents of the stdin, stdout, and stderr streams. But many engineering systems redirect both stdout and stderr to the same file. How should SARIF represent that.
@ghost ghost added bug CSD.1 Will be fixed in CSD.1. labels Feb 28, 2018
@ghost ghost self-assigned this Feb 28, 2018
@michaelcfanning
Copy link
Contributor

Resolution here: stderr is present, the tool populates it. stderr may be empty in this case. if stderr is not present at all in the SARIF, it is an indicator that it is not separated as a stream and stderr contents may be interleaved with stdout (if present)

@michaelcfanning
Copy link
Contributor

We propose "stdOutStdErr" as a new propery to cover interleaving case.

ghost pushed a commit that referenced this issue Mar 11, 2018
Change draft for #110 (interleaved stdout/stderr)
4f3772b
ghost pushed a commit that referenced this issue Mar 16, 2018
Merge changes for #110 (stdoutStderr)
862c475
@ghost ghost closed this as completed Mar 16, 2018
ghost pushed a commit that referenced this issue Mar 21, 2018
Add #110 to schema changes doc
39a0132
ghost pushed a commit that referenced this issue Mar 23, 2018
Schema: Implement #110 (stdoutStderr)
2d2252c
@ghost ghost mentioned this issue Mar 31, 2018
Merged
This issue was closed.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
bug CSD.1 Will be fixed in CSD.1. enhancement resolved-fixed
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
1 participant
@michaelcfanning