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.

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

quarto embedded pandoc stdout redirected to stderr #2699

Closed
3 of 4 tasks
castedo opened this issue Oct 1, 2022 · 1 comment · Fixed by #11153
Closed
3 of 4 tasks

quarto embedded pandoc stdout redirected to stderr #2699

castedo opened this issue Oct 1, 2022 · 1 comment · Fixed by #11153
Assignees
Labels
bug Something isn't working
Milestone

Comments

@castedo
Copy link

castedo commented Oct 1, 2022

Bug description

The stdout of the embedded pandoc is getting redirected to stderr. For example:

quarto pandoc --version 2> /dev/null

outputs nothing (2> redirects stderr) but

pandoc --version 2> /dev/null

does output the version message.

This holds for other outputs of pandoc to stdout.

I am not directly affected by this, but I can imagine some piping scenarios where this could be pretty annoying.

VERSION:

$ quarto --version
1.1.251
$ cat /etc/os-release 
NAME="Fedora Linux"
VERSION="36 (Container Image)"
ID=fedora
VERSION_ID=36
VERSION_CODENAME=""
PLATFORM_ID="platform:f36"
PRETTY_NAME="Fedora Linux 36 (Container Image)"

Checklist

  • Please include a minimal, fully reproducible example in a single .qmd file? Please provide the whole file rather than the snippet you believe is causing the issue.
  • Please format your issue so it is easier for us to read the bug report.
  • Please document the RStudio IDE version you're running (if applicable), by providing the value displayed in the "About RStudio" main menu dialog?
  • Please document the operating system you're running. If on Linux, please provide the specific distribution.
@castedo castedo added the bug Something isn't working label Oct 1, 2022
@castedo castedo changed the title quarto pandoc redirects stdin to stderr quarto embedded pandoc stdout redirected to stderr Oct 1, 2022
@cscheid cscheid self-assigned this Oct 1, 2022
@cscheid cscheid modified the milestones: v1.2, v1.3 Oct 1, 2022
@cscheid
Copy link
Collaborator

cscheid commented Oct 5, 2022

Thanks for the report! We're aware of this issue and know how to fix it (I actually have a fix in a private branch ready to go). The main issue is that the regression surface of the fix is quite large, because we use stdout/stderr redirection from downstream binaries in a large number of places in our code. We'll revisit soon after we start work on 1.3.

@cscheid cscheid modified the milestones: v1.3, v1.4 Jan 20, 2023
@cscheid cscheid modified the milestones: v1.4, v1.5 Nov 28, 2023
@cscheid cscheid modified the milestones: v1.5, Future May 8, 2024
@cscheid cscheid modified the milestones: Future, v1.6 Sep 16, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
bug Something isn't working
Projects
None yet
Development

Successfully merging a pull request may close this issue.

2 participants