Re: pkg/45527 VLC and insufficient PLIST

[Matthew Mondor <mm_lists%pulsar-zone.net@localhost>]

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

From: Matthew Mondor <mm_lists%pulsar-zone.net@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/45527 VLC and insufficient PLIST
Date: Mon, 14 Nov 2011 13:18:44 -0500

 >  On Mon, Nov 14, 2011 at 05:40:07PM +0000, John Marino wrote:
 >   >  --- PLIST.orig  2011-10-07 21:13:19.000000000 +0200
 >   >  +++ PLIST       2011-11-14 16:10:45.613994000 +0100
 >   >  @@ -101,10 +101,12 @@
 >   >   lib/vlc/plugins/access/libaccess_oss_plugin.la
 >   >   lib/vlc/plugins/access/libaccess_tcp_plugin.la
 >   >   lib/vlc/plugins/access/libaccess_udp_plugin.la
 >   >  +${PLIST.cdda}lib/vlc/plugins/access/libcdda_plugin.la
 >   >   lib/vlc/plugins/access/libdvdnav_plugin.la
 >   >   lib/vlc/plugins/access/libdvdread_plugin.la
 >   >   lib/vlc/plugins/access/libfilesystem_plugin.la
 >   >   lib/vlc/plugins/access/librtp_plugin.la
 >   >  +${PLIST.vcd}lib/vlc/plugins/access/libvcd_plugin.la
 >   >   ${PLIST.x11}lib/vlc/plugins/access/libxcb_screen_plugin.la
 >   >   lib/vlc/plugins/access/libzip_plugin.la
 >   >   lib/vlc/plugins/access_output/libaccess_output_dummy_plugin.la
 >   >  
 >   >  Any objection?
 >   >  The PLIST variables cdda and vcd are already defined in the makefile.
 >  
 >  Looks reasonable to me.
 >  
 >  It seems... odd that two PLIST vars are defined in the makefile and
 >  not currently used in the PLIST. Maybe someone messed it up when
 >  updating?
 
 When seeing this PR it made me remember of other vlc PLIST related
 issues that I filed as pkg/45209 (just as reference, as the issues were
 not with the same files).
 
 Thanks,