Gábor Csárdi

Results 2348 comments of Gábor Csárdi

So, I agree that this is not great, and I'll - investigate if this is the best we can do, - If yes, then try to have a better error...

Thanks! Unfortunately, this approach is not going to work, for a number of reasons. pak does not use the PPM API any more, to be able to match system requirements...

Right. The sysreqs.R and sysreqs2.R files are used in the pkgdepends package.

With https://github.com/r-lib/pkgcache/pull/96 and https://github.com/r-lib/pkgcache/pull/97 (devel) pak now supports system requirements for Bioconductor packages: ```r > pak::pkg_sysreqs(c("r-lib/xml2", "CHRONOS")) ── Install scripts ──────────────────────────────────────────────────────────────── Ubuntu 22.04 ── apt-get -y update apt-get -y install...

Sure, that makes sense.

I think the best would be to add an option or function to cli to do this.

I would add a `cli_extract_vars()` (internal?) function.

> I also see that all of the functions like `cli_abort()` that support inline formatting are doing so through a call to `cli::format_inline()`. That would be very surprising to me....

I would vote for option 3. Would you like to submit a PR? (No pressure.)

Thanks! You can set environment variables for the `setup-renv` step in your workflow, would that work for you?