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

Diego Biurrun diego at biurrun.de
Tue Aug 23 11:22:17 CEST 2005


On Tue, Aug 23, 2005 at 09:39:44AM +0200, Robert Millan wrote:
> On Tue, Aug 23, 2005 at 01:10:47AM +0200, Diego Biurrun wrote:
> > > > > -#ifdef __linux__
> > > > > +#if defined(__linux__) || (defined(__GLIBC__) && (defined(__FreeBSD_kernel__) || defined(__NetBSD_kernel__)))
> > > > 
> > > > Are you sure about this NetBSD change?
> > > 
> > > I'm the person who made the Debian GNU/kNetBSD port (the one based on Glibc).
> > > Although currently stalled, this system follows closely the configuration of
> > > its sister GNU/kFreeBSD, which provides Linux-style procfs.
> > > 
> > > If you feel this is too sloppy, though, I'd remove the __NetBSD_kernel__ part.
> > 
> > It looks as though just __GLIBC__ should be enough.
> 
> The problem with that is that procfs is a kernel feature, and this code uses
> Linux-style features (/proc/mounts).  We cannot garantee that all Glibc systems
> have these features (and in fact, I think GNU/Hurd doesn't).

OK, I'm satisfied now.

Does anybody still have objections against this patch?  Otherwise I will
apply the last revision over the week.

Diego




More information about the MPlayer-dev-eng mailing list