On Fri, Sep 23, 2022 at 04:22:47PM -0400, Kurt Mosiejczuk wrote:
> On Fri, Sep 23, 2022 at 10:15:51PM +0200, Antoine Jacoutot wrote:
> 
> > > > Can it be that it finds libzlib if installed?
> 
> > > > <...>
> > > > checking for zip_add in -lzip... yes
> > > > checking zip.h usability... yes   
> > > > checking zip.h presence... yes  
> > > > checking for zip.h... yes
> > > > <...>
> > > > => dpb junking
> > > > c++ -DHAVE_CONFIG_H -I. -DHAVE_MKSTEMP -DHAVE_LIBZIP -DDEFAULTGS=gs 
> > > > -DPSTOEDITDATADIR="/usr/local/share/pstoedit" -DPST
> > > > OEDITLIBDIR="/usr/local/lib/pstoedit" -Wall -Wextra -Wuninitialized 
> > > > -Wswitch-default -Wunused -Wshadow -Wwrite-strings
> > > > -Wcast-qual -Wpointer-arith -Wno-long-long -I/usr/local/include 
> > > > -D_LITTLE_ENDIAN -O2 -pipe -DINTERNALBOOL -MT drvpptx.l
> > > > o -MD -MP -MF .deps/drvpptx.Tpo -c drvpptx.cpp -fPIC -DPIC -o 
> > > > .libs/drvpptx.o
> > > > drvpptx.cpp:67:10: fatal error: 'zip.h' file not found
> > > > #include <zip.h>
> 
> > > > 1 error generated.
> 
> > > I haven't seen it, but I can believe it. Should we add archivers/libzip to
> > > LIB_DEPENDS?
> 
> > Can't we disable it with a knob?
> > Anyway, I have no strong opinion whether to enforce the dep or not.
> 
> There's no configure knob for doing without libzip. I tried adding 
> "--without-libzip" and it just complained about that option.

There is.
But you cannot just assume --disable-$randomlib works. 
Each configure script is different; in this case, you want: --without-pptx which
libzip is a dependency for.
In general, ./configure --help and/or looking at the configure script itself
will give you the needed information.

I just committed a fix.

-- 
Antoine

Reply via email to