UPDATE: www/p5-WWW-YouTube-Download to 0.50

2013-04-27 Thread Mikolaj Kucharski
Patch integrated upstream. New patch to support one more url type.
Will send this upstream.


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.9
diff -u -r1.9 Makefile
--- Makefile8 Apr 2013 19:24:19 -   1.9
+++ Makefile27 Apr 2013 12:16:44 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.47
+DISTNAME = WWW-YouTube-Download-0.50
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.7
diff -u -r1.7 distinfo
--- distinfo8 Apr 2013 19:24:19 -   1.7
+++ distinfo27 Apr 2013 12:16:44 -
@@ -1,2 +1,2 @@
-SHA256 (WWW-YouTube-Download-0.47.tar.gz) = 
ggyBBFUnCqV3QJ96XqtISjlZUf22PBupQm1BQMwJcZs=
-SIZE (WWW-YouTube-Download-0.47.tar.gz) = 23485
+SHA256 (WWW-YouTube-Download-0.50.tar.gz) = 
n1w2E5c7wSdkGv7eroQcctZCPpXTzS8F9g0wqqYu9yE=
+SIZE (WWW-YouTube-Download-0.50.tar.gz) = 19826
Index: patches/patch-bin_youtube-playlists
===
RCS file: patches/patch-bin_youtube-playlists
diff -N patches/patch-bin_youtube-playlists
--- patches/patch-bin_youtube-playlists 8 Apr 2013 19:24:19 -   1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,17 +0,0 @@
-$OpenBSD: patch-bin_youtube-playlists,v 1.1 2013/04/08 19:24:19 sthen Exp $
-
-- fix for playlists with only one item (reported in RT 81068, but not really 
fixed)
-
 bin/youtube-playlists.orig Mon Apr  1 18:07:46 2013
-+++ bin/youtube-playlists  Mon Apr  8 02:06:29 2013
-@@ -57,7 +57,9 @@ sub find_video_urls {
- my $urls = [];
- chatty('Parsing XML ... ');
- my $tree = $tpp->parse($xml);
--for my $entry (@{ $tree->{feed}{entry} || [] }) {
-+my $entries = ref($tree->{feed}{entry}) eq 'ARRAY' ?
-+  $tree->{feed}{entry} : [ $tree->{feed}{entry} ];
-+for my $entry (@$entries) {
- my $uri = URI->new($entry->{'media:group'}{'media:player'}{-url});
- $uri->query_param_delete('feature');
- push @$urls, $uri->as_string;
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
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-lib_WWW_YouTube_Download_pm   27 Apr 2013 12:16:44 -
@@ -0,0 +1,15 @@
+$OpenBSD$
+
+- support one more url type
+
+--- lib/WWW/YouTube/Download.pm.orig   Fri Apr 19 18:50:43 2013
 lib/WWW/YouTube/Download.pmSat Apr 27 12:33:16 2013
+@@ -271,7 +271,7 @@ sub _suffix {
+ 
+ sub _video_id {
+ my $stuff = shift;
+-if ($stuff =~ m{/.*?[?&;!]v=([^&#?=/;]+)}) {
++if ($stuff =~ m{/.*?[?&;!](?:v|video_id)=([^&#?=/;]+)}) {
+ return $1;
+ }
+ elsif ($stuff =~ m{/(?:e|v|embed)/([^&#?=/;]+)}) {


-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download to 0.47

2013-04-07 Thread Mikolaj Kucharski
On Mon, Apr 08, 2013 at 01:22:52AM +0100, Mikolaj Kucharski wrote:
> Hi,
> 
> Unbreaks after YouTube changes from last month, which I've missed. Works
> for me. Comparing to 0.41 this version brings:
> 
> - fixes the tool after site changes
> - switch from modinst to modbuild
> - various grammar/typo fixes
> - proxy support
> - new {user} substitution variable
> - new code example
> 
> Changes from 0.41:
> http://cpansearch.perl.org/src/XAICRON/WWW-YouTube-Download-0.47/Changes

Below diff adds real fix for RT 81068 (one item in the playlist).
Reported upstream.


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.8
diff -u -r1.8 Makefile
--- Makefile11 Mar 2013 11:44:47 -  1.8
+++ Makefile8 Apr 2013 01:08:59 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.41
+DISTNAME = WWW-YouTube-Download-0.47
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
@@ -12,7 +12,7 @@
 PERMIT_PACKAGE_CDROM = Yes
 
 CPAN_AUTHOR =  XAICRON
-CONFIGURE_STYLE =  modinst
+CONFIGURE_STYLE =  modbuild
 
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.6
diff -u -r1.6 distinfo
--- distinfo5 Nov 2012 10:59:43 -   1.6
+++ distinfo8 Apr 2013 01:08:59 -
@@ -1,2 +1,2 @@
-SHA256 (WWW-YouTube-Download-0.41.tar.gz) = 
cheRCs/ccYhSvjzrrX+CByCyCJ8K7Nyz1TbST7YpeFY=
-SIZE (WWW-YouTube-Download-0.41.tar.gz) = 35480
+SHA256 (WWW-YouTube-Download-0.47.tar.gz) = 
ggyBBFUnCqV3QJ96XqtISjlZUf22PBupQm1BQMwJcZs=
+SIZE (WWW-YouTube-Download-0.47.tar.gz) = 23485
Index: patches/patch-Makefile_PL
===
RCS file: patches/patch-Makefile_PL
diff -N patches/patch-Makefile_PL
--- patches/patch-Makefile_PL   7 Apr 2012 20:54:07 -   1.4
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,22 +0,0 @@
-$OpenBSD: patch-Makefile_PL,v 1.4 2012/04/07 20:54:07 sthen Exp $
-
-Module::Install::AuthorTests is missing in ports tree,
-but not really essential.
-
-Same for Module::Install::Repository.
-
-
 Makefile.PL.orig   Tue Jan 17 16:23:57 2012
-+++ Makefile.PLSat Apr  7 21:53:04 2012
-@@ -11,10 +11,9 @@ requires 'Term::ANSIColor'; # for youtube-download and
- requires 'XML::TreePP'; # for youtube-playlists
- 
- tests 't/*.t t/*/*.t';
--author_tests 'xt';
- 
- install_script 'bin/youtube-download', 'bin/youtube-playlists';
- 
- test_requires 'Test::More', '0.96';
--auto_set_repository;
-+#auto_set_repository;
- WriteAll;
Index: patches/patch-bin_youtube-playlists
===
RCS file: patches/patch-bin_youtube-playlists
diff -N patches/patch-bin_youtube-playlists
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-bin_youtube-playlists 8 Apr 2013 01:08:59 -
@@ -0,0 +1,17 @@
+$OpenBSD$
+
+- fix for playlists with only one item (reported in RT 81068, but not really 
fixed)
+
+--- bin/youtube-playlists.orig Mon Apr  1 18:07:46 2013
 bin/youtube-playlists  Mon Apr  8 02:06:29 2013
+@@ -57,7 +57,9 @@ sub find_video_urls {
+ my $urls = [];
+ chatty('Parsing XML ... ');
+ my $tree = $tpp->parse($xml);
+-for my $entry (@{ $tree->{feed}{entry} || [] }) {
++my $entries = ref($tree->{feed}{entry}) eq 'ARRAY' ?
++  $tree->{feed}{entry} : [ $tree->{feed}{entry} ];
++for my $entry (@$entries) {
+ my $uri = URI->new($entry->{'media:group'}{'media:player'}{-url});
+ $uri->query_param_delete('feature');
+ push @$urls, $uri->as_string;
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/pkg/PLIST,v
retrieving revision 1.2
diff -u -r1.2 PLIST
--- pkg/PLIST   24 Jan 2012 00:44:47 -  1.2
+++ pkg/PLIST   8 Apr 2013 01:08:59 -
@@ -8,4 +8,5 @@
 @man man/man3p/WWW::YouTube::Download.3p
 share/examples/p5-WWW-YouTube-Download/
 share/examples/p5-WWW-YouTube-Download/cb.pl
+share/examples/p5-WWW-YouTube-Download/menu.pl
 share/examples/p5-WWW-YouTube-Download/parallel.pl


-- 
best regards
q#



UPDATE: www/p5-WWW-YouTube-Download to 0.47

2013-04-07 Thread Mikolaj Kucharski
Hi,

Unbreaks after YouTube changes from last month, which I've missed. Works
for me. Comparing to 0.41 this version brings:

- fixes the tool after site changes
- switch from modinst to modbuild
- various grammar/typo fixes
- proxy support
- new {user} substitution variable
- new code example

Changes from 0.41:
http://cpansearch.perl.org/src/XAICRON/WWW-YouTube-Download-0.47/Changes


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.8
diff -u -r1.8 Makefile
--- Makefile11 Mar 2013 11:44:47 -  1.8
+++ Makefile8 Apr 2013 00:01:23 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.41
+DISTNAME = WWW-YouTube-Download-0.47
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
@@ -12,7 +12,7 @@
 PERMIT_PACKAGE_CDROM = Yes
 
 CPAN_AUTHOR =  XAICRON
-CONFIGURE_STYLE =  modinst
+CONFIGURE_STYLE =  modbuild
 
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.6
diff -u -r1.6 distinfo
--- distinfo5 Nov 2012 10:59:43 -   1.6
+++ distinfo8 Apr 2013 00:01:23 -
@@ -1,2 +1,2 @@
-SHA256 (WWW-YouTube-Download-0.41.tar.gz) = 
cheRCs/ccYhSvjzrrX+CByCyCJ8K7Nyz1TbST7YpeFY=
-SIZE (WWW-YouTube-Download-0.41.tar.gz) = 35480
+SHA256 (WWW-YouTube-Download-0.47.tar.gz) = 
ggyBBFUnCqV3QJ96XqtISjlZUf22PBupQm1BQMwJcZs=
+SIZE (WWW-YouTube-Download-0.47.tar.gz) = 23485
Index: patches/patch-Makefile_PL
===
RCS file: patches/patch-Makefile_PL
diff -N patches/patch-Makefile_PL
--- patches/patch-Makefile_PL   7 Apr 2012 20:54:07 -   1.4
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,22 +0,0 @@
-$OpenBSD: patch-Makefile_PL,v 1.4 2012/04/07 20:54:07 sthen Exp $
-
-Module::Install::AuthorTests is missing in ports tree,
-but not really essential.
-
-Same for Module::Install::Repository.
-
-
 Makefile.PL.orig   Tue Jan 17 16:23:57 2012
-+++ Makefile.PLSat Apr  7 21:53:04 2012
-@@ -11,10 +11,9 @@ requires 'Term::ANSIColor'; # for youtube-download and
- requires 'XML::TreePP'; # for youtube-playlists
- 
- tests 't/*.t t/*/*.t';
--author_tests 'xt';
- 
- install_script 'bin/youtube-download', 'bin/youtube-playlists';
- 
- test_requires 'Test::More', '0.96';
--auto_set_repository;
-+#auto_set_repository;
- WriteAll;
Index: pkg/PLIST
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/pkg/PLIST,v
retrieving revision 1.2
diff -u -r1.2 PLIST
--- pkg/PLIST   24 Jan 2012 00:44:47 -  1.2
+++ pkg/PLIST   8 Apr 2013 00:01:23 -
@@ -8,4 +8,5 @@
 @man man/man3p/WWW::YouTube::Download.3p
 share/examples/p5-WWW-YouTube-Download/
 share/examples/p5-WWW-YouTube-Download/cb.pl
+share/examples/p5-WWW-YouTube-Download/menu.pl
 share/examples/p5-WWW-YouTube-Download/parallel.pl



-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download to 0.41

2012-11-04 Thread Mikolaj Kucharski
Ok?

http://marc.info/?l=openbsd-ports&m=135033513526782&w=2

On Mon, Oct 15, 2012 at 10:01:49PM +0100, Mikolaj Kucharski wrote:
> Hi,
> 
> Maintenance update to latest release. Changes from last version:
> 
> - add playback_url function which returns direct url to the video
> - new option to only report direct URL of the video file (-U)
> - make colored output optional (-C)
> - wrong url in example to the playlist in the doc
> - fix downloading after youtube site changes (we can remove our diff)
> - fix for typo in documentation
> - don't leave failed empty files behind (RT 79931)
> 
> 
> This version still doesn't have a way to list of supported formats of
> a video. Please use patch -E to remove empty files.
> 
> Comments, okays?
> 

-- 
best regards
q#



UPDATE: www/p5-WWW-YouTube-Download to 0.41

2012-10-15 Thread Mikolaj Kucharski
Hi,

Maintenance update to latest release. Changes from last version:

- add playback_url function which returns direct url to the video
- new option to only report direct URL of the video file (-U)
- make colored output optional (-C)
- wrong url in example to the playlist in the doc
- fix downloading after youtube site changes (we can remove our diff)
- fix for typo in documentation
- don't leave failed empty files behind (RT 79931)


This version still doesn't have a way to list of supported formats of
a video. Please use patch -E to remove empty files.

Comments, okays?


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.6
diff -u -r1.6 Makefile
--- Makefile28 Sep 2012 00:20:16 -  1.6
+++ Makefile15 Oct 2012 20:45:54 -
@@ -3,8 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.40
-REVISION = 0
+DISTNAME = WWW-YouTube-Download-0.41
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.5
diff -u -r1.5 distinfo
--- distinfo24 Jan 2012 00:44:47 -  1.5
+++ distinfo15 Oct 2012 20:45:54 -
@@ -1,5 +1,2 @@
-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
+SHA256 (WWW-YouTube-Download-0.41.tar.gz) = 
cheRCs/ccYhSvjzrrX+CByCyCJ8K7Nyz1TbST7YpeFY=
+SIZE (WWW-YouTube-Download-0.41.tar.gz) = 35480
Index: patches/patch-bin_youtube-download
===
RCS file: patches/patch-bin_youtube-download
diff -N patches/patch-bin_youtube-download
--- patches/patch-bin_youtube-download  28 Sep 2012 00:20:16 -  1.1
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,15 +0,0 @@
-$OpenBSD: patch-bin_youtube-download,v 1.1 2012/09/28 00:20:16 sthen Exp $
-
-- fix for proper suffix support in downloaded filename
-
 bin/youtube-download.orig  Tue Jan 17 16:22:02 2012
-+++ bin/youtube-download   Thu Sep 27 22:42:51 2012
-@@ -47,7 +47,7 @@ main: {
- my $filename = $client->_format_filename($output, {
- video_id => $meta_data->{video_id},
- title=> decode_utf8($meta_data->{title}),
--suffix   => $meta_data->{suffix},
-+suffix   => $fmt ? $meta_data->{video_url_map}{$fmt}{suffix} : 
$meta_data->{suffix},
- });
- $filename = filename_normalize($filename);
- $filename = $encoder->encode($filename, sub { sprintf 'U+%x', shift 
});
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
--- patches/patch-lib_WWW_YouTube_Download_pm   28 Sep 2012 00:20:16 -  
1.3
+++ /dev/null   1 Jan 1970 00:00:00 -
@@ -1,17 +0,0 @@
-$OpenBSD: patch-lib_WWW_YouTube_Download_pm,v 1.3 2012/09/28 00:20:16 sthen 
Exp $
-
-- fix downloading after youtube changes
-
 lib/WWW/YouTube/Download.pm.orig   Sun Jan 22 07:54:47 2012
-+++ lib/WWW/YouTube/Download.pmThu Sep 27 22:45:08 2012
-@@ -222,7 +222,9 @@ sub _parse_stream_map {
- my $uri = URI->new;
- $uri->query($stuff);
- my $query = +{ $uri->query_form };
--$fmt_url_map->{$query->{itag}} = $query->{url};
-+my $sig = $query->{sig};
-+my $url = $query->{url};
-+$fmt_url_map->{$query->{itag}} = $url.'&signature='.$sig;
- }
- 
- return $fmt_url_map;


-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download

2012-01-23 Thread Nigel Taylor
On 01/23/12 23:18, Mikolaj Kucharski wrote:
> On Mon, Jan 23, 2012 at 03:42:44PM +, 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 -  1.4
> +++ Makefile  23 Jan 2012 22:45:56 -
> @@ -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 
> @@ -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 
> 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 -  1.4
> +++ distinfo  23 Jan 2012 22:45:56 -
> @@ -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 -  1.2
> +++ patches/patch-Makefile_PL 23 Jan 2012 22:45:56 -
> @@ -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 -  1.1.1.1
> +++ pkg/PLIST 23 Jan 2012 22:45:56 -
> @@ -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

Re: UPDATE: www/p5-WWW-YouTube-Download

2012-01-23 Thread Mikolaj Kucharski
On Mon, Jan 23, 2012 at 03:42:44PM +, 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
--- Makefile26 Aug 2011 09:03:13 -  1.4
+++ Makefile23 Jan 2012 22:45:56 -
@@ -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 
@@ -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 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- distinfo26 Aug 2011 09:03:13 -  1.4
+++ distinfo23 Jan 2012 22:45:56 -
@@ -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 -  1.2
+++ patches/patch-Makefile_PL   23 Jan 2012 22:45:56 -
@@ -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.PLMon 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.PLSun 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 -  1.1.1.1
+++ pkg/PLIST   23 Jan 2012 22:45:56 -
@@ -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


-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download

2012-01-23 Thread Nigel Taylor
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



UPDATE: www/p5-WWW-YouTube-Download

2012-01-22 Thread Mikolaj Kucharski
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



Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- Makefile26 Aug 2011 09:03:13 -  1.4
+++ Makefile22 Jan 2012 21:16:35 -
@@ -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 
@@ -20,9 +20,11 @@
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg
 
-RUN_DEPENDS =  www/p5-libwww \
-   www/p5-URI \
+RUN_DEPENDS =  devel/p5-Term-ANSIColor \
converters/p5-JSON \
-   www/p5-HTML-Parser
+   textproc/p5-XML-TreePP \
+   www/p5-HTML-Parser \
+   www/p5-URI \
+   www/p5-libwww
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- distinfo26 Aug 2011 09:03:13 -  1.4
+++ distinfo22 Jan 2012 21:16:35 -
@@ -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 -  1.2
+++ patches/patch-Makefile_PL   22 Jan 2012 21:16:35 -
@@ -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.PLMon 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.PLSun 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 -  1.1.1.1
+++ pkg/PLIST   22 Jan 2012 21:16:35 -
@@ -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


-- 
best regards
q#



UPDATE: www/p5-WWW-YouTube-Download to 0.39

2012-01-17 Thread Mikolaj Kucharski
Hi,

Author dropped `.pl' extension from the filenames, as requested.

As I wrote in previous verion of the update:

The app has new command line tools: youtube-download and
youtube-playlists. First one lets you download youtube videos from
command line (in similar way line many other tools like that). Second
script lets you list URLs to the videos from YouTube playlist url. That
lets you download few viedos at once, for example:

% youtube-playlists PLC9D0C04BB10E1797 | youtube-download

As requested by esipe@ I will be working with upstream to make
Term::ANSIColor optional.

Using this for few weeks now. It depends on new ports which I sent in
last hour to the list. Comments, okays?


It needs Term::ANSIColor, sent here:
http://marc.info/?l=openbsd-ports&m=132604872904102&w=2


and also it depends on XML::TreePP available from here:
http://marc.info/?l=openbsd-ports&m=132604979504307&w=2

ChangeLog for WWW::YouTube::Download:
http://cpansearch.perl.org/src/XAICRON/WWW-YouTube-Download-0.39/Changes


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- Makefile26 Aug 2011 09:03:13 -  1.4
+++ Makefile17 Jan 2012 21:30:26 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.32
+DISTNAME = WWW-YouTube-Download-0.39
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
@@ -20,9 +20,11 @@
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg
 
-RUN_DEPENDS =  www/p5-libwww \
-   www/p5-URI \
+RUN_DEPENDS =  devel/p5-Term-ANSIColor \
converters/p5-JSON \
-   www/p5-HTML-Parser
+   textproc/p5-XML-TreePP \
+   www/p5-HTML-Parser \
+   www/p5-URI \
+   www/p5-libwww
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- distinfo26 Aug 2011 09:03:13 -  1.4
+++ distinfo17 Jan 2012 21:30:26 -
@@ -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.39.tar.gz) = vcDvoqYYtj71ZfC8xee1Sw==
+RMD160 (WWW-YouTube-Download-0.39.tar.gz) = tD/gEZ6YHnwQwDxBFGtlBcZ/9Vk=
+SHA1 (WWW-YouTube-Download-0.39.tar.gz) = dOtMnmZsKl1S5H6cHAMbuzKVw+A=
+SHA256 (WWW-YouTube-Download-0.39.tar.gz) = 
eCKR9RWuHq0gZRhTFJT6Ioagl3aBfjNsgIhrYt88dMU=
+SIZE (WWW-YouTube-Download-0.39.tar.gz) = 34235
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 -  1.2
+++ patches/patch-Makefile_PL   17 Jan 2012 21:30:26 -
@@ -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.PLMon 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.PLTue Jan 17 21:26:29 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';
+
+ install_script 'bin/youtube-download', 'bin/youtube-playlists';
  
- test_requires 'Test::More', '0.96';
- auto_include;
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 -  1.1.1.1
+++ pkg/PLIST   17 Jan 2012 21:30:26 -
@@ -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


-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download to 0.38

2012-01-13 Thread Mikolaj Kucharski
I will be working with upstream to get issues reported by espie@ fixed.
Already sent an email to author, but didn't get any reply yet. I know we
are approaching the lock, so I would appreciate an okay for version
under below URL:

http://marc.info/?l=openbsd-ports&m=132606419107798&w=2


On Sun, Jan 08, 2012 at 10:34:34PM +, Mikolaj Kucharski wrote:
> On Sun, Jan 08, 2012 at 08:40:12PM +0100, Marc Espie wrote:
> > Can you at least provide links to the executable with sensible names ?
> > or heck, rename them and the corresponding manpage.
> > 
> > Outside of Windows, it doesn't make any sense to have a .pl extension,
> > the scripts should be called 'youtube-playlists' and 'youtube-download'.
> 
> Sure, I'll talk with author if he can drop .pl extensions. Also will
> work with him to make Term::ANSIColor optional in further releases, but
> for now is below inlined patch okay?
> 
> 
> Changes from my previous (todays) patch:
> 
> - droped .pl extension from the scripts
> - typo and style fix
> - missing req in Makefile.PL

-- 
best regards
q#



Re: UPDATE: www/p5-WWW-YouTube-Download to 0.38

2012-01-08 Thread Mikolaj Kucharski
On Sun, Jan 08, 2012 at 08:40:12PM +0100, Marc Espie wrote:
> Can you at least provide links to the executable with sensible names ?
> or heck, rename them and the corresponding manpage.
> 
> Outside of Windows, it doesn't make any sense to have a .pl extension,
> the scripts should be called 'youtube-playlists' and 'youtube-download'.

Sure, I'll talk with author if he can drop .pl extensions. Also will
work with him to make Term::ANSIColor optional in further releases, but
for now is below inlined patch okay?


Changes from my previous (todays) patch:

- droped .pl extension from the scripts
- typo and style fix
- missing req in Makefile.PL


Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- Makefile26 Aug 2011 09:03:13 -  1.4
+++ Makefile8 Jan 2012 21:10:40 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.32
+DISTNAME = WWW-YouTube-Download-0.38
 CATEGORIES =   www

 MAINTAINER =   Mikolaj Kucharski 
@@ -20,9 +20,18 @@
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg

-RUN_DEPENDS =  www/p5-libwww \
-   www/p5-URI \
+RUN_DEPENDS =  devel/p5-Term-ANSIColor \
converters/p5-JSON \
-   www/p5-HTML-Parser
+   textproc/p5-XML-TreePP \
+   www/p5-HTML-Parser \
+   www/p5-URI \
+   www/p5-libwww
+
+post-install:
+   ${MODCPAN_POST_INSTALL}
+   mv -f ${PREFIX}/bin/youtube-download{.pl,}
+   mv -f ${PREFIX}/bin/youtube-playlists{.pl,}
+   mv -f ${PREFIX}/man/man1/youtube-download{.pl,}.1
+   mv -f ${PREFIX}/man/man1/youtube-playlists{.pl,}.1
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- distinfo26 Aug 2011 09:03:13 -  1.4
+++ distinfo8 Jan 2012 21:10:40 -
@@ -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.38.tar.gz) = 1pk9bEwpc6zwQMrcyIfR3w==
+RMD160 (WWW-YouTube-Download-0.38.tar.gz) = skvDA0K/4EsHW2t0EQpbuG41IsU=
+SHA1 (WWW-YouTube-Download-0.38.tar.gz) = 9GE/kZ13ScuoIdCNIe38rK0CThE=
+SHA256 (WWW-YouTube-Download-0.38.tar.gz) = 
MN0ZSYcgRXIxAMZW23ZM7AP8Axfi6kNebw/yKw1R7UY=
+SIZE (WWW-YouTube-Download-0.38.tar.gz) = 34185
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 -  1.2
+++ patches/patch-Makefile_PL   8 Jan 2012 21:10:40 -
@@ -3,13 +3,17 @@
 Module::Install::AuthorTests is missing in ports tree,
 but not really essential.
 
 Makefile.PL.orig   Tue Jan  4 15:17:14 2011
-+++ Makefile.PLMon Jan 10 22:22:02 2011
-@@ -8,7 +8,6 @@ requires 'JSON';
+--- Makefile.PL.orig   Thu Oct  6 21:40:56 2011
 Makefile.PLSun Jan  8 19:58:58 2012
+@@ -7,10 +7,10 @@ requires 'URI::QueryParam';
+ requires 'LWP::UserAgent';
+ requires 'JSON';
  requires 'HTML::Entities';
++requires 'Term::ANSIColor'; # for youtube-download and youtube-playlists
+ requires 'XML::TreePP'; # for youtube-playlists.pl
  
  tests 't/*.t t/*/*.t';
 -author_tests 'xt';
  
- test_requires 'Test::More', '0.96';
- auto_include;
+ install_script 'bin/youtube-download.pl', 'bin/youtube-playlists.pl';
+ 
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
--- /dev/null   1 Jan 1970 00:00:00 -
+++ patches/patch-lib_WWW_YouTube_Download_pm   8 Jan 2012 21:10:40 -
@@ -0,0 +1,15 @@
+$OpenBSD$
+
+- typo and remove period at the end of the line
+
+--- lib/WWW/YouTube/Download.pm.orig   Wed Dec  7 15:44:45 2011
 lib/WWW/YouTube/Download.pmSun Jan  8 20:03:59 2012
+@@ -268,7 +268,7 @@ __END__
+ 
+ =head1 NAME
+ 
+-WWW::YouTube::Download - Very simply YouTube video download interface.
++WWW::YouTube::Download - Very simple YouTube video download interface
+ 
+ =head1 SYNOPSIS
+ 
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 -

Re: UPDATE: www/p5-WWW-YouTube-Download to 0.38

2012-01-08 Thread Marc Espie
On Sun, Jan 08, 2012 at 07:04:05PM +, Mikolaj Kucharski wrote:
> Hi,
> 
> The app has new command line tools: youtube-download.pl and
> youtube-playlists.pl. First one lets you download youtube videos from
> command line (in similar way line many other tools like that). Second
> script lets you list URLs to the videos from YouTube playlist url. That
> lets you download few viedos at once, for example:
> 
> % youtube-playlists.pl PLC9D0C04BB10E1797 | youtube-download.pl
> 
> 
> Using this for few weeks now. It depends on new ports which I sent in
> last hour to the list. Comments, okays?
> 
> 
> Changes between in-tree 0.32 to attached 0.38:
> 
> 0.38Thu Dec  8 00:44:52 2011
> - fix typo (mikolaj++)
> - remove wrong code (mikolaj++)
> 0.37Wed Dec  7 02:33:23 2011
> - fix wrong characters in filename in youtube-download.pl (mikolaj++)
> - fix doc in youtube-download.pl (mikolaj++)
> - file_name option is DEPRECATED
> 0.36Mon Oct 31 02:45:48 2011
> - fix default filename problem in youtube-download.pl
> 0.35Tue Oct 18 00:28:58 2011
> - fix typo in youtube-playlists.pl (rt:71721)
> 0.34Sat Oct 15 16:14:57 2011
> - workaround for win32
> - fix suffix problem
> 0.33Wed Oct 12 01:17:34 2011
> - added youtube-download.pl and youtube-playlist.pl (request by 
> st3vil++)
> - fix doc

Can you at least provide links to the executable with sensible names ?
or heck, rename them and the corresponding manpage.

Outside of Windows, it doesn't make any sense to have a .pl extension,
the scripts should be called 'youtube-playlists' and 'youtube-download'.



UPDATE: www/p5-WWW-YouTube-Download to 0.38

2012-01-08 Thread Mikolaj Kucharski
Hi,

The app has new command line tools: youtube-download.pl and
youtube-playlists.pl. First one lets you download youtube videos from
command line (in similar way line many other tools like that). Second
script lets you list URLs to the videos from YouTube playlist url. That
lets you download few viedos at once, for example:

% youtube-playlists.pl PLC9D0C04BB10E1797 | youtube-download.pl


Using this for few weeks now. It depends on new ports which I sent in
last hour to the list. Comments, okays?


Changes between in-tree 0.32 to attached 0.38:

0.38Thu Dec  8 00:44:52 2011
- fix typo (mikolaj++)
- remove wrong code (mikolaj++)
0.37Wed Dec  7 02:33:23 2011
- fix wrong characters in filename in youtube-download.pl (mikolaj++)
- fix doc in youtube-download.pl (mikolaj++)
- file_name option is DEPRECATED
0.36Mon Oct 31 02:45:48 2011
- fix default filename problem in youtube-download.pl
0.35Tue Oct 18 00:28:58 2011
- fix typo in youtube-playlists.pl (rt:71721)
0.34Sat Oct 15 16:14:57 2011
- workaround for win32
- fix suffix problem
0.33Wed Oct 12 01:17:34 2011
- added youtube-download.pl and youtube-playlist.pl (request by 
st3vil++)
- fix doc

http://cpansearch.perl.org/src/XAICRON/WWW-YouTube-Download-0.38/Changes



Index: Makefile
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/Makefile,v
retrieving revision 1.4
diff -u -r1.4 Makefile
--- Makefile26 Aug 2011 09:03:13 -  1.4
+++ Makefile8 Jan 2012 18:25:12 -
@@ -3,7 +3,7 @@
 COMMENT =  simple YouTube interface for downloading videos
 
 MODULES =  cpan
-DISTNAME = WWW-YouTube-Download-0.32
+DISTNAME = WWW-YouTube-Download-0.38
 CATEGORIES =   www
 
 MAINTAINER =   Mikolaj Kucharski 
@@ -20,9 +20,11 @@
 MODCPAN_EXAMPLES = Yes
 MODCPAN_EXAMPLES_DIST =eg
 
-RUN_DEPENDS =  www/p5-libwww \
-   www/p5-URI \
+RUN_DEPENDS =  devel/p5-Term-ANSIColor \
converters/p5-JSON \
-   www/p5-HTML-Parser
+   textproc/p5-XML-TreePP \
+   www/p5-HTML-Parser \
+   www/p5-URI \
+   www/p5-libwww
 
 .include 
Index: distinfo
===
RCS file: /cvs/ports/www/p5-WWW-YouTube-Download/distinfo,v
retrieving revision 1.4
diff -u -r1.4 distinfo
--- distinfo26 Aug 2011 09:03:13 -  1.4
+++ distinfo8 Jan 2012 18:25:12 -
@@ -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.38.tar.gz) = 1pk9bEwpc6zwQMrcyIfR3w==
+RMD160 (WWW-YouTube-Download-0.38.tar.gz) = skvDA0K/4EsHW2t0EQpbuG41IsU=
+SHA1 (WWW-YouTube-Download-0.38.tar.gz) = 9GE/kZ13ScuoIdCNIe38rK0CThE=
+SHA256 (WWW-YouTube-Download-0.38.tar.gz) = 
MN0ZSYcgRXIxAMZW23ZM7AP8Axfi6kNebw/yKw1R7UY=
+SIZE (WWW-YouTube-Download-0.38.tar.gz) = 34185
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 -  1.2
+++ patches/patch-Makefile_PL   8 Jan 2012 18:25:12 -
@@ -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.PLMon Jan 10 22:22:02 2011
-@@ -8,7 +8,6 @@ requires 'JSON';
- requires 'HTML::Entities';
+--- Makefile.PL.orig   Thu Oct  6 21:40:56 2011
 Makefile.PLTue Nov 22 12:10:04 2011
+@@ -10,7 +10,6 @@ requires 'HTML::Entities';
+ requires 'XML::TreePP'; # for youtube-playlists.pl
  
  tests 't/*.t t/*/*.t';
 -author_tests 'xt';
  
- test_requires 'Test::More', '0.96';
- auto_include;
+ install_script 'bin/youtube-download.pl', 'bin/youtube-playlists.pl';
+ 
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 -  1.1.1.1
+++ pkg/PLIST   8 Jan 2012 18:25:12 -
@@ -1,6 +1,10 @@
 @comment $OpenBSD: PLIST,v 1.1.1.1 2011/04/25 13:04:17 todd Exp $
+bin/youtube-download.pl
+bin/youtube-playlists.pl
 ${P5SITE}/WWW/YouTube/
 ${P5SITE}/WWW/YouTube/Download.pm
+@man man/man1/youtube-download.pl.1
+@man man/man1/youtube-playlists.pl.1
 @man man/man3p/WWW::YouTube::Download.3p
 share/examples/p5-WWW-YouTube-Download/
 share/examples/p5-WWW-YouTube-Download/cb.pl


-- 
be