-
Notifications
You must be signed in to change notification settings - Fork 298
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.
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
cmd/cue: "file already exists" when using --outfile=/dev/null #3624
Comments
@kshpytsya thanks for raising. Please can you confirm the details present in the "Bug Report" issue template? https://github.com/cue-lang/cue/issues/new/choose Happy for you to edit this issue, or create a new one - whichever you prefer. |
So much for "sat". $ cue version cue version cue version v0.11.0 go version go1.23.3 -buildmode exe -compiler gc DefaultGODEBUG asynctimerchan=1,gotypesalias=0,httpservecontentkeepheaders=1,tls3des=1,tlskyber=0,x509keypairleaf=0,x509negativeserial=1 CGO_ENABLED 1 GOARCH amd64 GOOS linux GOAMD64 v1 cue.lang.version v0.11.0 Does this issue reproduce with the latest stable release?Yes What did you do?
What did you expect to see?zennish empty output and 0 exit code. What did you see instead?
and 1 for exit code. |
The reason for having a standard form for bug reports is that, when reviewing a number of them, having such a standard form makes the human job of parsing them easier. It also confirms exactly what version was in use, which is then further relevant for confirming any fixes. Hopefully that's a reasonable position. In any case, thank you for taking the time to raise an initial issue, and follow up with details. |
Happy to review a change to fix this edge case, as I imagine it's going to be rather simple. We would need a testscript test as well, even if it is linux-specific, but it could possibly be portable via |
Just for your awareness, @kshpytsya, the
Because the command also permits files to be specified, with a qualifier prefix indicating their encoding if it can't be inferred from the filename extension, an alternative alternative is:
I don't believe either of these invalidate the need for a fix for your initial issue, but they might be useful. |
I noticed the same error message writing to both |
Sapienti sat.
P.S. I am aware of the following alternative:
The text was updated successfully, but these errors were encountered: