From nobody@FreeBSD.org  Sun Jan  9 10:39:45 2005
Return-Path: <nobody@FreeBSD.org>
Received: from mx1.FreeBSD.org (mx1.freebsd.org [216.136.204.125])
	by hub.freebsd.org (Postfix) with ESMTP id 1213C16A4CE
	for <freebsd-gnats-submit@FreeBSD.org>; Sun,  9 Jan 2005 10:39:45 +0000 (GMT)
Received: from www.freebsd.org (www.freebsd.org [216.136.204.117])
	by mx1.FreeBSD.org (Postfix) with ESMTP id F145B43D46
	for <freebsd-gnats-submit@FreeBSD.org>; Sun,  9 Jan 2005 10:39:44 +0000 (GMT)
	(envelope-from nobody@FreeBSD.org)
Received: from www.freebsd.org (localhost [127.0.0.1])
	by www.freebsd.org (8.13.1/8.13.1) with ESMTP id j09Adio4021845
	for <freebsd-gnats-submit@FreeBSD.org>; Sun, 9 Jan 2005 10:39:44 GMT
	(envelope-from nobody@www.freebsd.org)
Received: (from nobody@localhost)
	by www.freebsd.org (8.13.1/8.13.1/Submit) id j09AdiY8021787;
	Sun, 9 Jan 2005 10:39:44 GMT
	(envelope-from nobody)
Message-Id: <200501091039.j09AdiY8021787@www.freebsd.org>
Date: Sun, 9 Jan 2005 10:39:44 GMT
From: Stephan van Maris <svmaris@e-village.nl>
To: freebsd-gnats-submit@FreeBSD.org
Subject: Current DVD::Rip (0.50.18_1) incompatible with current transcode (0.6.12) in Cluster-mode
X-Send-Pr-Version: www-2.3

>Number:         75985
>Category:       ports
>Synopsis:       Current DVD::Rip (0.50.18_1) incompatible with current transcode (0.6.12) in Cluster-mode
>Confidential:   no
>Severity:       non-critical
>Priority:       low
>Responsible:    lofi
>State:          closed
>Quarter:        
>Keywords:       
>Date-Required:  
>Class:          sw-bug
>Submitter-Id:   current-users
>Arrival-Date:   Sun Jan 09 10:40:24 GMT 2005
>Closed-Date:    Tue Jan 11 22:41:55 GMT 2005
>Last-Modified:  Tue Jan 11 22:41:55 GMT 2005
>Originator:     Stephan van Maris
>Release:        6-CURRENT
>Organization:
>Environment:
FreeBSD pezdispenser.local 6.0-CURRENT FreeBSD 6.0-CURRENT #0: Fri Jan  7 07:57:09 CET 2005     root@pezdispenser.local:/usr/obj/usr/src/sys/PEZDISPENSER  i386
>Description:
When DVD::Rip is multiplexing the video- and the audio-stream with the
avimerge program (which is included in multimedia/transcode), it sends
the commandline arguments in an order that 0.6.12 does not understand.
Transcode 0.6.14 (on my Gentoo box) has no problems with it.
>How-To-Repeat:

>Fix:
Until 0.6.14 will be available in ports, you can change the order
of the arguments in 

/usr/local/lib/perl5/site_perl/5.8.5/Video/DVDRip/Cluster/Title.pm

on the machine running dvdrip-master on line 314 so that the
'-i xxxxx' will be at the end of the avimerge command.
>Release-Note:
>Audit-Trail:
Responsible-Changed-From-To: freebsd-ports-bugs->lofi 
Responsible-Changed-By: lofi 
Responsible-Changed-When: Sun Jan 9 14:25:35 GMT 2005 
Responsible-Changed-Why:  
Over to maintainer (myself). 


http://www.freebsd.org/cgi/query-pr.cgi?pr=75985 

From: Michael Nottebrock <michaelnottebrock@gmx.net>
To: freebsd-gnats-submit@freebsd.org, svmaris@e-village.nl
Cc:  
Subject: Re: ports/75985: Current DVD::Rip (0.50.18_1) incompatible with current transcode (0.6.12) in Cluster-mode
Date: Sun, 9 Jan 2005 15:23:49 +0100

 Can you please submit a patch?

From: Stephan van Maris <svmaris@e-village.nl>
To: Michael Nottebrock <michaelnottebrock@gmx.net>
Cc: freebsd-gnats-submit@freebsd.org
Subject: Re: ports/75985: Current DVD::Rip (0.50.18_1) incompatible with current
 transcode (0.6.12) in Cluster-mode
Date: Mon, 10 Jan 2005 09:38:16 +0100

 Sure:
 
 (ps. I noticed that this affects non-cluster operations too and is not
 limited to the machine running dvdrip-master)
 
 patch-lib::Video::DVDRip::Title.pm
 
 --- Video-DVDRip-0.50.18/lib/Video/DVDRip/Cluster/Title.pm      Sat Apr 10 11:18:03 2004
 +++ Video-DVDRip-0.50.18-patched/lib/Video/DVDRip/Cluster/Title.pm      Sun Jan  9 17:41:11 2005
 @@ -276,10 +276,10 @@
        } else {
                $command .=
                        "dr_exec avimerge".
 -                       " -i $avi_file".
                        " -p $audio_file".
                        " -a $target_nr".
 -                       " -o $avi_file.merged &&".
 +                       " -o $avi_file.merged ".
 +                       " -i $avi_file &&".
                        " mv $avi_file.merged $target_file &&".
                        " rm $audio_file &&".
                        " echo DVDRIP_SUCCESS";
 @@ -312,10 +312,9 @@
 
        my $command =
                "mkdir -m 0775 -p '$audio_video_psu_dir' && ".
 -               "${nice}dr_exec avimerge -i $avi_chunks_dir/*".
 -               " -o $audio_video_psu_file ";
 -       
 +               "${nice}dr_exec avimerge -o $audio_video_psu_file";
        $command .= " -p $audio_psu_file " if not $self->is_ogg;
 +        $command .= " -i $avi_chunks_dir/*";
 
        $command .= " && rm $avi_chunks_dir/*"
                        if $self->with_cleanup;
 
 
 
 patch-lib::Video::DVDRip::Title.pm
 
 --- Video-DVDRip-0.50.18/lib/Video/DVDRip/Title.pm      Sun Apr 18 16:19:58 2004
 +++ Video-DVDRip-0.50.18-patched/lib/Video/DVDRip/Title.pm      Sun Jan  9 17:34:16 2005
 @@ -2152,10 +2152,10 @@
 
                $command .=
                        "dr_exec avimerge".
 -                       " -i $avi_file".
                        " -p $audio_file".
                        " -a $target_nr".
 -                       " -o $avi_file.merged &&".
 +                       " -o $avi_file.merged".
 +                       " -i $avi_file &&".
                        " mv $avi_file.merged $avi_file &&".
                        " rm $audio_file &&".
                        " echo DVDRIP_SUCCESS";
 
 
 
 
 Michael Nottebrock wrote:
 > Can you please submit a patch?
 > 
 

From: Michael Nottebrock <michaelnottebrock@gmx.net>
To: freebsd-gnats-submit@freebsd.org, svmaris@e-village.nl
Cc:  
Subject: Re: ports/75985: Current DVD::Rip (0.50.18_1) incompatible with current transcode (0.6.12) in Cluster-mode
Date: Mon, 10 Jan 2005 18:26:48 +0100

 Can you resend the patches as proper attachments (either to GNATS or directly 
 to me)? The patches you sent have tabs converted to spaces and don't apply.

From: Stephan van Maris <svmaris@e-village.nl>
To: Michael Nottebrock <michaelnottebrock@gmx.net>
Cc: freebsd-gnats-submit@freebsd.org
Subject: Re: ports/75985: Current DVD::Rip (0.50.18_1) incompatible with current
 transcode (0.6.12) in Cluster-mode
Date: Tue, 11 Jan 2005 03:31:45 +0100

 I sent the patches directly to you.
 
 On another note (since I've been actively using the Cluster-mode to
 backup my DVD-collection): The 'Smart Deinterlacer' will crash randomly
 on different nodes. So far I have had best results with the 'Zoom to Full
 Frame'-option. I'll look into this problem later, but so far it seems like
 a buggy transcode.
 
 Michael Nottebrock wrote:
 > Can you resend the patches as proper attachments (either to GNATS or directly 
 > to me)? The patches you sent have tabs converted to spaces and don't apply.
 > 

From: Stephan van Maris <svmaris@e-village.nl>
To: Michael Nottebrock <michaelnottebrock@gmx.net>
Cc: freebsd-gnats-submit@freebsd.org
Subject: Re: ports/75985: Current DVD::Rip (0.50.18_1) incompatible with current
 transcode (0.6.12) in Cluster-mode
Date: Tue, 11 Jan 2005 03:41:37 +0100

 I just remembered that I did something else to get DVD::Rip to
 work: When you press 'Grab Frame' on the 'Clip & Zoom'-tab, DVD::Rip
 calls 'identify' (part of the ImageMagick-tools), to get info on the
 frame. But it passes the '-ping' option which apparently has been removed
 a while ago, so it returns an error and doesn't work.
 
 I worked around it by making a shell-wrapper for the 'real' identify-command
 which only passes $2 (without the '-ping') to the binary, but I will
 supply patches for DVD::Rip shortly.
State-Changed-From-To: open->closed 
State-Changed-By: lofi 
State-Changed-When: Tue Jan 11 22:41:07 GMT 2005 
State-Changed-Why:  
Committed, thanks! (Imagemagick-issue fixed as well) 


http://www.freebsd.org/cgi/query-pr.cgi?pr=75985 
>Unformatted:
