<div dir="auto"><div><br><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, 28 May 2020, 01:43 Dylan Aïssi, <<a href="mailto:daissi@debian.org">daissi@debian.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Hi,<br>
<br>
Le mer. 27 mai 2020 à 20:04, Nilesh Patra <<a href="mailto:npatra974@gmail.com" target="_blank" rel="noreferrer">npatra974@gmail.com</a>> a écrit :<br>
><br>
> 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.<br>
> This passed on my local machine though. Since I wanted to be sure, I moved this there.<br>
<br>
There is probably a bug on the salsa-ci side, because both tests<br>
passed on my computer. Let's skip salsa-ci for now.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Ahh, alrighty.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
I have moved back these dependencies to test-deps and I have updated<br>
your patch (str_replace_all --> gsub) to not add a new dependency.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That's good, and makes sense. I didn't know about this, possibly because I don't speak R :P</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
You probably already noticed, the debian/test/control files for R<br>
packages require a lot of manual work to keep the list of test-deps<br>
up-to-date. Because I don't want to do this manually, I moved the code<br>
to tests these packages into pkg-r-autopkgtest. The list of test-deps<br>
is automatically generated at run time from the DESCRIPTION file<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">That's nice!</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Currently, it is enabled only for bioconductor packages, if there is<br>
no big bug with this transition, we will be able to remove the<br>
debian/test/control files and to enable this for other R packages.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">Got it, thanks for the explanation.</div><div dir="auto"><br></div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
Thanks again for #961591.<br></blockquote></div></div><div dir="auto"><br></div><div dir="auto">:)</div><div dir="auto">And thanks a lot for letting me have this upload (i.e. my name on d/ch for this upload)</div><div dir="auto"><br></div><div dir="auto">Kind regards,</div><div dir="auto">Nilesh</div><div dir="auto"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
</blockquote></div></div></div>