[MPlayer-dev-eng] Re: [PATCH] GNU/kFreeBSD support

[Dominik 'Rathann' Mierzejewski]

On Monday, 22 August 2005 at 09:20, Robert Millan wrote:
> On Sun, Aug 21, 2005 at 10:12:35PM +0200, Dominik 'Rathann' Mierzejewski wrote:
> > > > > diff -ur MPlayer-1.0pre7.old/libmpdvdkit2/dvd_reader.c MPlayer-1.0pre7/libmpdvdkit2/dvd_reader.c
> > > > > --- MPlayer-1.0pre7.old/libmpdvdkit2/dvd_reader.c	2005-03-11 03:40:28.000000000 +0100
> > > > > +++ MPlayer-1.0pre7/libmpdvdkit2/dvd_reader.c	2005-08-05 18:44:36.000000000 +0200
> > > > [...]
> > > > > @@ -393,8 +393,8 @@
> > > > >  	    }
> > > > >  	    fclose( mntfile );
> > > > >  	}
> > > > > -#elif defined(__linux__)
> > > > > -        mntfile = fopen( MOUNTED, "r" );
> > > > > +#elif defined(__linux__) || defined(__GLIBC__)
> > > > > +        mntfile = fopen(_PATH_MOUNTED, "r" );
> > > > >          if( mntfile ) {
> > > > >              struct mntent *me;
> > > > >   
> > > > 
> > > > Why this change?
> > > 
> > > Because the code that follows (getmntent, etc) is Glibc-specific but not
> > > Linux-specific.  GNU/kFreeBSD can use that.
> > 
> > I meant changing MOUNTED to _PATH_MOUNTED.
> 
> According to glibc documentation, MOUNTED is a deprecated macro, only kept for
> backwards compatibility.

Deprecated in which version? I think we're still supporting libc5 systems,
so don't make any changes that won't work there.

R.

-- 
MPlayer RPMs maintainer: http://rpm.greysector.net/mplayer/
"I am Grey. I stand between the candle and the star. We are Grey.
 We stand between the darkness ... and the light."
        -- Delenn in Grey Council in Babylon 5:"Babylon Squared"