tags 608790 patch
tags 608790 pending
thanks

On Thu, Jan 13, 2011 at 11:10:53PM +0300, Laurent Lemoine wrote:
> It's better with your new amd64 deb - Now I get images from my two cameras.

good to hear, i'll prepare an upload today or tomorrow.

i touched up the patch a little, as the previous one i posted had some
whitespace inconsistancies:

Index: zoneminder/src/zm_local_camera.cpp
===================================================================
--- zoneminder.orig/src/zm_local_camera.cpp     2011-01-14 11:49:44.000000000 
-0800
+++ zoneminder/src/zm_local_camera.cpp  2011-01-14 11:54:14.000000000 -0800
@@ -908,7 +908,8 @@
             struct v4l2_crop crop;
             memset( &crop, 0, sizeof(crop) );
             crop.type = V4L2_BUF_TYPE_VIDEO_CAPTURE;
-            if ( vidioctl( vid_fd, VIDIOC_G_CROP, &crop ) < 0 )
+            int crop_rc = vidioctl( vid_fd, VIDIOC_G_CROP, &crop );
+            if ((crop_rc  < 0 ) && (errno != EINVAL))
             {
                 Error( "Failed to query crop: %s", strerror(errno) );
                 if ( verbose )
@@ -917,8 +918,14 @@
                     sprintf( output, "error%d\n", errno );
                 return( false );
             }
-            if ( verbose )
-                sprintf( output+strlen(output), "  Current: %d x %d\n", 
crop.c.width, crop.c.height );
+            if ( verbose ) {
+                if (crop_rc >= 0) {
+                   sprintf( output+strlen(output), "  Current: %d x %d\n", 
crop.c.width, crop.c.height );
+                }
+                else {
+                    sprintf( output+strlen(output), "  Current: Cropping is 
not supported\n");
+                }
+            }
 
             struct v4l2_input input;
             int inputIndex = 0;


 
> But I still can't get informations with
> 
> zmu -d /dev/video0 -q -v -U *** -P **** (failed to switch or failed
> to querry)

since that's not absolutely needed for zoneminder to function, we'll have to
address that later.

 
> And I still can't probe for cameras (like Piratebab) in the webgui.

i think that's an upstream issue that will need more work, also later.


> Thanks for your work, and thanks for the working deb! :)

thanks for testing.


live well,
  vagrant



-- 
To UNSUBSCRIBE, email to debian-bugs-rc-requ...@lists.debian.org
with a subject of "unsubscribe". Trouble? Contact listmas...@lists.debian.org

Reply via email to