> Date: Tue, 30 Nov 2021 23:04:49 +0100
> From: Patrick Wildt <patr...@blueri.se>
> 
> On Thu, Nov 11, 2021 at 08:27:21PM +0900, SASANO Takayoshi wrote:
> > Hi,
> > 
> > Recently I tried to crossbuild for arm64 on amd64.
> > I found "TARGET=arm64 make cross-tools" stops when building libfido2.
> > 
> > libfido2 requires libz but this is not built yet at that time.
> > lib/Makefile needs to tweak like this.
> > 
> > openbsd-current-vm# diff -uNpr Makefile~ Makefile
> > --- Makefile~   Sun Jan  3 05:04:36 2021
> > +++ Makefile    Thu Nov 11 19:47:42 2021
> > @@ -2,10 +2,10 @@
> >  #      $NetBSD: Makefile,v 1.20.4.1 1996/06/14 17:22:38 cgd Exp $
> > 
> >  SUBDIR=        csu libagentx libarch libc libcbor libcrypto libcurses \
> > -       libedit libelf libevent libexpat \
> > +       libedit libelf libevent libexpat libz \
> >         libfido2 libform libfuse libkeynote libkvm libl libm libmenu \
> >         libossaudio libpanel libpcap libradius librthread \
> >         librpcsvc libskey libsndio libssl libtls libusbhid \
> > -       libutil liby libz
> > +       libutil liby
> > 
> >  .include <bsd.subdir.mk>
> > openbsd-current-vm#
> > 
> > It works good, but I think there is better build order.
> > 
> > -- 
> > SASANO Takayoshi (JG1UAA) <u...@mx5.nisiq.net>
> > 
> 
> Hi,
> 
> I just had the same issue and after a short discussion it seems like
> it's a better idea to separate libfido2 into a separate line to show
> that it should be built at the end due to having libz as dependency.
> 
> Comments? ok?
> 
> Patrick
> 
> diff --git a/lib/Makefile b/lib/Makefile
> index 7f95cb1e3e7..1f1586b0518 100644
> --- a/lib/Makefile
> +++ b/lib/Makefile
> @@ -3,9 +3,10 @@
>  
>  SUBDIR=      csu libagentx libarch libc libcbor libcrypto libcurses \
>       libedit libelf libevent libexpat \
> -     libfido2 libform libfuse libkeynote libkvm libl libm libmenu \
> +     libform libfuse libkeynote libkvm libl libm libmenu \
>       libossaudio libpanel libpcap libradius librthread \
>       librpcsvc libskey libsndio libssl libtls libusbhid \
>       libutil liby libz

Maybe add a comment here that libfido2 depends on libz?

> +SUBDIR+=libfido2
>  
>  .include <bsd.subdir.mk>
> 
> 

Reply via email to