Thanks, it was an oversight - I'll put it in my tree (+ missing
REVISION bumps) for after unlock.


On 2010/08/11 18:12, Joachim Schipper wrote:
> p5-Tk has a somewhat questionable PLIST; I noticed this because
> p5-Tk-main gives an error when uninstalling ("directory not empty").
> 
> If there's no particular reason to split the files this way, how about
> the following diff? Otherwise, if there *is* a reason to split the files
> this way, how about moving e.g. ${P5ARCH}/Tk/demos into pkg/PLIST-main?
> 
>               Joachim
> 
> Index: pkg/PLIST-main
> ===================================================================
> RCS file: /usr/cvs/ports/ports/x11/p5-Tk/pkg/PLIST-main,v
> retrieving revision 1.2
> diff -u -p -r1.2 PLIST-main
> --- pkg/PLIST-main    2 Oct 2008 08:31:29 -0000       1.2
> +++ pkg/PLIST-main    11 Aug 2010 16:02:48 -0000
> @@ -213,8 +213,6 @@ ${P5ARCH}/Tk/cbxarrow.xbm
>  ${P5ARCH}/Tk/chooseColor.pod
>  ${P5ARCH}/Tk/chooseDirectory.pod
>  ${P5ARCH}/Tk/composite.pod
> -${P5ARCH}/Tk/demos/widtrib/all.pl
> -${P5ARCH}/Tk/demos/widtrib/cursor.pl
>  ${P5ARCH}/Tk/event.pod
>  ${P5ARCH}/Tk/exit.pod
>  ${P5ARCH}/Tk/file.xbm
> @@ -683,8 +681,6 @@ ${P5ARCH}/fix_4_os2.pl
>  @man man/man3p/Tk::chooseColor.3p
>  @man man/man3p/Tk::chooseDirectory.3p
>  @man man/man3p/Tk::composite.3p
> -...@man man/man3p/Tk::demos::widget_lib::slide.3p
> -...@man man/man3p/Tk::demos::widget_lib::trace2.3p
>  @man man/man3p/Tk::event.3p
>  @man man/man3p/Tk::exit.3p
>  @man man/man3p/Tk::fileevent.3p
> Index: pkg/PLIST-demo
> ===================================================================
> RCS file: /usr/cvs/ports/ports/x11/p5-Tk/pkg/PLIST-demo,v
> retrieving revision 1.7
> diff -u -p -r1.7 PLIST-demo
> --- pkg/PLIST-demo    13 Jan 2005 03:23:10 -0000      1.7
> +++ pkg/PLIST-demo    11 Aug 2010 16:02:48 -0000
> @@ -104,8 +104,12 @@ ${P5ARCH}/Tk/demos/widtrib/Tiler.pl
>  ${P5ARCH}/Tk/demos/widtrib/lib/
>  ${P5ARCH}/Tk/demos/widtrib/lib/npuz/
>  ${P5ARCH}/Tk/demos/widtrib/lib/npuz/Xcamel.npuz
> +${P5ARCH}/Tk/demos/widtrib/all.pl
> +${P5ARCH}/Tk/demos/widtrib/cursor.pl
>  ${P5ARCH}/Tk/demos/widtrib/npuz.pl
>  ${P5ARCH}/Tk/demos/widtrib/plop.pl
>  @man man/man1/widget.1
> +...@man man/man3p/Tk::demos::widget_lib::slide.3p
> +...@man man/man3p/Tk::demos::widget_lib::trace2.3p
>  @man man/man3p/Tk::Widget.3p
>  @man man/man3p/Tk::WidgetDemo.3p
> 

Reply via email to