From owner-freebsd-ports-bugs@FreeBSD.ORG Wed Feb 6 21:10:03 2008 Return-Path: Delivered-To: freebsd-ports-bugs@hub.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:4f8:fff6::34]) by hub.freebsd.org (Postfix) with ESMTP id 7D81D16A417 for ; Wed, 6 Feb 2008 21:10:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (freefall.freebsd.org [IPv6:2001:4f8:fff6::28]) by mx1.freebsd.org (Postfix) with ESMTP id 4717113C467 for ; Wed, 6 Feb 2008 21:10:03 +0000 (UTC) (envelope-from gnats@FreeBSD.org) Received: from freefall.freebsd.org (gnats@localhost [127.0.0.1]) by freefall.freebsd.org (8.14.2/8.14.2) with ESMTP id m16LA2F6093752 for ; Wed, 6 Feb 2008 21:10:02 GMT (envelope-from gnats@freefall.freebsd.org) Received: (from gnats@localhost) by freefall.freebsd.org (8.14.2/8.14.1/Submit) id m16LA201093751; Wed, 6 Feb 2008 21:10:02 GMT (envelope-from gnats) Date: Wed, 6 Feb 2008 21:10:02 GMT Message-Id: <200802062110.m16LA201093751@freefall.freebsd.org> To: freebsd-ports-bugs@FreeBSD.org From: Pankov Pavel Cc: Subject: Re: ports/120301: www/Youtube_dl avi support broken X-BeenThere: freebsd-ports-bugs@freebsd.org X-Mailman-Version: 2.1.5 Precedence: list Reply-To: Pankov Pavel List-Id: Ports bug reports List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Wed, 06 Feb 2008 21:10:03 -0000 The following reply was made to PR ports/120301; it has been noted by GNATS. From: Pankov Pavel To: bug-followup@freebsd.org, mel@rachie.is-a-geek.net Cc: Subject: Re: ports/120301: www/Youtube_dl avi support broken Date: Thu, 7 Feb 2008 00:07:13 +0300 --Boundary-00=_BGiqHamoAWAj+lA Content-Type: text/plain; charset="us-ascii" Content-Transfer-Encoding: 7bit Content-Disposition: inline Sorry. Patch attached. If it'll be broken by my mail client, it's here: http://vds.pankov.pp.ru/youtube-dl-fix.patch --Boundary-00=_BGiqHamoAWAj+lA Content-Type: text/x-diff; charset="us-ascii"; name="youtube-dl-fix.patch" Content-Transfer-Encoding: 7bit Content-Disposition: attachment; filename="youtube-dl-fix.patch" Index: www/youtube_dl/files/conv2avi.patch =================================================================== RCS file: /home/ncvs/ports/www/youtube_dl/files/conv2avi.patch,v retrieving revision 1.2 diff -u -r1.2 conv2avi.patch --- www/youtube_dl/files/conv2avi.patch 4 Feb 2008 10:24:47 -0000 1.2 +++ www/youtube_dl/files/conv2avi.patch 6 Feb 2008 09:59:34 -0000 @@ -1,44 +1,37 @@ -Index: www/youtube_dl/files/conv2avi.patch -=================================================================== -RCS file: www/youtube_dl/files/conv2avi.patch -diff -N www/youtube_dl/files/conv2avi.patch ---- /dev/null 1 Jan 1970 00:00:00 -0000 -+++ www/youtube_dl/files/conv2avi.patch 31 Jan 2008 19:32:45 -0000 -@@ -0,0 +1,37 @@ -+--- youtube-dl.orig Wed Jan 30 14:18:50 2008 -++++ youtube-dl Thu Jan 31 21:36:49 2008 -+@@ -208,6 +208,7 @@ -+ cmdl_parser.add_option('-t', '--title', action='store_true', dest='use_title', help='use title in file name') -+ cmdl_parser.add_option('-l', '--literal', action='store_true', dest='use_literal', help='use literal title in file name') -+ cmdl_parser.add_option('-n', '--netrc', action='store_true', dest='use_netrc', help='use .netrc authentication data') -++cmdl_parser.add_option('-a', '--avi', action='store_true', dest='use_avi', help='output file in avi format') -+ cmdl_parser.add_option('-g', '--get-url', action='store_true', dest='get_url', help='print final video URL only') -+ cmdl_parser.add_option('-2', '--title-too', action='store_true', dest='get_title', help='used with -g, print title too') -+ (cmdl_opts, cmdl_args) = cmdl_parser.parse_args() -+@@ -382,6 +383,26 @@ -+ -+ except KeyboardInterrupt: -+ sys.exit('\n') -++ -++# Convert to avi if needed -++if cmdl_opts.use_avi: -++ if cmdl_opts.outfile is None: -++ avi_filename = '%s.avi' % video_filename[:-4] -++ else: -++ avi_filename = '%s.avi' % video_filename -++ -++ try: -++ os.system('%%CONV2AVI_CMD%% "%s" "%s" > %s 2> %s' % (video_filename, avi_filename, os.path.devnull, os.path.devnull)) -++ cond_print('Video file converted to %s\n' % avi_filename) -++ except OSError: -++ sys.exit('Error: unable to convert file.\n') -++ except KeyboardInterrupt: -++ sys.exit('\n') -++ -++ try: -++ os.unlink(video_filename) -++ except OSError: -++ sys.exit('Error: unable to remove downloaded file.\n') -+ -+ # Finish -+ sys.exit() +--- youtube-dl.orig Wed Jan 30 14:18:50 2008 ++++ youtube-dl Thu Jan 31 21:36:49 2008 +@@ -208,6 +208,7 @@ + cmdl_parser.add_option('-t', '--title', action='store_true', dest='use_title', help='use title in file name') + cmdl_parser.add_option('-l', '--literal', action='store_true', dest='use_literal', help='use literal title in file name') + cmdl_parser.add_option('-n', '--netrc', action='store_true', dest='use_netrc', help='use .netrc authentication data') ++cmdl_parser.add_option('-a', '--avi', action='store_true', dest='use_avi', help='output file in avi format') + cmdl_parser.add_option('-g', '--get-url', action='store_true', dest='get_url', help='print final video URL only') + cmdl_parser.add_option('-2', '--title-too', action='store_true', dest='get_title', help='used with -g, print title too') + (cmdl_opts, cmdl_args) = cmdl_parser.parse_args() +@@ -382,6 +383,26 @@ + + except KeyboardInterrupt: + sys.exit('\n') ++ ++# Convert to avi if needed ++if cmdl_opts.use_avi: ++ if cmdl_opts.outfile is None: ++ avi_filename = '%s.avi' % video_filename[:-4] ++ else: ++ avi_filename = '%s.avi' % video_filename ++ ++ try: ++ os.system('%%CONV2AVI_CMD%% "%s" "%s" > %s 2> %s' % (video_filename, avi_filename, os.path.devnull, os.path.devnull)) ++ cond_print('Video file converted to %s\n' % avi_filename) ++ except OSError: ++ sys.exit('Error: unable to convert file.\n') ++ except KeyboardInterrupt: ++ sys.exit('\n') ++ ++ try: ++ os.unlink(video_filename) ++ except OSError: ++ sys.exit('Error: unable to remove downloaded file.\n') + + # Finish + sys.exit() --Boundary-00=_BGiqHamoAWAj+lA--