Hi, Le mer. 27 mai 2020 à 20:04, Nilesh Patra <npatra...@gmail.com> a écrit : > > The second test (needing the library) failed on gitlab-ci, w/o adding > r-cran-xml and r-cran-runit, possible because this doesn't pick up depends > from test-depends. > This passed on my local machine though. Since I wanted to be sure, I moved > this there.
There is probably a bug on the salsa-ci side, because both tests passed on my computer. Let's skip salsa-ci for now. I have moved back these dependencies to test-deps and I have updated your patch (str_replace_all --> gsub) to not add a new dependency. You probably already noticed, the debian/test/control files for R packages require a lot of manual work to keep the list of test-deps up-to-date. Because I don't want to do this manually, I moved the code to tests these packages into pkg-r-autopkgtest. The list of test-deps is automatically generated at run time from the DESCRIPTION file. Currently, it is enabled only for bioconductor packages, if there is no big bug with this transition, we will be able to remove the debian/test/control files and to enable this for other R packages. Thanks again for #961591. Best, Dylan
