Re: [PATCH] get_maintainer: lazy load JSON::PP only when --json is used
From: Oleg Nesterov
Date: Fri Oct 30 2026 - 20:11:37 EST
On 10/30, Joe Perches wrote:
>
> On Sat, 2026-10-31 at 00:35 +0100, Oleg Nesterov wrote:
> > On 10/30, Joe Perches wrote:
> > >
> > > I think that was added in Linus' 7.2 in mid August and in -next since April
> >
> > Yes, I hit this problem months ago, but didn't have time to send the fix.
>
> Nor apparently have time to send an email to the get_maintainer
> maintainer about it either.
No. I knew that this problem is trivial and I can send the trivial fix
when I have time to check the code and write the changelog.
> > Great, then perhaps you can ACK my trivial patch? Then we can discuss
> > the possible improvements.
>
> Maybe when it's done without needing updates.
Sorry, I don't understand...
> In the meantime, you have an easy workaround with dnf install perl-core.
Sure. Or I guess I can install perl-JSON-PP.noarch... most probably this
is the necessary module.
OK. So in essence you NACK this change. OK, maintainer is always right.
I won't argue even if I don't understand why you dislike this trivial change.
Afaics it can't make any regression, it just allows to continue to use
get_maintainer.pl without JSON::PP, as it was always possible.
Oleg.