← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Wildcard tests + removed variadic interface

 

John,

I merged your patch set into the development branch.

Thanks,

Wayne

On 1/9/2019 4:54 AM, John Beard wrote:
> Hi Wayne,
> 
> It means someone else added a test! :-)
> 
> Here is a rebase of these patches.
> 
> Cheers,
> 
> John
> 
> On Wed, Jan 9, 2019 at 1:58 AM Wayne Stambaugh <stambaughw@xxxxxxxxx> wrote:
>>
>> John,
>>
>> Would you please rebase your patches?  Patch 0001 no longer applies
>> cleaning.  Apparently someone made a change the caused a conflict before
>> I could get around to merging them.
>>
>> Thanks,
>>
>> Wayne
>>
>> On 1/8/2019 12:35 PM, John Beard wrote:
>>> Hi Seth,
>>>
>>> Here are updated patches (only #3 is changed).
>>>
>>> I have still hardcoded the expected results in the test, otherwise
>>> it'll be a fairly opaque test of two copy-pasted lines.
>>>
>>> The windows filter is now " (*.*)|*.*": is that right?
>>>
>>> Cheers,
>>>
>>> John
>>>
>>> On Tue, Jan 8, 2019 at 4:48 PM Seth Hillbrand <seth@xxxxxxxxxxxxx> wrote:
>>>>
>>>> Am 2019-01-08 11:46, schrieb John Beard:
>>>>> Hi Seth,
>>>>>
>>>>> Good idea - I was just moving what the code currently did, but sounds
>>>>> like a sensible fix. I'll amend patch 0003.
>>>>>
>>>>> Does it otherwise work on Windows?
>>>>
>>>> Unfortunately, I don't have a Windows setup so we'll need to wait for an
>>>> available Windows tester on the patch.
>>>>
>>>> -S
>>>>
>>>> _______________________________________________
>>>> Mailing list: https://launchpad.net/~kicad-developers
>>>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>>>> Unsubscribe : https://launchpad.net/~kicad-developers
>>>> More help   : https://help.launchpad.net/ListHelp
>>
>> _______________________________________________
>> Mailing list: https://launchpad.net/~kicad-developers
>> Post to     : kicad-developers@xxxxxxxxxxxxxxxxxxx
>> Unsubscribe : https://launchpad.net/~kicad-developers
>> More help   : https://help.launchpad.net/ListHelp


References