Re: [Buildroot] [PATCH 1/1] package/pkg-autotools: fix runtime path

[Thomas Petazzoni via buildroot <buildroot@buildroot.org>]

Hello Fabrice,

On Fri, 29 Jul 2022 13:41:37 +0200
Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote:

> Commit 505b5a6cdd337c2fd5965aa875a1a53419d62ed1 wrongly stated that "a
> configure script bundled with a package may be generated with an older
> version of Autotools, that may not yet support the --runstatedir option.
> This will give an "Unsupported arguments" warning and the option will be
> ignored.". Indeed, this will raise the following build failure with
> attr, libtirpc, readline, etc.:
> 
> configure: error: unrecognized option: `--runstatedir=/run'
> 
> So to support configure scripts using older version, the variable is set
> directly instead of using the --runstatedir argument.
> 
> Fixes:
>  - http://autobuild.buildroot.org/results/c853f14773d875be30e32ae24daf80e181fef34e
>  - http://autobuild.buildroot.org/results/045211cd66cf78c3d89d2d8ac41ec4edf0b0ee58
>  - http://autobuild.buildroot.org/results/8c5160eef99de2f1f72f33d5e5ea94d417855b2c
> 
> This reverts commit 505b5a6cdd337c2fd5965aa875a1a53419d62ed1.

Thanks for the fix, but you are not reverting
505b5a6cdd337c2fd5965aa875a1a53419d62ed1. Commit
505b5a6cdd337c2fd5965aa875a1a53419d62ed1 added --runstatedir=/run, you
are changing it to runstatedir=/run. So yes, you are reverting to the
initial patch proposal, but strictly speaking, your patch is not
reverting 505b5a6cdd337c2fd5965aa875a1a53419d62ed1, i.e git revert
505b5a6cdd337c2fd5965aa875a1a53419d62ed1 would not have the same effect
as your patch.

Thanks!

Thomas
-- 
Thomas Petazzoni, co-owner and CEO, Bootlin
Embedded Linux and Kernel engineering and training
https://bootlin.com
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot