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