All of lore.kernel.org
 help / color / mirror / Atom feed
From: "Eugen.Hristev--- via buildroot" <buildroot@buildroot.org>
To: <peter@korsgaard.com>, <fontaine.fabrice@gmail.com>
Cc: buildroot@buildroot.org
Subject: Re: [Buildroot] [PATCH 1/1] package/dhcp: use internal bind
Date: Tue, 12 Apr 2022 12:32:05 +0000	[thread overview]
Message-ID: <1fe3cf8d-3db1-23ea-8fc6-66b1e5ca043c@microchip.com> (raw)
In-Reply-To: <34fddc8f-8710-65f9-330d-81fefe64354a@microchip.com>

On 4/12/22 2:43 PM, Eugen Hristev - M18282 wrote:
> On 4/12/22 2:33 PM, Peter Korsgaard wrote:
>>>>>>> "Fabrice" == Fabrice Fontaine <fontaine.fabrice@gmail.com> writes:
>>
>> Hi,
>>
>>>> Hello Peter, Fabrice,
>>    >>
>>    >> While your patch solves the isc/boolean.h problem, now I get several
>>    >> other problems when building an arm926 platform with this package :
>>    >> (sam9x60ek)
>>    >>
>>    >> Could you help please? 2022.01.1 is broken for me atm.
>>
>>    > You should apply:
>>    > https://git.buildroot.net/buildroot/commit/?id=789a08dac5808401629d6fc586b7377f54ebb316
>>
>> That is already in 202.02.1:
>>
>> https://git.buildroot.net/buildroot/commit/?h=2022.02.x&id=0397064a9a147903e9e0c16b2506319ea4a246fb
>>
>>> as well as
>>    > https://patchwork.ozlabs.org/project/buildroot/patch/DU0P251MB07797CAAE80C985DC8976335E3E79@DU0P251MB0779.EURP251.PROD.OUTLOOK.COM/
>>
>> Is that an ack? ;)
> 
> I am testing that patch to see if it improves anything, but as you said
> Peter, the commit above is already in my tree.

It appears that with this patch, I am getting both errors :


 >>> dhcp 4.4.2-P1 Building
PATH="/home/eugen/buildroot-2022.02/buildroot/output/host/bin:/home/eugen/buildroot-2022.02/buildroot/output/host/sbin:/usr/local/sbin:/usr/local/bin:/usr/sbin:/usr/bin:/sbin:/bin:/usr/games:/usr/local/games:/snap/bin" 
  /usr/bin/make -j1  -C 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/
Making all in ./bind
Configuring BIND libraries for DHCP.
configure: WARNING: using cross tools not prefixed with host triplet
Building BIND libraries - this takes some time.
Building isc library in 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/bind-9.11.14/lib/isc
In file included from 
/home/eugen/buildroot-2022.02/buildroot/output/host/arm-buildroot-linux-gnueabi/sysroot/usr/include/bits/libc-header-start.h:33,
                  from 
/home/eugen/buildroot-2022.02/buildroot/output/host/arm-buildroot-linux-gnueabi/sysroot/usr/include/limits.h:26,
                  from 
/home/eugen/buildroot-2022.02/buildroot/output/host/lib/gcc/arm-buildroot-linux-gnueabi/10.3.0/include-fixed/limits.h:195,
                  from 
/home/eugen/buildroot-2022.02/buildroot/output/host/lib/gcc/arm-buildroot-linux-gnueabi/10.3.0/include-fixed/syslimits.h:7,
                  from 
/home/eugen/buildroot-2022.02/buildroot/output/host/lib/gcc/arm-buildroot-linux-gnueabi/10.3.0/include-fixed/limits.h:34,
                  from ./include/isc/platform.h:218,
                  from ./include/isc/bind9.h:18,
                  from ./include/isc/types.h:19,
                  from ./include/isc/safe.h:20,
                  from ./safe.c:18:
/home/eugen/buildroot-2022.02/buildroot/output/host/arm-buildroot-linux-gnueabi/sysroot/usr/include/features.h:412:4: 
warning: #warning _FORTIFY_SOURCE requires compiling with optimization 
(-O) [-Wcpp]
   412 | #  warning _FORTIFY_SOURCE requires compiling with optimization 
(-O)
       |    ^~~~~~~
stats.c: In function ‘setcounter’:
stats.c:300:29: error: ‘val’ undeclared (first use in this function); 
did you mean ‘value’?
   300 |  stats->counters[counter] = val;
       |                             ^~~
       |                             value
stats.c:300:29: note: each undeclared identifier is reported only once 
for each function it appears in
stats.c:286:20: warning: unused parameter ‘value’ [-Wunused-parameter]
   286 |     const uint64_t value)
       |     ~~~~~~~~~~~~~~~^~~~~
make[4]: *** [Makefile:290: stats.lo] Error 1
Building dns library in 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/bind-9.11.14/lib/dns
libtool:   error: cannot find the library '../../lib/isc/libisc.la' or 
unhandled argument '../../lib/isc/libisc.la'
make[4]: *** [Makefile:593: libdns.la] Error 1
Building isccfg library in 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/bind-9.11.14/lib/isccfg
libtool:   error: cannot find the library '../../lib/dns/libdns.la' or 
unhandled argument '../../lib/dns/libdns.la'
make[4]: *** [Makefile:482: libisccfg.la] Error 1
Building irs library in 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/bind-9.11.14/lib/irs
/usr/bin/ar: `u' modifier ignored since `D' is the default (see `U')
Installing BIND libraries to 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind.
stats.c: In function ‘setcounter’:
stats.c:300:29: error: ‘val’ undeclared (first use in this function); 
did you mean ‘value’?
   300 |  stats->counters[counter] = val;
       |                             ^~~
       |                             value
stats.c:300:29: note: each undeclared identifier is reported only once 
for each function it appears in
stats.c:286:20: warning: unused parameter ‘value’ [-Wunused-parameter]
   286 |     const uint64_t value)
       |     ~~~~~~~~~~~~~~~^~~~~
make[4]: *** [Makefile:290: stats.lo] Error 1
libtool:   error: cannot find the library '../../lib/isc/libisc.la' or 
unhandled argument '../../lib/isc/libisc.la'
make[4]: *** [Makefile:593: libdns.la] Error 1
libtool:   error: cannot find the library '../../lib/dns/libdns.la' or 
unhandled argument '../../lib/dns/libdns.la'
make[4]: *** [Makefile:482: libisccfg.la] Error 1
mkdir 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/include
mkdir 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/include/irs
mkdir 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/lib
Making all in includes
/usr/bin/make  all-am
Making all in tests
make[3]: Nothing to be done for 'all'.
Making all in common
Making all in .
/bin/bash ../libtool  --tag=CC   --mode=compile 
/home/eugen/buildroot-2022.02/buildroot/output/host/bin/arm-buildroot-linux-gnueabi-gcc 
-DHAVE_CONFIG_H -I. -I../includes  -I.. -DLOCALSTATEDIR='"/var"' 
-D_PATH_DHCPD_CONF=\"/etc/dhcp/dhcpd.conf\" 
-D_PATH_DHCLIENT_CONF=\"/etc/dhcp/dhclient.conf\"  -D_LARGEFILE_SOURCE 
-D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64  -Ofast -g0 
-D_FORTIFY_SOURCE=1 -DISC_CHECK_NONE=1  -I../includes 
-I/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/include 
-c -o alloc.lo alloc.c
libtool: compile: 
/home/eugen/buildroot-2022.02/buildroot/output/host/bin/arm-buildroot-linux-gnueabi-gcc 
-DHAVE_CONFIG_H -I. -I../includes -I.. -DLOCALSTATEDIR=\"/var\" 
-D_PATH_DHCPD_CONF=\"/etc/dhcp/dhcpd.conf\" 
-D_PATH_DHCLIENT_CONF=\"/etc/dhcp/dhclient.conf\" -D_LARGEFILE_SOURCE 
-D_LARGEFILE64_SOURCE -D_FILE_OFFSET_BITS=64 -Ofast -g0 
-D_FORTIFY_SOURCE=1 -DISC_CHECK_NONE=1 -I../includes 
-I/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/bind/include 
-c alloc.c  -fPIC -DPIC -o .libs/alloc.o
In file included from ../includes/dhcpd.h:91,
                  from alloc.c:29:
../includes/omapip/isclib.h:51:10: fatal error: isc/boolean.h: No such 
file or directory
    51 | #include <isc/boolean.h>
       |          ^~~~~~~~~~~~~~~
compilation terminated.
make[4]: *** [Makefile:561: alloc.lo] Error 1
make[3]: *** [Makefile:619: all-recursive] Error 1
make[2]: *** [Makefile:495: all-recursive] Error 1
make[1]: *** [package/pkg-generic.mk:292: 
/home/eugen/buildroot-2022.02/buildroot/output/build/dhcp-4.4.2-P1/.stamp_built] 
Error 2
make: *** [Makefile:84: _all] Error 2


>>
>>    > Having said that, I fear that dhcp will have to be removed soon due to
>>    > security concerns:
>>    > https://gitlab.isc.org/isc-projects/dhcp/-/issues/233
>>
>> Yeah. Eugen, is there any specific reason why you use the dhcp package
>> over the other DHCP clients we provide?
> 
> Historical reasons, and the fact that people have working configurations
> using this package... I was unaware that it was due to be removed.
> We will try to switch to a different package in the future. Thanks for
> letting me know.
> However, if the package is still in this buildroot release, it has to
> work right ?
>>
>> --
>> Bye, Peter Korsgaard
>>
> 

_______________________________________________
buildroot mailing list
buildroot@buildroot.org
https://lists.buildroot.org/mailman/listinfo/buildroot

  reply	other threads:[~2022-04-12 12:32 UTC|newest]

Thread overview: 32+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2022-04-03 10:03 [Buildroot] [PATCH 1/1] package/dhcp: use internal bind Fabrice Fontaine
2022-04-04  6:19 ` James Hilliard
2022-04-04 19:31 ` Arnout Vandecappelle
2022-04-04 19:55 ` Yann E. MORIN
2022-04-09 14:09 ` Peter Korsgaard
2022-04-12 11:15   ` Eugen.Hristev--- via buildroot
2022-04-12 11:22     ` Fabrice Fontaine
2022-04-12 11:33       ` Peter Korsgaard
2022-04-12 11:43         ` Eugen.Hristev--- via buildroot
2022-04-12 12:32           ` Eugen.Hristev--- via buildroot [this message]
2022-04-12 12:34           ` Peter Korsgaard
2022-04-12 18:17             ` Fabrice Fontaine
2022-04-13  7:54               ` Eugen.Hristev--- via buildroot
2022-04-13  8:29                 ` Peter Korsgaard
2022-04-14  9:10                   ` Eugen.Hristev--- via buildroot
2022-04-14  9:59                     ` Fabrice Fontaine
2022-04-14 16:44                       ` Fabrice Fontaine
2022-04-19  5:21                         ` Eugen.Hristev--- via buildroot
2022-04-19  9:08                           ` Fabrice Fontaine
2022-04-19 10:19                             ` Eugen.Hristev--- via buildroot
2022-04-19 16:12                               ` Fabrice Fontaine
2022-04-21  9:25                                 ` Eugen.Hristev--- via buildroot
2022-04-04 13:07 Edmundo Ferreira
2022-04-05  3:54 Tim Hammer
2022-04-05  6:19 ` Fabrice Fontaine
2022-04-05 12:15   ` Tim Hammer
2022-04-05 16:18     ` Peter Seiderer
2022-04-06  1:58       ` Tim Hammer
2022-04-06  7:08         ` Fabrice Fontaine
2022-04-06 12:58           ` Tim Hammer
2022-04-06 13:30             ` Fabrice Fontaine
2022-04-06 14:06               ` Fabrice Fontaine

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=1fe3cf8d-3db1-23ea-8fc6-66b1e5ca043c@microchip.com \
    --to=buildroot@buildroot.org \
    --cc=Eugen.Hristev@microchip.com \
    --cc=fontaine.fabrice@gmail.com \
    --cc=peter@korsgaard.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.