Ethereal-dev: Re: [ethereal-dev] No ZLIB and latest CVS sources

Note: This archive is from the project's previous web site, ethereal.com. This list is no longer active.

From: Greg Kilfoyle <gregk@xxxxxxxxxxx>
Date: Wed, 26 Jan 2000 11:40:54 -0800
Guy Harris wrote:
> 
> > My configure results in HAVE_LIBZ being undefined. In this case there is
> > a conflict with the declaration of file_seek() between file_wrappers.c
> > and file_wrappers.h. One has a return type of 'int' and the other has
> > 'long'.
> >
> > I just changed the header to match the .c definition and recompiled
> > (successfully).
> 
> I'm checking in a change to
> 
>         1) declare and define "file_seek()" to return a "long" if libz
>            is present;
> 
>         2) define it only as a macro that expands to "ftell" - along the
>            lines of what Itojun did for "file_tell()" - if libz isn't
>            present.

I haven't seen the change in CVS yet and there was no patch attached :(

> 
> I've attached the patch that does that; it seems to work if I explicitly
> disable zlib support here, but you might want to check whether it works
> - if it doesn't, don't reply to "gharris@xxxxxxxxxxxx", reply either to
> the list or to "guy@xxxxxxxxxxxx", as, by the time the mail arrives, I
> may be at work, and the latter address goes both to work and home but
> the former one goes only to home.

--
Greg Kilfoyle (gregk@xxxxxxxxxxx)