On Fri, 26 Apr 2024 10:34:44 +0200
Arsen Arsenović <[email protected]> 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
>
> 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
