[Pkg-phototools-devel] Bug#531978: Bug#531978: Bug#531978: libgphoto2: Please modernize udev rules

[Martin Pitt]

Hello Cyril,

sorry for the delay, I was on holidays.

Cyril Brulebois [2009-08-02 14:34 +0200]:
> | Files in second .changes but not in first
> | -----------------------------------------
> | -rw-r--r--  root/root   /etc/udev/rules.d/z60_libgphoto2-2.rules
> | -rw-r--r--  root/root   /usr/share/doc/libgphoto2-2/README.apidocs
> | -rw-r--r--  root/root   DEBIAN/conffiles
> | -rwxr-xr-x  root/root   DEBIAN/preinst
> | 
> | Files in first .changes but not in second
> | -----------------------------------------
> | -rw-r--r--  root/root   /usr/share/doc/libgphoto2-2-dev/DAEMON.gz
> | -rwxr-xr-x  root/root   /lib/udev/check-ptp-camera
> 
> Did you observe that? The first file seems to use the deprecated filename
> scheme, doesn't it? 

Indeed it does. It should be /lib/udev/rules.d/40-libgphoto2-2.rules.
I guess Debian's dh_installudev does not yet do the right thing here.
So perhaps you can modify the patch to do it manually, i. e. replace

  +	CAMLIBS=`ls -d debian/libgphoto2-$(major)/usr/lib/libgphoto2/2*` ./packaging/generic/print-camera-list udev-rules version 136 mode 0664 group plugdev > debian/libgphoto2-$(major).udev
  +	dh_installudev
  +	rm debian/libgphoto2-$(major).udev

with

  + mkdir -p debian/libgphoto2-$(major)/lib/udev/rules.d
  + CAMLIBS=`ls -d debian/libgphoto2-$(major)/usr/lib/libgphoto2/2*` ./packaging/generic/print-camera-list udev-rules version 136 mode 0664 group plugdev > debian/libgphoto2-$(major)/lib/udev/rules.d/40-libgphoto2-2.rules

?

Dropping preinst, conffiles (due to /etc -> /lib) looks ok. My patch
didn't touch documentation files, were these from other changes that
you made?

> Also, it looks quite empty, so I'm not sure everything's
> working properly. Quoting it:

This was the case with the first patch (forgot to specify CAMLIBS).
Did you try the latest one with CAMLIBS?

Thanks,

Martin
-- 
Martin Pitt                        | http://www.piware.de
Ubuntu Developer (www.ubuntu.com)  | Debian Developer  (www.debian.org)
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 197 bytes
Desc: Digital signature
URL: <http://lists.alioth.debian.org/pipermail/pkg-phototools-devel/attachments/20090823/ff05e7a8/attachment.pgp>