r-hub / rhub

R-hub API client
https://r-hub.github.io/rhub/
Other
353 stars 51 forks source link

Install pandoc #582

Closed gaborcsardi closed 4 months ago

gaborcsardi commented 1 year ago

Always? Should be fast enough.

ms609 commented 7 months ago

Could this be as simple as adding

      - uses: r-lib/actions/setup-pandoc@v2

as a step in the rhub.yaml template file?

This adds 1s when run, which I think counts as "fast enough"!

ms609 commented 7 months ago

Addendum: now I come to try this with valgrind/asan I see that this suggestion works for other-platforms:, but not linux-containers:, where sudo is not available – I see now that this is perhaps more complicated than it first appears!

gaborcsardi commented 7 months ago

where sudo is not available

You can run the command without sudo.

ms609 commented 7 months ago

I didn't see an obvious way to configure the setup-pandoc action to run without sudo.

gaborcsardi commented 7 months ago

Oh, right, sry, I meant that you can install pandoc with the OS package manager. setup-pandoc is not going to work on the Fedora containers, anyway.

gaborcsardi commented 7 months ago

But again, in these cases you don't actually need to build the vignettes.

danielinteractive commented 6 months ago

Agree that this would be important, I also just got

Error running filter pandoc-citeproc:
Could not find executable pandoc-citeproc

on clang18.

chrismainey commented 5 months ago

Agree that this would be important, I also just got

Error running filter pandoc-citeproc:
Could not find executable pandoc-citeproc

on clang18.

Also receiving this, missing pandoc-citeproc on ubuntu clang and 'linux (R-devel)' to Ubuntu 22.04.4. Checks on same package complete fine on windows with R-devel, other linux versions with R-current, and mac OS with R-current and R-devel.

Log: Run r-hub/actions/run-check@v1 Run r-lib/actions/check-r-package@v2 Run ## -------------------------------------------------------------------- ── R CMD build ─────────────────────────────────────────────────────────────────

gaborcsardi commented 4 months ago

@danielinteractive @chrismainey Can you please link to a failing build? Thanks!

gaborcsardi commented 4 months ago

Never mind, this should be fixed now: https://github.com/gaborcsardi/FunnelPlotR/actions/runs/9018187734/job/24778227972

Please reopen or open another issue if you still have problems with Pandoc. Thanks!