[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
** Changed in: libgphoto2 (Ubuntu) Status: Fix Released = In Progress -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Nikias, thanks for spotting this. Odd, it worked fine in my local build. I'll track this down. -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
This bug was fixed in the package libgphoto2 - 2.4.6-1ubuntu5 --- libgphoto2 (2.4.6-1ubuntu5) karmic; urgency=low * debian/rules: Set CAMLIBS to local directory when calling print-camera-list, so that it also works on the buildds. * debian/patches/02_iphone_fdi: Update to use correct interface class/protocol, thanks to Nikias Bassen! (LP: #386002) -- Martin Pitt martin.p...@ubuntu.com Mon, 15 Jun 2009 09:17:37 +0200 ** Changed in: libgphoto2 (Ubuntu) Status: In Progress = Fix Released -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Thanks Marcus! ** Changed in: libgphoto2 (Ubuntu) Status: New = In Progress ** Changed in: libgphoto2 (Ubuntu) Assignee: (unassigned) = Martin Pitt (pitti) -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
I guess the udev rules need a similar change then? -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
This bug was fixed in the package libgphoto2 - 2.4.6-1ubuntu4 --- libgphoto2 (2.4.6-1ubuntu4) karmic; urgency=low * debian/rules: Fix name of FDI file. * Add debian/patches/02_iphone_fdi: Detect iPhones in hal FDI properly. Taken from upstream SVN, thanks Marcus Meissner! (LP: #386002) -- Martin Pitt martin.p...@ubuntu.com Sun, 14 Jun 2009 22:25:59 +0200 ** Changed in: libgphoto2 (Ubuntu) Status: In Progress = Fix Released -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Martin, what udev rules are you talking about? The default ubuntu udev rules do not deal with Apple PTP devices. Everything else is implemented with libiphone+usbmuxd which ships with a udev rule on its own. -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Ah, forgot something, will there be a backport for Jaunty? -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
Re: [Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Nikias Bassen [2009-06-14 21:07 -]: Ah, forgot something, will there be a backport for Jaunty? No, this isn't a bug which matches our stable release update criteria. (See https://wiki.ubuntu.com/StableReleaseUpdates) -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
Re: [Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Nikias Bassen [2009-06-14 21:06 -]: Martin, what udev rules are you talking about? print-camera-list doesn't just produce hal FDI files, but also udev rules (hal is currently being deprecated). However, the patch was specific to hal FDIs, so I wondered whether the udev rules also need to be updated. -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Martin, thanks for the quick answers. I took a look at the udev rules that are produced by print-camera-list, they seem to be used to set the ownership and permissions of the usb device files. AFAIK these udev rules are not used within Jaunty; and in Karmic, not even the fdi rules are generated automatically. So it should be no problem. -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
Re: [Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Nikias Bassen [2009-06-14 22:13 -]: Martin, thanks for the quick answers. I took a look at the udev rules that are produced by print-camera-list, they seem to be used to set the ownership and permissions of the usb device files. AFAIK these udev rules are not used within Jaunty; and in Karmic, not even the fdi rules are generated automatically. So it should be no problem. The Karmic package generates both udev rules (in mode 136) and the fdi during package build now and ships them statically. Check /lib/udev/rules.d/40-libgphoto2-2.rules. It's not yet used, but it hopefully will soon. -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
Ah well I see. But I just found a problem. The fdi file shipped with the debian file does not contain any matching except info.subsystem usb. This is because print-camera-list is built with a libtool wrapper script that seems to be used during package building instead of the final binary file. This should not, but indeed does, produce a different output than the final binary file which is located in packaging/generic/.libs/print-camera-list after building. Possibly the udev file is also generated Besides that, there is another problem. If I use the correct print-camera-list program with the hal-fdi argument, the fdi file is generated BUT the matching for Apple iPhones is still incorrect, using usb.interface.class=0, usb.interface.subclass=0, and usb.interface.protocol=0. Attached is the corrected patch from above using 6:1:1. @Marcus: You should fix this in libgphoto2 SVN, too. @Martin: Perhaps you should use ./packaging/generic/.libs/print-camera-list udev-rules [...] and ./packaging/generic/.libs/print-camera-list hal-fdi instead of ./packaging/generic/print-camera-list udev-rules [...] and ./packaging/generic/print-camera-list hal-fdi inside the debian/rules file which worked for me. ** Attachment added: xx-corrected.patch http://launchpadlibrarian.net/27908913/xx-corrected.patch -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
concerning debian/rules: My suggestions works but print-camera-list is executed with the installed version of libgphoto2 and not the one that has just been compiled. Perhaps it might be better NOT to ship static files for udev/fdi and put the file creation back into libgphoto2-2.postinst to make sure the files are really created with the latest version of the library. Attached is a patch that adds iPod Touch 1G and renames the already present iPod Touch entry to iPod Touch 2G, just to be complete. If the iPhone 3G S product_id has been verified, you can add it too (I bet it is 0x1294). ** Attachment added: ipod_touch.patch http://launchpadlibrarian.net/27909661/ipod_touch.patch -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
please dont yet, it is not working as intended. everything is now detected as Apple iPhone with this patches -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
[Bug 386002] Re: Please package changes in r12140..12143 (iPhone/Touch)
better use this patch ** Attachment added: xx.pat http://launchpadlibrarian.net/27839674/xx.pat -- Please package changes in r12140..12143 (iPhone/Touch) https://bugs.launchpad.net/bugs/386002 You received this bug notification because you are a member of Ubuntu Bugs, which is subscribed to Ubuntu. -- ubuntu-bugs mailing list ubuntu-bugs@lists.ubuntu.com https://lists.ubuntu.com/mailman/listinfo/ubuntu-bugs
