[sane-devel] [PATCHv2] sane-desc: add support for udev's hwdb

Stef stef.dev at free.fr
Thu Aug 1 16:46:50 UTC 2013

On 01/08/2013 17:56, Tom Gundersen wrote:
> On Thu, Aug 1, 2013 at 2:33 PM, Stef <stef.dev at free.fr> wrote:
>> On 31/07/2013 15:39, Tom Gundersen wrote:
>>> On Wed, Jul 31, 2013 at 3:27 PM, Stef <stef.dev at free.fr> wrote:
>>>> On 31/07/2013 14:49, Tom Gundersen wrote:
>>>>> Hi Stef,
>>>>> On Wed, Jul 31, 2013 at 2:39 PM, Stef <stef.dev at free.fr> wrote:
>>>>>>        before applying this patch I'm considering adding a testsuite for
>>>>>> sane-desc first. Since it is too big to be in-lined, I have posted the
>>>>>> corresponding patch athttp://dl.free.fr/vzjrF6ete . It adds a tools
>>>>>> subdirectory to the testsuite directory. In this directory, a test
>>>>>> .desc
>>>>>> file is parsed with sane-desc for all known modes and the resulting
>>>>>> files
>>>>>> are compared to the expected files.
>>>>>> I am waiting for your comments and suggestions.
>>>>> Sounds like a good idea. However, I was unable to download the file. I
>>>>> get "code incorrect"... Any chance you could upload it elsewhere?
>>>>> -t
>>>>       Hello,
>>>>       the http://dl.free.fr/vzjrF6ete URL is working for me in seamonkey
>>>> and
>>>> firefox. Maybe some typo ? I can send the patch directly to you if it
>>>> helps.
>>> Thanks, now it worked. Should the patch have included an
>>> testsuite/tools/Makefile.in ? As it is, ./configure fails.
>>> Cheers,
>>> Tom
>> Hello,
>>      here is the missing file.
> Thanks. As far as I can tell this works well (I tried breaking
> sane-desc and it was reported as expected). It also works with my hwdb
> patch applied.
> Will you commit this and the hwdb patch so I can just submit a
> separate patch adding the two new hwdb modes to the testsuite?
> Cheers,
> Tom

     yes, send a patch, I'll commit all this this week-end.


More information about the sane-devel mailing list