On 01/23/12 23:18, Mikolaj Kucharski wrote:
> On Mon, Jan 23, 2012 at 03:42:44PM +0000, Nigel Taylor wrote:
>> On 01/22/12 21:23, Mikolaj Kucharski wrote:
>>> Update to version 0.40 below. This unbreaks YouTube as they change the
>>> code recently.
>>>
>>> Please also read posts from my previous updates for more detailed
>>> information about changes between below (0.40) and in-tree (0.32) version:
>>>
>>> http://marc.info/?l=openbsd-ports&m=132683686806468&w=2
>>>
>> Hi,
>>
>> Term::ANSIColor is a core perl routine it should not be necessary to
>> create and install a new package. If that was the case then a specific
>> version requirement should be defined in the WWW::YouTube::Download
>> distribution.
>>
>> The RUN_DEPEND on devel/p5-Term-ANSIColor should be removed, and the
>> core perl version used.
>>
>> Only a quick test, but works without p5-Term-ANSIColor, if it's a red
>> colour for errors.
>>
>> Nigel
> 
> I don't know what I was smoking when I was preparing the update first
> time. Indeed Term::ANSIColor is in base and there is no need to port it.
> 
> 
> New version below.
> 
> 
> Index: Makefile
> ===================================================================
> RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
> retrieving revision 1.4
> diff -u -r1.4 Makefile
> --- Makefile  26 Aug 2011 09:03:13 -0000      1.4
> +++ Makefile  23 Jan 2012 22:45:56 -0000
> @@ -3,7 +3,7 @@
>  COMMENT =    simple YouTube interface for downloading videos
>  
>  MODULES =    cpan
> -DISTNAME =   WWW-YouTube-Download-0.32
> +DISTNAME =   WWW-YouTube-Download-0.40
>  CATEGORIES = www
>  
>  MAINTAINER = Mikolaj Kucharski <[email protected]>
> @@ -20,9 +20,10 @@
>  MODCPAN_EXAMPLES =   Yes
>  MODCPAN_EXAMPLES_DIST =      eg
>  
> -RUN_DEPENDS =        www/p5-libwww \
> +RUN_DEPENDS =        converters/p5-JSON \
> +             textproc/p5-XML-TreePP \
> +             www/p5-HTML-Parser \
>               www/p5-URI \
> -             converters/p5-JSON \
> -             www/p5-HTML-Parser
> +             www/p5-libwww
>  
>  .include <bsd.port.mk>
> Index: distinfo
> ===================================================================
> RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
> retrieving revision 1.4
> diff -u -r1.4 distinfo
> --- distinfo  26 Aug 2011 09:03:13 -0000      1.4
> +++ distinfo  23 Jan 2012 22:45:56 -0000
> @@ -1,5 +1,5 @@
> -MD5 (WWW-YouTube-Download-0.32.tar.gz) = l3usE74P8TNq8kcWXJBEjw==
> -RMD160 (WWW-YouTube-Download-0.32.tar.gz) = W8Xm5yO20hcElh6xa7toSUV12bI=
> -SHA1 (WWW-YouTube-Download-0.32.tar.gz) = 8UG1kxu3RRJu9JHUaosXxbFod8Q=
> -SHA256 (WWW-YouTube-Download-0.32.tar.gz) = 
> Adhf7sgwMcGTY/5PiOIahk611lKlen4LTrSXEIprxxU=
> -SIZE (WWW-YouTube-Download-0.32.tar.gz) = 35796
> +MD5 (WWW-YouTube-Download-0.40.tar.gz) = DttyVo2GhAhXhcqbfRCZGA==
> +RMD160 (WWW-YouTube-Download-0.40.tar.gz) = qAg0iNZeDzpKfgM+m1DslL0r4NA=
> +SHA1 (WWW-YouTube-Download-0.40.tar.gz) = Gz7TBWljITlmuYuu+vezUb2yh7c=
> +SHA256 (WWW-YouTube-Download-0.40.tar.gz) = 
> QugU9aOA22XOXrgt+DK49GJmcy57bS/qUYX6RuIl7Kg=
> +SIZE (WWW-YouTube-Download-0.40.tar.gz) = 34282
> Index: patches/patch-Makefile_PL
> ===================================================================
> RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/patches/patch-Makefile_PL,v
> retrieving revision 1.2
> diff -u -r1.2 patch-Makefile_PL
> --- patches/patch-Makefile_PL 31 Jul 2011 18:09:57 -0000      1.2
> +++ patches/patch-Makefile_PL 23 Jan 2012 22:45:56 -0000
> @@ -3,13 +3,13 @@
>  Module::Install::AuthorTests is missing in ports tree,
>  but not really essential.
>  
> ---- Makefile.PL.orig Tue Jan  4 15:17:14 2011
> -+++ Makefile.PL      Mon Jan 10 22:22:02 2011
> -@@ -8,7 +8,6 @@ requires 'JSON';
> - requires 'HTML::Entities';
> +--- Makefile.PL.orig Tue Jan 17 16:23:57 2012
> ++++ Makefile.PL      Sun Jan 22 20:58:39 2012
> +@@ -11,7 +11,6 @@ requires 'Term::ANSIColor'; # for youtube-download and
> + requires 'XML::TreePP'; # for youtube-playlists
>   
>   tests 't/*.t t/*/*.t';
>  -author_tests 'xt';
>   
> - test_requires 'Test::More', '0.96';
> - auto_include;
> + install_script 'bin/youtube-download', 'bin/youtube-playlists';
> + 
> Index: patches/patch-lib_WWW_YouTube_Download_pm
> ===================================================================
> RCS file: patches/patch-lib_WWW_YouTube_Download_pm
> diff -N patches/patch-lib_WWW_YouTube_Download_pm
> Index: pkg/PLIST
> ===================================================================
> RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/pkg/PLIST,v
> retrieving revision 1.1.1.1
> diff -u -r1.1.1.1 PLIST
> --- pkg/PLIST 25 Apr 2011 13:04:17 -0000      1.1.1.1
> +++ pkg/PLIST 23 Jan 2012 22:45:56 -0000
> @@ -1,6 +1,10 @@
>  @comment $OpenBSD: PLIST,v 1.1.1.1 2011/04/25 13:04:17 todd Exp $
> +bin/youtube-download
> +bin/youtube-playlists
>  ${P5SITE}/WWW/YouTube/
>  ${P5SITE}/WWW/YouTube/Download.pm
> +@man man/man1/youtube-download.1
> +@man man/man1/youtube-playlists.1
>  @man man/man3p/WWW::YouTube::Download.3p
>  share/examples/p5-WWW-YouTube-Download/
>  share/examples/p5-WWW-YouTube-Download/cb.pl
> 
> 


CVS updated with changes.

Nigel

Reply via email to