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

[Tom Gundersen]

On 3 Aug 2013 08:40, "Stef" <stef.dev at free.fr> wrote:
>
> On 01/08/2013 18:46, Stef wrote:
>>
>> 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
>>
>>     Hello,
>>
>>     yes, send a patch, I'll commit all this this week-end.
>>
>> Regards,
>>     Stef
>>
>
>     Hello,
>
>     I have committed all these patches.
>
> Regards,
>     Stef

Thanks!

Tom
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.alioth.debian.org/pipermail/sane-devel/attachments/20130803/02571b1c/attachment.html>