Re: Multiple new R ports

From: Nuno Teixeira <eduardo_at_freebsd.org>
Date: Sat, 24 Feb 2024 10:15:55 UTC
Sugested packages are as they say: sugested.

Usually I only add packages to TEST_DEPENDS as required to a success test.
Also, please add a comment in Makefile when tests fail and respective
bug url if available.

Cheers,

Einar Bjarni Halldórsson <einar@isnic.is> escreveu (sábado, 24/02/2024
à(s) 09:32):
>
>
> > On 24 Feb 2024, at 06:24, Einar Bjarni Halldórsson <einar@isnic.is> wrote:
> >
> >> On 22 Feb 2024, at 11:43, Nuno Teixeira <eduardo@freebsd.org> wrote:
> >>
> >> I did take a look at review and noticed that TEST_DEPENDS are not used
> >> and that could means that test target works without any depends or
> >> ports are not tested at all.
> >> My curiosity is what do you think about the importance (or not) to
> >> have testunit feature working on R ports?
> >>
>
> I’ve got tests working on all the ports, except for R-cran-systemfonts.
> I have a PR [1] for upstream to fix that.
>
> Tests do give out notice that suggested packages aren’t installed to be tested,
> what’s your opinion on that? I don’t want to add too much to the packages and
> I definately don’t want to add options at this moment.
>
> Would it be fine to add the ports as is, without suggested packages?
>
> .einar
>
> [1] https://github.com/r-lib/systemfonts/pull/111



-- 
Nuno Teixeira
FreeBSD Committer (ports)