I looked at the old patch and it was great because Javier did the hard
work of rewriting the scansnap backend to work with the S1500. The only
things he did incorrectly is leaving the Epson scanner USB IDs in
the supported_usb_devices variable, and similary for the
usb_device_descriptions variable and the NUM_SUPPORTED_USB_DEVICES
define.

I see in patch set 9, you added a new 'fujitsu' backend, which is
essentially what Javier's patch should have been. IMO, if you are going
to keep fujitsu.c, it's better to just replace to remove the scansnap
backend altogether. It is just a duplicate of the epson backend.
Alternatively, don't add fujitsu.c, but replace scanscap.c's contents
with fujitsu.c's contents.

FWIW, I took Javier's patch and removed the Epson scanner USB IDs, and
it worked fine for me. I imagine without the Epson IDs, it would not
interfere with Epson scanners. It looks like you will have it worked
out soon.  Please backport the fix to Debian Squeeze too. Thanks!


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

Reply via email to