On Fri, Aug 5, 2022 at 7:15 PM Ionen Wolkens <io...@gentoo.org> wrote:
>
> On Fri, Aug 05, 2022 at 06:47:42PM -0400, Mike Pagano wrote:
> > Based upon code from check-reqs.eclass by Andreas Sturmlechner
> >
> > Provide support for users who requested the ability to skip
> > CONFIG_* checks. (e.g. from within a chroot for testing purposes)
> >
> > Bug: https://bugs.gentoo.org/862315
> > Signed-off-by: Mike Pagano <mpag...@gentoo.org>
> > ---
> >   eclass/linux-info.eclass | 11 ++++++++++-
> >   1 file changed, 10 insertions(+), 1 deletion(-)
> >
> > diff --git a/eclass/linux-info.eclass b/eclass/linux-info.eclass
> > index 7e130062a..59e86490f 100644
> > --- a/eclass/linux-info.eclass
> > +++ b/eclass/linux-info.eclass
> > @@ -29,6 +29,15 @@
> >   # A Couple of env vars are available to effect usage of this eclass
> >   # These are as follows:
> >
> > +
> > +# @ECLASS_VARIABLE: CHECKCONFIG_DONOTHING
> > +# @USER_VARIABLE
> > +# @DEFAULT_UNSET
> > +# @DESCRIPTION:
> > +# Do not error out in check_extra_config if CONFIG settings are not met.
> > +# This is a user flag and should under _no circumstances_ be set in the 
> > ebuild.
> > +[[ -n ${I_KNOW_WHAT_I_AM_DOING} ]] && CHECKCONFIG_DONOTHING=1
>
> So this enables it if I_KNOW_WHAT_I_AM_DOING is set?
>
> Generally I feel giving more purposes to that variable is a bad idea.
> What starts out as "don't bother me about size/ram checks" ignores
> a lot of other things that may be not be expected.

I agree. Please avoid abusing the I_KNOW_WHAT_I_AM_DOING variable any further.

Reply via email to