* [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf
@ 2021-11-18 21:06 Fabrice Fontaine
2021-11-19 23:32 ` Peter Seiderer
2021-11-22 20:42 ` Arnout Vandecappelle
0 siblings, 2 replies; 10+ messages in thread
From: Fabrice Fontaine @ 2021-11-18 21:06 UTC (permalink / raw)
To: buildroot; +Cc: Matt Weber, Fabrice Fontaine
For an unknown reason, the build fails since bump to version 1.4.60 in
commit d62b7f54f00291a50274409fb5510527fd725ffe on:
array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode
for (uint32_t i = 0; i < sz; ++i) {
^
Running autoreconf fixes the build.
Fixes:
- http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7
Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com>
---
package/lighttpd/lighttpd.mk | 2 ++
1 file changed, 2 insertions(+)
diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk
index 9c18a6038e..c39a5db20f 100644
--- a/package/lighttpd/lighttpd.mk
+++ b/package/lighttpd/lighttpd.mk
@@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause
LIGHTTPD_LICENSE_FILES = COPYING
LIGHTTPD_CPE_ID_VENDOR = lighttpd
LIGHTTPD_DEPENDENCIES = host-pkgconf
+# Issue in configure script provided in tarball
+LIGHTTPD_AUTORECONF = YES
LIGHTTPD_CONF_OPTS = \
--without-wolfssl \
--libdir=/usr/lib/lighttpd \
--
2.33.0
_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot
^ permalink raw reply related [flat|nested] 10+ messages in thread* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-18 21:06 [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf Fabrice Fontaine @ 2021-11-19 23:32 ` Peter Seiderer 2021-11-20 9:23 ` Fabrice Fontaine 2021-11-22 20:42 ` Arnout Vandecappelle 1 sibling, 1 reply; 10+ messages in thread From: Peter Seiderer @ 2021-11-19 23:32 UTC (permalink / raw) To: Fabrice Fontaine; +Cc: Matt Weber, buildroot Hello Fabrice, On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > For an unknown reason, the build fails since bump to version 1.4.60 in > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > for (uint32_t i = 0; i < sz; ++i) { > ^ > > Running autoreconf fixes the build. The difference is during the configure step: - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 And the use of the '-std=gnu11' flag for the compile steps..., any more lightweight option to enable C11? Regards, Peter > > Fixes: > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > --- > package/lighttpd/lighttpd.mk | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > index 9c18a6038e..c39a5db20f 100644 > --- a/package/lighttpd/lighttpd.mk > +++ b/package/lighttpd/lighttpd.mk > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > LIGHTTPD_LICENSE_FILES = COPYING > LIGHTTPD_CPE_ID_VENDOR = lighttpd > LIGHTTPD_DEPENDENCIES = host-pkgconf > +# Issue in configure script provided in tarball > +LIGHTTPD_AUTORECONF = YES > LIGHTTPD_CONF_OPTS = \ > --without-wolfssl \ > --libdir=/usr/lib/lighttpd \ _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-19 23:32 ` Peter Seiderer @ 2021-11-20 9:23 ` Fabrice Fontaine 2021-11-20 19:53 ` Peter Seiderer 0 siblings, 1 reply; 10+ messages in thread From: Fabrice Fontaine @ 2021-11-20 9:23 UTC (permalink / raw) To: Peter Seiderer; +Cc: Matt Weber, Buildroot Mailing List Hello Peter, Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > Hello Fabrice, > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > for (uint32_t i = 0; i < sz; ++i) { > > ^ > > > > Running autoreconf fixes the build. > > The difference is during the configure step: > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > And the use of the '-std=gnu11' flag for the compile steps..., any more > lightweight option to enable C11? Then, I assume that upstream forgot to regenerate the configure script after https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 I opened an issue here: https://redmine.lighttpd.net/issues/3116 > > Regards, > Peter > > > > > Fixes: > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > --- > > package/lighttpd/lighttpd.mk | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > index 9c18a6038e..c39a5db20f 100644 > > --- a/package/lighttpd/lighttpd.mk > > +++ b/package/lighttpd/lighttpd.mk > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > LIGHTTPD_LICENSE_FILES = COPYING > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > +# Issue in configure script provided in tarball > > +LIGHTTPD_AUTORECONF = YES > > LIGHTTPD_CONF_OPTS = \ > > --without-wolfssl \ > > --libdir=/usr/lib/lighttpd \ > Best Regards, Fabrice _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-20 9:23 ` Fabrice Fontaine @ 2021-11-20 19:53 ` Peter Seiderer 2021-11-20 20:15 ` Fabrice Fontaine 0 siblings, 1 reply; 10+ messages in thread From: Peter Seiderer @ 2021-11-20 19:53 UTC (permalink / raw) To: Fabrice Fontaine; +Cc: Matt Weber, Buildroot Mailing List Hello Fabrice, On Sat, 20 Nov 2021 10:23:11 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Hello Peter, > > Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > Hello Fabrice, > > > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > > for (uint32_t i = 0; i < sz; ++i) { > > > ^ > > > > > > Running autoreconf fixes the build. > > > > The difference is during the configure step: > > > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > > > And the use of the '-std=gnu11' flag for the compile steps..., any more > > lightweight option to enable C11? > Then, I assume that upstream forgot to regenerate the configure script after > https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 > I opened an issue here: https://redmine.lighttpd.net/issues/3116 Thanks for effort, but upstream seems very convinced of doing nothing wrong (and not ready to investigate further)...., despite using autoconf-2.69 (from 2012, see lighttpd-1.4.61/configure) and: $ diff -u lighttpd-1.4.59/configure lighttpd-1.4.61/configure [...] @@ -4440,298 +4460,6 @@ fi - case $ac_cv_prog_cc_stdc in #( - no) : - ac_cv_prog_cc_c99=no; ac_cv_prog_cc_c89=no ;; #( - *) : - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $CC option to accept ISO C99" >&5 -$as_echo_n "checking for $CC option to accept ISO C99... " >&6; } -if ${ac_cv_prog_cc_c99+:} false; then : - $as_echo_n "(cached) " >&6 -else - ac_cv_prog_cc_c99=no -ac_save_CC=$CC [...] The whole c99 check vanished...., and: $ diff -u lighttpd-1.4.59/configure.ac lighttpd-1.4.61/configure.ac [...] @@ -61,7 +61,6 @@ dnl Checks for programs. AC_PROG_CC -AC_PROG_CC_STDC AX_PROG_CC_FOR_BUILD LT_PATH_LD AC_PROG_INSTALL But there is although support for the meson build system in the sources..., maybe time to change to it? Regards, Peter > > > > > Regards, > > Peter > > > > > > > > Fixes: > > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > --- > > > package/lighttpd/lighttpd.mk | 2 ++ > > > 1 file changed, 2 insertions(+) > > > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > > index 9c18a6038e..c39a5db20f 100644 > > > --- a/package/lighttpd/lighttpd.mk > > > +++ b/package/lighttpd/lighttpd.mk > > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > > LIGHTTPD_LICENSE_FILES = COPYING > > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > > +# Issue in configure script provided in tarball > > > +LIGHTTPD_AUTORECONF = YES > > > LIGHTTPD_CONF_OPTS = \ > > > --without-wolfssl \ > > > --libdir=/usr/lib/lighttpd \ > > > Best Regards, > > Fabrice _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-20 19:53 ` Peter Seiderer @ 2021-11-20 20:15 ` Fabrice Fontaine 2021-11-20 20:23 ` Peter Seiderer 0 siblings, 1 reply; 10+ messages in thread From: Fabrice Fontaine @ 2021-11-20 20:15 UTC (permalink / raw) To: Peter Seiderer; +Cc: Matt Weber, Buildroot Mailing List Le sam. 20 nov. 2021 à 20:54, Peter Seiderer <ps.report@gmx.net> a écrit : > > Hello Fabrice, > > On Sat, 20 Nov 2021 10:23:11 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > Hello Peter, > > > > Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > Hello Fabrice, > > > > > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > > > for (uint32_t i = 0; i < sz; ++i) { > > > > ^ > > > > > > > > Running autoreconf fixes the build. > > > > > > The difference is during the configure step: > > > > > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > > > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > > > > > And the use of the '-std=gnu11' flag for the compile steps..., any more > > > lightweight option to enable C11? > > Then, I assume that upstream forgot to regenerate the configure script after > > https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 > > I opened an issue here: https://redmine.lighttpd.net/issues/3116 > > > Thanks for effort, but upstream seems very convinced of doing nothing wrong (and > not ready to investigate further)...., despite using autoconf-2.69 (from 2012, > see lighttpd-1.4.61/configure) and: > > $ diff -u lighttpd-1.4.59/configure lighttpd-1.4.61/configure > [...] > @@ -4440,298 +4460,6 @@ > fi > > > - case $ac_cv_prog_cc_stdc in #( > - no) : > - ac_cv_prog_cc_c99=no; ac_cv_prog_cc_c89=no ;; #( > - *) : > - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $CC option to accept ISO C99" >&5 > -$as_echo_n "checking for $CC option to accept ISO C99... " >&6; } > -if ${ac_cv_prog_cc_c99+:} false; then : > - $as_echo_n "(cached) " >&6 > -else > - ac_cv_prog_cc_c99=no > -ac_save_CC=$CC > [...] > > > The whole c99 check vanished...., and: > > $ diff -u lighttpd-1.4.59/configure.ac lighttpd-1.4.61/configure.ac > [...] > @@ -61,7 +61,6 @@ > > dnl Checks for programs. > AC_PROG_CC > -AC_PROG_CC_STDC > AX_PROG_CC_FOR_BUILD > LT_PATH_LD > AC_PROG_INSTALL > > > But there is although support for the meson build system in the sources..., > maybe time to change to it? I don't know if this is a good idea to change it before the 2021.11 release as meson will probably raise its own build failures. > > Regards, > Peter > > > > > > > > > Regards, > > > Peter > > > > > > > > > > > Fixes: > > > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > --- > > > > package/lighttpd/lighttpd.mk | 2 ++ > > > > 1 file changed, 2 insertions(+) > > > > > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > > > index 9c18a6038e..c39a5db20f 100644 > > > > --- a/package/lighttpd/lighttpd.mk > > > > +++ b/package/lighttpd/lighttpd.mk > > > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > > > LIGHTTPD_LICENSE_FILES = COPYING > > > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > > > +# Issue in configure script provided in tarball > > > > +LIGHTTPD_AUTORECONF = YES > > > > LIGHTTPD_CONF_OPTS = \ > > > > --without-wolfssl \ > > > > --libdir=/usr/lib/lighttpd \ > > > > > Best Regards, > > > > Fabrice > Best Regards, Fabrice _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-20 20:15 ` Fabrice Fontaine @ 2021-11-20 20:23 ` Peter Seiderer 2021-11-20 20:39 ` Fabrice Fontaine 0 siblings, 1 reply; 10+ messages in thread From: Peter Seiderer @ 2021-11-20 20:23 UTC (permalink / raw) To: Fabrice Fontaine; +Cc: Matt Weber, Buildroot Mailing List Hello Farbrice, On Sat, 20 Nov 2021 21:15:40 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Le sam. 20 nov. 2021 à 20:54, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > Hello Fabrice, > > > > On Sat, 20 Nov 2021 10:23:11 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > Hello Peter, > > > > > > Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > > > Hello Fabrice, > > > > > > > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > > > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > > > > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > > > > for (uint32_t i = 0; i < sz; ++i) { > > > > > ^ > > > > > > > > > > Running autoreconf fixes the build. > > > > > > > > The difference is during the configure step: > > > > > > > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > > > > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > > > > > > > And the use of the '-std=gnu11' flag for the compile steps..., any more > > > > lightweight option to enable C11? > > > Then, I assume that upstream forgot to regenerate the configure script after > > > https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 > > > I opened an issue here: https://redmine.lighttpd.net/issues/3116 > > > > > > Thanks for effort, but upstream seems very convinced of doing nothing wrong (and > > not ready to investigate further)...., despite using autoconf-2.69 (from 2012, > > see lighttpd-1.4.61/configure) and: > > > > $ diff -u lighttpd-1.4.59/configure lighttpd-1.4.61/configure > > [...] > > @@ -4440,298 +4460,6 @@ > > fi > > > > > > - case $ac_cv_prog_cc_stdc in #( > > - no) : > > - ac_cv_prog_cc_c99=no; ac_cv_prog_cc_c89=no ;; #( > > - *) : > > - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $CC option to accept ISO C99" >&5 > > -$as_echo_n "checking for $CC option to accept ISO C99... " >&6; } > > -if ${ac_cv_prog_cc_c99+:} false; then : > > - $as_echo_n "(cached) " >&6 > > -else > > - ac_cv_prog_cc_c99=no > > -ac_save_CC=$CC > > [...] > > > > > > The whole c99 check vanished...., and: > > > > $ diff -u lighttpd-1.4.59/configure.ac lighttpd-1.4.61/configure.ac > > [...] > > @@ -61,7 +61,6 @@ > > > > dnl Checks for programs. > > AC_PROG_CC > > -AC_PROG_CC_STDC > > AX_PROG_CC_FOR_BUILD > > LT_PATH_LD > > AC_PROG_INSTALL > > > > > > But there is although support for the meson build system in the sources..., > > maybe time to change to it? > I don't know if this is a good idea to change it before the 2021.11 > release as meson will probably raise its own build failures. Good point for the autoreconf solution (and meson for next ;-))... Regards, Peter > > > > Regards, > > Peter > > > > > > > > > > > > > Regards, > > > > Peter > > > > > > > > > > > > > > Fixes: > > > > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > > --- > > > > > package/lighttpd/lighttpd.mk | 2 ++ > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > > > > index 9c18a6038e..c39a5db20f 100644 > > > > > --- a/package/lighttpd/lighttpd.mk > > > > > +++ b/package/lighttpd/lighttpd.mk > > > > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > > > > LIGHTTPD_LICENSE_FILES = COPYING > > > > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > > > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > > > > +# Issue in configure script provided in tarball > > > > > +LIGHTTPD_AUTORECONF = YES > > > > > LIGHTTPD_CONF_OPTS = \ > > > > > --without-wolfssl \ > > > > > --libdir=/usr/lib/lighttpd \ > > > > > > > Best Regards, > > > > > > Fabrice > > > Best Regards, > > Fabrice _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-20 20:23 ` Peter Seiderer @ 2021-11-20 20:39 ` Fabrice Fontaine 2021-11-21 21:32 ` Peter Seiderer 0 siblings, 1 reply; 10+ messages in thread From: Fabrice Fontaine @ 2021-11-20 20:39 UTC (permalink / raw) To: Peter Seiderer; +Cc: Matt Weber, Buildroot Mailing List Le sam. 20 nov. 2021 à 21:23, Peter Seiderer <ps.report@gmx.net> a écrit : > > Hello Farbrice, > > On Sat, 20 Nov 2021 21:15:40 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > Le sam. 20 nov. 2021 à 20:54, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > Hello Fabrice, > > > > > > On Sat, 20 Nov 2021 10:23:11 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > Hello Peter, > > > > > > > > Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > > > > > Hello Fabrice, > > > > > > > > > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > > > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > > > > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > > > > > > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > > > > > for (uint32_t i = 0; i < sz; ++i) { > > > > > > ^ > > > > > > > > > > > > Running autoreconf fixes the build. > > > > > > > > > > The difference is during the configure step: > > > > > > > > > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > > > > > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > > > > > > > > > And the use of the '-std=gnu11' flag for the compile steps..., any more > > > > > lightweight option to enable C11? > > > > Then, I assume that upstream forgot to regenerate the configure script after > > > > https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 > > > > I opened an issue here: https://redmine.lighttpd.net/issues/3116 > > > > > > > > > Thanks for effort, but upstream seems very convinced of doing nothing wrong (and > > > not ready to investigate further)...., despite using autoconf-2.69 (from 2012, > > > see lighttpd-1.4.61/configure) and: > > > > > > $ diff -u lighttpd-1.4.59/configure lighttpd-1.4.61/configure > > > [...] > > > @@ -4440,298 +4460,6 @@ > > > fi > > > > > > > > > - case $ac_cv_prog_cc_stdc in #( > > > - no) : > > > - ac_cv_prog_cc_c99=no; ac_cv_prog_cc_c89=no ;; #( > > > - *) : > > > - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $CC option to accept ISO C99" >&5 > > > -$as_echo_n "checking for $CC option to accept ISO C99... " >&6; } > > > -if ${ac_cv_prog_cc_c99+:} false; then : > > > - $as_echo_n "(cached) " >&6 > > > -else > > > - ac_cv_prog_cc_c99=no > > > -ac_save_CC=$CC > > > [...] > > > > > > > > > The whole c99 check vanished...., and: > > > > > > $ diff -u lighttpd-1.4.59/configure.ac lighttpd-1.4.61/configure.ac > > > [...] > > > @@ -61,7 +61,6 @@ > > > > > > dnl Checks for programs. > > > AC_PROG_CC > > > -AC_PROG_CC_STDC > > > AX_PROG_CC_FOR_BUILD > > > LT_PATH_LD > > > AC_PROG_INSTALL > > > > > > > > > But there is although support for the meson build system in the sources..., > > > maybe time to change to it? > > I don't know if this is a good idea to change it before the 2021.11 > > release as meson will probably raise its own build failures. > > Good point for the autoreconf solution (and meson for next ;-))... Sure, if you agree, I'll let you handle the meson part ;-) > > Regards, > Peter > > > > > > > Regards, > > > Peter > > > > > > > > > > > > > > > > > Regards, > > > > > Peter > > > > > > > > > > > > > > > > > Fixes: > > > > > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > > > > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > > > --- > > > > > > package/lighttpd/lighttpd.mk | 2 ++ > > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > > > > > index 9c18a6038e..c39a5db20f 100644 > > > > > > --- a/package/lighttpd/lighttpd.mk > > > > > > +++ b/package/lighttpd/lighttpd.mk > > > > > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > > > > > LIGHTTPD_LICENSE_FILES = COPYING > > > > > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > > > > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > > > > > +# Issue in configure script provided in tarball > > > > > > +LIGHTTPD_AUTORECONF = YES > > > > > > LIGHTTPD_CONF_OPTS = \ > > > > > > --without-wolfssl \ > > > > > > --libdir=/usr/lib/lighttpd \ > > > > > > > > > Best Regards, > > > > > > > > Fabrice > > > > > Best Regards, > > > > Fabrice > Best Regards, Fabrice _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-20 20:39 ` Fabrice Fontaine @ 2021-11-21 21:32 ` Peter Seiderer 0 siblings, 0 replies; 10+ messages in thread From: Peter Seiderer @ 2021-11-21 21:32 UTC (permalink / raw) To: Fabrice Fontaine; +Cc: Matt Weber, Buildroot Mailing List Hello Fabrice, On Sat, 20 Nov 2021 21:39:13 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > Le sam. 20 nov. 2021 à 21:23, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > Hello Farbrice, > > > > On Sat, 20 Nov 2021 21:15:40 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > Le sam. 20 nov. 2021 à 20:54, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > > > Hello Fabrice, > > > > > > > > On Sat, 20 Nov 2021 10:23:11 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > > > Hello Peter, > > > > > > > > > > Le sam. 20 nov. 2021 à 00:33, Peter Seiderer <ps.report@gmx.net> a écrit : > > > > > > > > > > > > Hello Fabrice, > > > > > > > > > > > > On Thu, 18 Nov 2021 22:06:23 +0100, Fabrice Fontaine <fontaine.fabrice@gmail.com> wrote: > > > > > > > > > > > > > For an unknown reason, the build fails since bump to version 1.4.60 in > > > > > > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > > > > > > > > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > > > > > > for (uint32_t i = 0; i < sz; ++i) { > > > > > > > ^ > > > > > > > > > > > > > > Running autoreconf fixes the build. > > > > > > > > > > > > The difference is during the configure step: > > > > > > > > > > > > - checking for .../host/bin/arm-none-linux-gnueabi-gcc option to accept ISO C89... none needed > > > > > > + checking for .../host/bin/arm-none-linux-gnueabi-gcc option to enable C11 features... -std=gnu11 > > > > > > > > > > > > And the use of the '-std=gnu11' flag for the compile steps..., any more > > > > > > lightweight option to enable C11? > > > > > Then, I assume that upstream forgot to regenerate the configure script after > > > > > https://github.com/lighttpd/lighttpd1.4/commit/73865025729897d11494de56a29c77c4fc8e5702 > > > > > I opened an issue here: https://redmine.lighttpd.net/issues/3116 > > > > > > > > > > > > Thanks for effort, but upstream seems very convinced of doing nothing wrong (and > > > > not ready to investigate further)...., despite using autoconf-2.69 (from 2012, > > > > see lighttpd-1.4.61/configure) and: > > > > > > > > $ diff -u lighttpd-1.4.59/configure lighttpd-1.4.61/configure > > > > [...] > > > > @@ -4440,298 +4460,6 @@ > > > > fi > > > > > > > > > > > > - case $ac_cv_prog_cc_stdc in #( > > > > - no) : > > > > - ac_cv_prog_cc_c99=no; ac_cv_prog_cc_c89=no ;; #( > > > > - *) : > > > > - { $as_echo "$as_me:${as_lineno-$LINENO}: checking for $CC option to accept ISO C99" >&5 > > > > -$as_echo_n "checking for $CC option to accept ISO C99... " >&6; } > > > > -if ${ac_cv_prog_cc_c99+:} false; then : > > > > - $as_echo_n "(cached) " >&6 > > > > -else > > > > - ac_cv_prog_cc_c99=no > > > > -ac_save_CC=$CC > > > > [...] > > > > > > > > > > > > The whole c99 check vanished...., and: > > > > > > > > $ diff -u lighttpd-1.4.59/configure.ac lighttpd-1.4.61/configure.ac > > > > [...] > > > > @@ -61,7 +61,6 @@ > > > > > > > > dnl Checks for programs. > > > > AC_PROG_CC > > > > -AC_PROG_CC_STDC > > > > AX_PROG_CC_FOR_BUILD > > > > LT_PATH_LD > > > > AC_PROG_INSTALL > > > > > > > > > > > > But there is although support for the meson build system in the sources..., > > > > maybe time to change to it? > > > I don't know if this is a good idea to change it before the 2021.11 > > > release as meson will probably raise its own build failures. > > > > Good point for the autoreconf solution (and meson for next ;-))... > Sure, if you agree, I'll let you handle the meson part ;-) Done, see [1]... Regards, Peter [1] https://patchwork.ozlabs.org/project/buildroot/patch/20211121212937.13983-1-ps.report@gmx.net/ > > > > Regards, > > Peter > > > > > > > > > > Regards, > > > > Peter > > > > > > > > > > > > > > > > > > > > > Regards, > > > > > > Peter > > > > > > > > > > > > > > > > > > > > Fixes: > > > > > > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > > > > > > > > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > > > > > > --- > > > > > > > package/lighttpd/lighttpd.mk | 2 ++ > > > > > > > 1 file changed, 2 insertions(+) > > > > > > > > > > > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > > > > > > index 9c18a6038e..c39a5db20f 100644 > > > > > > > --- a/package/lighttpd/lighttpd.mk > > > > > > > +++ b/package/lighttpd/lighttpd.mk > > > > > > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > > > > > > LIGHTTPD_LICENSE_FILES = COPYING > > > > > > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > > > > > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > > > > > > +# Issue in configure script provided in tarball > > > > > > > +LIGHTTPD_AUTORECONF = YES > > > > > > > LIGHTTPD_CONF_OPTS = \ > > > > > > > --without-wolfssl \ > > > > > > > --libdir=/usr/lib/lighttpd \ > > > > > > > > > > > Best Regards, > > > > > > > > > > Fabrice > > > > > > > Best Regards, > > > > > > Fabrice > > > Best Regards, > > Fabrice > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-18 21:06 [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf Fabrice Fontaine 2021-11-19 23:32 ` Peter Seiderer @ 2021-11-22 20:42 ` Arnout Vandecappelle 2021-11-23 18:33 ` Peter Seiderer 1 sibling, 1 reply; 10+ messages in thread From: Arnout Vandecappelle @ 2021-11-22 20:42 UTC (permalink / raw) To: Fabrice Fontaine, buildroot; +Cc: Matt Weber On 18/11/2021 22:06, Fabrice Fontaine wrote: > For an unknown reason, the build fails since bump to version 1.4.60 in > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > for (uint32_t i = 0; i < sz; ++i) { > ^ > > Running autoreconf fixes the build. > > Fixes: > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> After reading this thread and the upstream bug report, I did a bunch more analysis and drilled it down to the real issue. See the bug report and the updated commit message: Since upstream commit bb1d35dc1767 [1] (which is part of the bump to version 1.4.60), configure.ac no longer checks for a specific C standard. It will therefore use the "default" standard. With autoconf 2.69 (which is what is used to generate the configure script in the tarball), the default standard is C89, which requires no option to be passed to the compiler. However, that means that older compilers, e.g. GCC 4.8, will indeed use C89 and not C99. This leads to the following error. array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode for (uint32_t i = 0; i < sz; ++i) { ^ In autoconf 2.71 (which is used by Buildroot), the default standard has changed: it is now C11 if available, C99 failing that, and finally fall back to C89. Since GCC 4.8 (and later) support C11, that's what will be selected, so C99 is allowed as well. Thus, running autoreconf fixes the build. The issue was reported upstream [2]. Fixes: - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 [1] https://redmine.lighttpd.net/projects/lighttpd/repository/14/revisions/bb1d35dc17671bc308e25651b4f5fa0c3008cc69 [2] https://redmine.lighttpd.net/issues/3116 Applied to master, thanks. Regards, Arnout > --- > package/lighttpd/lighttpd.mk | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > index 9c18a6038e..c39a5db20f 100644 > --- a/package/lighttpd/lighttpd.mk > +++ b/package/lighttpd/lighttpd.mk > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > LIGHTTPD_LICENSE_FILES = COPYING > LIGHTTPD_CPE_ID_VENDOR = lighttpd > LIGHTTPD_DEPENDENCIES = host-pkgconf > +# Issue in configure script provided in tarball > +LIGHTTPD_AUTORECONF = YES > LIGHTTPD_CONF_OPTS = \ > --without-wolfssl \ > --libdir=/usr/lib/lighttpd \ > _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
* Re: [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf 2021-11-22 20:42 ` Arnout Vandecappelle @ 2021-11-23 18:33 ` Peter Seiderer 0 siblings, 0 replies; 10+ messages in thread From: Peter Seiderer @ 2021-11-23 18:33 UTC (permalink / raw) To: Arnout Vandecappelle; +Cc: Matt Weber, Fabrice Fontaine, buildroot Hello Arnout, On Mon, 22 Nov 2021 21:42:39 +0100, Arnout Vandecappelle <arnout@mind.be> wrote: > On 18/11/2021 22:06, Fabrice Fontaine wrote: > > For an unknown reason, the build fails since bump to version 1.4.60 in > > commit d62b7f54f00291a50274409fb5510527fd725ffe on: > > > > array.c:140:2: error: 'for' loop initial declarations are only allowed in C99 mode > > for (uint32_t i = 0; i < sz; ++i) { > > ^ > > > > Running autoreconf fixes the build. > > > > Fixes: > > - http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > > > Signed-off-by: Fabrice Fontaine <fontaine.fabrice@gmail.com> > > After reading this thread and the upstream bug report, I did a bunch more > analysis and drilled it down to the real issue. See the bug report and the > updated commit message: > > Since upstream commit bb1d35dc1767 [1] (which is part of the bump to > version 1.4.60), configure.ac no longer checks for a specific C > standard. It will therefore use the "default" standard. With autoconf > 2.69 (which is what is used to generate the configure script in the > tarball), the default standard is C89, which requires no option to be > passed to the compiler. However, that means that older compilers, e.g. > GCC 4.8, will indeed use C89 and not C99. This leads to the following > error. > > array.c:140:2: error: 'for' loop initial declarations are only allowed in > C99 mode > for (uint32_t i = 0; i < sz; ++i) { > ^ > > In autoconf 2.71 (which is used by Buildroot), the default standard has > changed: it is now C11 if available, C99 failing that, and finally fall > back to C89. Since GCC 4.8 (and later) support C11, that's what will be > selected, so C99 is allowed as well. Thus, running autoreconf fixes the > build. > > The issue was reported upstream [2]. > > Fixes: > - > http://autobuild.buildroot.org/results/0349d8fed35b0766796dd9ba3b8de6ff8bd68fe7 > > [1] > https://redmine.lighttpd.net/projects/lighttpd/repository/14/revisions/bb1d35dc17671bc308e25651b4f5fa0c3008cc69 > [2] https://redmine.lighttpd.net/issues/3116 > > > Applied to master, thanks. Wow, thanks for further investigation and detailed/proper write up! Regards, Peter > > Regards, > Arnout > > > --- > > package/lighttpd/lighttpd.mk | 2 ++ > > 1 file changed, 2 insertions(+) > > > > diff --git a/package/lighttpd/lighttpd.mk b/package/lighttpd/lighttpd.mk > > index 9c18a6038e..c39a5db20f 100644 > > --- a/package/lighttpd/lighttpd.mk > > +++ b/package/lighttpd/lighttpd.mk > > @@ -12,6 +12,8 @@ LIGHTTPD_LICENSE = BSD-3-Clause > > LIGHTTPD_LICENSE_FILES = COPYING > > LIGHTTPD_CPE_ID_VENDOR = lighttpd > > LIGHTTPD_DEPENDENCIES = host-pkgconf > > +# Issue in configure script provided in tarball > > +LIGHTTPD_AUTORECONF = YES > > LIGHTTPD_CONF_OPTS = \ > > --without-wolfssl \ > > --libdir=/usr/lib/lighttpd \ > > > _______________________________________________ > buildroot mailing list > buildroot@buildroot.org > https://lists.buildroot.org/mailman/listinfo/buildroot _______________________________________________ buildroot mailing list buildroot@buildroot.org https://lists.buildroot.org/mailman/listinfo/buildroot ^ permalink raw reply [flat|nested] 10+ messages in thread
end of thread, other threads:[~2021-11-23 18:33 UTC | newest] Thread overview: 10+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- 2021-11-18 21:06 [Buildroot] [PATCH 1/1] package/lighttpd: run autoreconf Fabrice Fontaine 2021-11-19 23:32 ` Peter Seiderer 2021-11-20 9:23 ` Fabrice Fontaine 2021-11-20 19:53 ` Peter Seiderer 2021-11-20 20:15 ` Fabrice Fontaine 2021-11-20 20:23 ` Peter Seiderer 2021-11-20 20:39 ` Fabrice Fontaine 2021-11-21 21:32 ` Peter Seiderer 2021-11-22 20:42 ` Arnout Vandecappelle 2021-11-23 18:33 ` Peter Seiderer
This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox