Re: [hatari-devel] configure does not check for zlib.h

[ Thread Index | Date Index | More lists.tuxfamily.org/hatari-devel Archives ]


On Wed, May 30, 2012 at 10:35 PM, Nicolas Pomarède
<npomarede@xxxxxxxxxxxx> wrote:
> On 30/05/2012 22:32, Christer Solskogen wrote:
>>
>> And I think it should, since without it compilation will fail.
>>
>>
>
> Hello
>
> are you sure ? In CMakelists.txt :
>
> find_package(ZLIB)
> if(ZLIB_FOUND)
>        set(HAVE_LIBZ 1)
>        set(HAVE_ZLIB_H 1)
> endif(ZLIB_FOUND)
>

Well, yes. It does check if it exist, you are right. But it will still
create a makefile.
I think configure should barf the same way it does when sdl.h is
missing. Since if zlib.h is missing you get this:

/home/solskogen/hatari/src/dim.c:11:18: fatal error: zlib.h: No such
file or directory
compilation terminated.
make[2]: *** [src/CMakeFiles/Floppy.dir/dim.c.o] Error 1
make[1]: *** [src/CMakeFiles/Floppy.dir/all] Error 2
make: *** [all] Error 2

-- 
chs,



Mail converted by MHonArc 2.6.19+ http://listengine.tuxfamily.org/