Hi!

I use ubuntu 6.06 Dapper,
kernel:
Linux ubuntu 2.6.15-28-386 #1 PREEMPT Thu Feb 1 15:51:56 UTC 2007 i686 GNU/Linux
mplayer:
MPlayer 2:0.99+1.0pre7try2+cvs20060117-0ubuntu8 (C) 2000-2006 MPlayer Team
mencoder:
MEncoder 2:0.99+1.0pre7try2+cvs20060117-0ubuntu8 (C) 2000-2006 MPlayer Team
acidrip:
0.14-0.2ubuntu4~dapper1

and the output of mencoder while crop-detecting is: "crop area ..."

So I think it the fix should be:

------------------------------------------------------------------------------------------------------------

--- AcidRip/acidrip.pm.old      2006-10-02 18:51:59.000000000 -0400
+++ AcidRip/acidrip.pm  2007-03-19 23:28:00.000000000 +0100
@@ -680,7 +680,7 @@
     message( "Running " . get_command("cropdetect") );
     open( CROP, get_command("cropdetect") . " 2>&1 |" );
     while (<CROP>) {
-      if ( $_ =~ /crop area.*\s(crop=\d*:\d*:\d*:\d*)/ ) { $crop{$1}++ }
+      if ( $_ =~ /crop area.*\s(crop=\d*:\d*:\d*:\d*)/i ) { $crop{$1}++ }
       $crop_output .= $_;
       gui_check($_);
     }

------------------------------------------------------------------------------------------------------------
to provide case insensitive match.
It fixed my problem.


Roti

** Attachment added: "case insensitive patch"
   
http://librarian.launchpad.net/6871609/Patch%20for%20proper%20crop%20detection%202.diff

-- 
Crop detection fails
https://launchpad.net/bugs/63582

-- 
ubuntu-bugs mailing list
ubuntu-bugs@lists.ubuntu.com
https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs

Reply via email to