[prev in list] [next in list] [prev in thread] [next in thread] 

List:       gentoo-dev
Subject:    Re: [gentoo-dev] [PATCH 1/3] To make "tc_has_feature ada" actually work
From:       Arsen_Arsenović <arsen () gentoo ! org>
Date:       2024-04-30 8:51:43
Message-ID: 86bk5rutb4.fsf () gentoo ! org
[Download RAW message or body]


Alfredo Tupone <tupone@gentoo.org> writes:

> On Fri, 26 Apr 2024 10:34:44 +0200
> Arsen Arsenović <arsen@gentoo.org> wrote:
>
>> looks OK at a glance, but could you summarize the issues the patch set
>> fixes in the commit messages (and reword them to follow convention
>> generally)?  it is important to have context while looking at a git
>> log.
>
>> > -	IUSE+=" ada"
>> > +	IUSE+=" ada" TC_FEATURES+=( ada )  
>
> The change is about adding TC_FEATURES+=( ada )
>
> Some lines towards the end there is 
>
>
> # TODO: Add a pkg_setup & pkg_pretend check for whether the active
> compiler # supports Ada.
> if tc_has_feature ada ; then
>     BDEPEND+=" ada? ( || ( sys-devel/gcc[ada] dev-lang/gnat-gpl[ada] )
> )"
> fi
>
>
> it looks to me that if TC_FEATURES+=( ada ) is missing, that dependency
> is not added

Indeed, but it would be nice to be able to tell that from the git log.

>> 
>> does this apply for all versions of GCC?
>
> I really don't know. Ada source are provided with gcc-8 too (the oldest
> version in the tree). I personally built gcc-10..13

That's good enough for our case I think (given we only have 8+ at all,
and only 11+ are unmasked).

Overall, probably OK.
-- 
Arsen Arsenović

["signature.asc" (application/pgp-signature)]

[prev in list] [next in list] [prev in thread] [next in thread]