Re: acidrip: doesnt work with latest versions of mencoder
Le 19.10.2005 22:40:27, Corey Hickey a écrit :
Jean-Luc Coulon (f5ibh) wrote:
me) dont dont accept the -xvidencopts suboption which is used by
acidrip, typically with something like:
-xvidencopts :bitrate=759:pass=2 ...
Yes, the ':' before bitrate is a syntax error. I don't know if
earlier
versions of mencoder were tolerant of that.
I don't know perl, but I can see what's going on.
Line 221 of AcidRip/acidrip.pm:
$menc{'video'} = -ovc xvid -xvidencopts
$::settings-{'xvid_options'}:bitrate=$::settings-{'video
_bitrate'};
xvid_options is empty, leaving only the ':'. Mencoder accepts a
trailing
':', so you ought to be able to just reorder that line, putting
xvid_options at the end. Try the attached patch. If that fixes the
xvidencopts part, you might have to follow my example for some of the
other nearby lines in AcidRip/acidrip.pm.
Yes, it works, thanks.
But I think even this way it is not clean: next time, mencoder will
refuse also the trailing :. So I think the best is to test if
xvid_options is empty (it is initialised to '' at the beginning) and
create 2 different lines depending of the result of the test.
Something like:
if ( $::settings-{'video_codec'} eq 'xvid' ) {
if ($xvid_options) ne '' {
$menc{'video'} = -ovc xvid -xvidencopts
$::settings-{'xvid_options'}:bitrate=$::settings-{'video_bitrate'};
} else {
$menc{'video'} = -ovc xvid -xvidencopts
bitrate=$::settings-{'video_bitrate'};
}
or:
if ( $::settings-{'video_codec'} eq 'xvid' ) {
$memc{'video'} = -ovc xvid -xvidencopts ;
$menc{'video'} .= $::settings-{'xvid_options'}: if
$xvid_options ne '';
$menc{'video'} .= bitrate=$::settings-{'video_bitrate'};
$menc{'video'} .= :pass=$::settings-{'video_pass'} if
$::settings-{'video_passes'} 1;
}
-Corey
Jean-Luc
pgpNlsQ6meBej.pgp
Description: PGP signature
acidrip: doesnt work with latest versions of mencoder
Hi, The latest versions of mencoder (cvs) that are newer than 20051011 (for me) dont dont accept the -xvidencopts suboption which is used by acidrip, typically with something like: -xvidencopts :bitrate=759:pass=2 ... Is there any way to use latest version of mencoder with acidrip? Regards Jean-Luc pgpEABGCXbBlF.pgp Description: PGP signature
Re: acidrip: doesnt work with latest versions of mencoder
Le 19.10.2005 20:58:13, Jean-Luc Coulon (f5ibh) a écrit : Hi, The latest versions of mencoder (cvs) that are newer than 20051011 (for me) dont dont accept the -xvidencopts suboption which is used by acidrip, typically with something like: -xvidencopts :bitrate=759:pass=2 ... *** In fact the proble seems to be here: there is nothing before bitrate but the leading ':' is still here. Older versionq of mencoder accept this syntax. Is there any way to use latest version of mencoder with acidrip? Regards J-L pgpG1wFSXn7WE.pgp Description: PGP signature
Re: acidrip: doesnt work with latest versions of mencoder
Jean-Luc Coulon (f5ibh) wrote:
Hi,
The latest versions of mencoder (cvs) that are newer than 20051011 (for
me) dont dont accept the -xvidencopts suboption which is used by
acidrip, typically with something like:
-xvidencopts :bitrate=759:pass=2 ...
Yes, the ':' before bitrate is a syntax error. I don't know if earlier
versions of mencoder were tolerant of that.
In any case, you'd be best off contacting Chris Phillips, the author of
acidrip. His address is listed at the bottom of the acidrip file (the
perl script itself). Just use a text editor and scroll to the end. Or,
if you want to be fancy, use 'perldoc acidrip'.
-Corey
--
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of unsubscribe. Trouble? Contact [EMAIL PROTECTED]
Re: acidrip: doesnt work with latest versions of mencoder
Le 19.10.2005 21:15:33, Corey Hickey a écrit :
Jean-Luc Coulon (f5ibh) wrote:
Hi,
The latest versions of mencoder (cvs) that are newer than 20051011
(for
me) dont dont accept the -xvidencopts suboption which is used by
acidrip, typically with something like:
-xvidencopts :bitrate=759:pass=2 ...
Yes, the ':' before bitrate is a syntax error. I don't know if earlier
versions of mencoder were tolerant of that.
yes, I managed to write an avi file with cvs20051011
In any case, you'd be best off contacting Chris Phillips, the author
of
acidrip. His address is listed at the bottom of the acidrip file (the
perl script itself). Just use a text editor and scroll to the end. Or,
if you want to be fancy, use 'perldoc acidrip'.
Well, I've opended a bug on sourceforge.
I think this works also ?
-Corey
Jean-Luc
pgpnOGzvdJzzL.pgp
Description: PGP signature
Re: acidrip: doesnt work with latest versions of mencoder
Jean-Luc Coulon (f5ibh) wrote:
me) dont dont accept the -xvidencopts suboption which is used by
acidrip, typically with something like:
-xvidencopts :bitrate=759:pass=2 ...
Yes, the ':' before bitrate is a syntax error. I don't know if earlier
versions of mencoder were tolerant of that.
I don't know perl, but I can see what's going on.
Line 221 of AcidRip/acidrip.pm:
$menc{'video'} = -ovc xvid -xvidencopts
$::settings-{'xvid_options'}:bitrate=$::settings-{'video
_bitrate'};
xvid_options is empty, leaving only the ':'. Mencoder accepts a trailing
':', so you ought to be able to just reorder that line, putting
xvid_options at the end. Try the attached patch. If that fixes the
xvidencopts part, you might have to follow my example for some of the
other nearby lines in AcidRip/acidrip.pm.
-Corey
diff -aur acidrip-0.14.orig/AcidRip/acidrip.pm acidrip-0.14/AcidRip/acidrip.pm
--- acidrip-0.14.orig/AcidRip/acidrip.pm2004-07-25 07:03:09.0
-0700
+++ acidrip-0.14/AcidRip/acidrip.pm 2005-10-19 13:37:57.0 -0700
@@ -218,7 +218,7 @@
$menc{'video'} .= :pass=$::settings-{'video_pass'} if
$::settings-{'video_passes'} 1;
}
if ( $::settings-{'video_codec'} eq 'xvid' ) {
-$menc{'video'} = -ovc xvid -xvidencopts
$::settings-{'xvid_options'}:bitrate=$::settings-{'video_bitrate'};
+$menc{'video'} = -ovc xvid -xvidencopts
bitrate=$::settings-{'video_bitrate'}:$::settings-{'xvid_options'};
$menc{'video'} .= :pass=$::settings-{'video_pass'} if
$::settings-{'video_passes'} 1;
}
if ( $::settings-{'video_codec'} eq 'nuv' ) {
