Re: [Libcdio-devel] libcdio 1.1.0rc1 is available for testing

[Rocky Bernstein]

Sorry  - commit f473388 should have the 2nd #if to #ifdef. To make applying
patches more reliable, I suggest using
https://savannah.gnu.org/patch/?group=libcdio

The rc1 tarball has been refreshed with the change as well.

On Wed, Dec 6, 2017 at 2:43 AM, Ozkan Sezer <address@hidden> wrote:

> On 12/6/17, Rocky Bernstein <address@hidden> wrote:
> > All of the combined recent changes are now packaged in
> >
> > http://rocky.github.io/libcdio-1.1.0rc1.tar.bz2
> > <http://rocky.github.io/libcdio-1.0.0rc1.tar.bz2> .
> >
> > Please test if you can.
> >
> >
> > Changes include:
> >
> > - Remove all remaining memory leads, invalid
> >   reads, writes (as per valgrind) in library,
> >   test and demo code
> > - Types CdioISO9660{Dir,File}List_t,  have been added
> >   and iso9660_{dir,file}list_{new,free} have been added.
> >
> > If I have missed something let me know.
> >
> > I will be testing this on various systems I have available,
> >
> > Thanks.
> >
>
> Still have this in dev  branch:
>
> iso9660.c: In function 'iso9660_get_ltime':
> iso9660.c:280:5: warning: "HAVE_STRUCT_TM_TM_ZONE" is not defined
>
> Cure:
>
> diff --git a/lib/iso9660/iso9660.c b/lib/iso9660/iso9660.c
> index 4def3c5..3f24f14 100644
> --- a/lib/iso9660/iso9660.c
> +++ b/lib/iso9660/iso9660.c
> @@ -277,7 +277,7 @@ iso9660_get_ltime (const iso9660_ltime_t *p_ldate,
>  #ifndef HAVE_TM_GMTOFF
>    p_tm->tm_sec += p_ldate->lt_gmtoff * (15 * 60);
>  #endif
> -#if HAVE_STRUCT_TM_TM_ZONE == 1
> +#ifdef HAVE_STRUCT_TM_TM_ZONE
>    /* Initialize everything */
>    p_tm->tm_zone = 0;
>  #endif
>
>