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

Maven output mixed with computation output on stdout #3571

Closed
1 of 5 tasks
aztecrex opened this issue Aug 7, 2019 · 0 comments · Fixed by #3624
Closed
1 of 5 tasks

Maven output mixed with computation output on stdout #3571

aztecrex opened this issue Aug 7, 2019 · 0 comments · Fixed by #3624
Assignees
@RomainMuller
Labels
bug This issue is a bug. language/java Related to Java bindings

Comments

@aztecrex
Copy link

aztecrex commented Aug 7, 2019 •
edited

  • I'm submitting a ...

    • 🪲 bug report
    • 🚀 feature request
    • 📚 construct library gap
    • ☎️ security issue or vulnerability => Please see policy
    • ❓ support request => Please see note at the top of this template.

  • What is the current behavior?
    If the current behavior is a 🪲bug🪲: Please provide the steps to reproduce

cdk list | grep 'BUILD SUCCESS'

Produces [INFO] BUIlD SUCCESS. In fact all maven output is sent to stdout along with the computed output (list of stacks).

  • What is the expected behavior (or behavior of feature suggested)?

Do not sent maven output to stdout. Suppress it unless asked for or send it to stderr.

  • What is the motivation / use case for changing the behavior or adding this feature?

Scripts need to distinguish between informational output and data. Pipes won't work well under the current behavior.

  • Please tell us about your environment:

    • CDK CLI Version: 1.3.0
    • Module Version: xx.xx.xx
    • OS: OSX Mojave
    • Language: Java

  • Other information (e.g. detailed explanation, stacktraces, related issues, suggestions how to fix, links for us to have context, eg. associated pull-request, stackoverflow, gitter, etc)
@aztecrex aztecrex added the needs-triage This issue or PR still needs to be triaged. label Aug 7, 2019
@aztecrex aztecrex changed the title Maven output mixed with command output to stdout Maven output mixed with computation output on stdout Aug 7, 2019
@eladb eladb added language/java Related to Java bindings bug This issue is a bug. and removed needs-triage This issue or PR still needs to be triaged. labels Aug 12, 2019
eladb pushed a commit that referenced this issue Aug 12, 2019
fix(java): surpress maven output in cdk.json
8272706 
Update the java app init template to use use `--quiet` when running `mvn exec`
so that output can be parsed.

Fixes #3571
@eladb eladb mentioned this issue Aug 12, 2019
Merged
@mergify mergify bot closed this as completed in #3624 Aug 12, 2019
mergify bot pushed a commit that referenced this issue Aug 12, 2019
@mergify
fix(java): surpress maven output in cdk.json (#3624)
02e097b 
Update the java app init template to use use `--quiet` when running `mvn exec`
so that output can be parsed.

Fixes #3571
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@RomainMuller RomainMuller

Labels
bug This issue is a bug. language/java Related to Java bindings
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

fix(java): surpress maven output in cdk.json aws/aws-cdk
3 participants
@RomainMuller @eladb @aztecrex