Re: pkg/43086: /multimedia/vlc doesn't compile on linux

["OBATA Akio" <obache%netbsd.org@localhost>]

The following reply was made to PR pkg/43086; it has been noted by GNATS.

From: "OBATA Akio" <obache%netbsd.org@localhost>
To: gnats-bugs%netbsd.org@localhost
Cc: 
Subject: Re: pkg/43086: /multimedia/vlc doesn't compile on linux
Date: Wed, 31 Mar 2010 15:35:41 +0900

 On Wed, 31 Mar 2010 15:05:00 +0900, <schaecsn%gmx.net@localhost> wrote:
 
 >> Description:
 > /multimedia/vlc doesn't compile on linux. This applies to pkgsrc 2009Q4 and 
 > current.
 >
 > Specifically: file modules/access/file.c does not compile.
 >
 > The original code in file.c checks via "#if defined(MNT_LOCAL)" if MNT_LOCAL 
 > is defined and uses it only then. The vlc patch patch-aa uses MNT_LOCAL 
 > without checking its existence.
 >I don't know where MNT_LOCAL is supposed to be defined. It's not defined on 
 >my linux box (slackware 13):
 >
 > "find -R MNT_LOCAL /usr/include" returns nothing and a similiar find in the 
 > vlc source code just finds this symbol in modules/access/file.c
 >
 >> How-To-Repeat:
 > #bmake
 >> Fix:
 
 As 1st part of patch-aa, statvfs.h is not included by linux,
 so how about change 2nd part from
 
 #ifdef HAVE_FSTATVFS
 
 to following?
 
 #if !defined(__linux__) && defined(HAVE_FSTATVFS)