Received-SPF: pass (google.com: best guess record for domain of linux-nfs-owner@vger.kernel.org designates 209.132.180.67 as permitted sender) client-ip=209.132.180.67;
Date:   Mon, 20 Jan 2020 09:48:20 +0100
From:   Petr Vorel <petr.vorel@gmail.com>
To:     Giulio Benetti <giulio.benetti@benettiengineering.com>
Cc:     Steve Dickson <SteveD@redhat.com>, linux-nfs@vger.kernel.org,
        Mike Frysinger <vapier@gentoo.org>
Subject: Re: [nfs-utils RESENT PATCH 1/1] locktes/rpcgen: tweak how we
 override compiler settings
Message-ID: <20200120084820.GB31890@dell5510>
Reply-To: Petr Vorel <petr.vorel@gmail.com>
References: <20200105120502.765426-1-petr.vorel@gmail.com>
 <fb4dd073-856a-7807-eb71-f594e58732cb@RedHat.com>
 <bda9e61c-1a06-5ab3-339f-c38e9a68fb73@RedHat.com>
 <20200114183603.GA24556@dell5510>
 <db2c8006-5520-e34e-b759-42783f965d1c@RedHat.com>
 <5149cf4e-7d78-2cbd-99e4-a4cb66822308@benettiengineering.com>
MIME-Version: 1.0
Content-Type: text/plain; charset=us-ascii
Content-Disposition: inline
In-Reply-To: <5149cf4e-7d78-2cbd-99e4-a4cb66822308@benettiengineering.com>
User-Agent: Mutt/1.12.2 (2019-09-21)
Sender: linux-nfs-owner@vger.kernel.org
Precedence: bulk

Hi Giulio, Steve,

> > Note... Giulio's patch is doing something similar
> > https://lore.kernel.org/linux-nfs/20200115160806.99991-1-giulio.benetti@benettiengineering.com/T/#u

> > Does something like that as well as setting the AM_XXX help the your cross-compile?

> IMHO tools/* utility must be built with cross-compiler too, not with
> /usr/bin/gcc. Buildroot provide host-nfs-utils for that, especially for
> rpcgen.

> Please take a look at my WIP patch for bumping nfs-utils to latest in
> Buildroot:
> https://github.com/giuliobenetti/buildroot/commit/12671eb21d62a5474dc476381015069382775668

> and please note this line:
> --with-rpcgen=$(HOST_DIR)/bin/rpcgen

> that means that nfs-utils must use already host-nfs-utils/rpcgen instead of
> internal one to generate rpcs. This is why tools/* is not needed as host to
> build target. Indeed host-nfs-utils is built when nfs-utils is built. At
> least I understand this.

> Can you Petr confirm that?
> Because at this point the patch you're pointing is not needed.

Sorry, I overlooked this mail. Ack, it's correct for buildroot, as I already
wrote yesterday.
But not sure if the patch isn't needed for Gentoo packaging.
I guess if needed, it still can be rebased and applied later.

Kind regards,
Petr