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;
Subject: Re: [nfs-utils PATCH] locktest: Makefile.am: remove host compiler
 costraint
To:     Petr Vorel <petr.vorel@gmail.com>
Cc:     Giulio Benetti <giulio.benetti@benettiengineering.com>,
        linux-nfs@vger.kernel.org, Mike Frysinger <vapier@gentoo.org>
References: <20200115160806.99991-1-giulio.benetti@benettiengineering.com>
 <20200119191047.GB11432@dell5510>
 <6606c604-61ef-a3fa-8ced-1d9dfb822f64@RedHat.com>
 <20200122161353.GA21075@dell5510>
From:   Steve Dickson <SteveD@RedHat.com>
Message-ID: <3cf84430-9384-3a91-0e0e-21ff2d47d68d@RedHat.com>
Date:   Wed, 22 Jan 2020 11:44:17 -0500
User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:68.0) Gecko/20100101
 Thunderbird/68.2.2
MIME-Version: 1.0
In-Reply-To: <20200122161353.GA21075@dell5510>
Content-Type: text/plain; charset=windows-1252
Content-Language: en-US
Content-Transfer-Encoding: 7bit
Sender: linux-nfs-owner@vger.kernel.org
Precedence: bulk


On 1/22/20 11:13 AM, Petr Vorel wrote:
> Hi Steve,
> 
>>>> Currently locktest can be built only for host because CC_FOR_BUILD is
>>>> specified as CC, but this leads to build failure when passing CFLAGS not
>>>> available on host gcc(i.e. -mlongcalls) and most of all locktest would
>>>> be available on target systems the same way as rpcgen etc. So remove CC
>>>> and LIBTOOL assignments.
> 
>>>> Signed-off-by: Giulio Benetti <giulio.benetti@benettiengineering.com>
>>> Reviewed-by: Petr Vorel <petr.vorel@gmail.com>
>>> Tested-by: Petr Vorel <petr.vorel@gmail.com>
>>> NOTE: as I understand it's a compilation issue of a tool, so I didn't run
>>> rpcgen, I just test various compilation variants for buildroot.
>> Just to be clear... Giulio's patch, removing CC and LIBTOOL from the
>> locktest/Makefile.am does allows your cross build to succeed, correct?
> Yes, for buildroot it's ok. I'm just not able to verify Gentoo.
Thanks... Lets cross that path when get there... 

steved.

> 
>> steved.
> 
> Kind regards,
> Petr
>