kicad-developers team mailing list archive
-
kicad-developers team
-
Mailing list archive
-
Message #33322
Re: [PATCH] Defer canvas type setting save until destruction of EDA_DRAW_FRAME
-
To:
<kicad-developers@xxxxxxxxxxxxxxxxxxx>
-
From:
Maciej Sumiński <maciej.suminski@xxxxxxx>
-
Date:
Thu, 18 Jan 2018 10:23:14 +0100
-
Authentication-results:
spf=pass (sender IP is 188.184.36.46) smtp.mailfrom=cern.ch; lists.launchpad.net; dkim=none (message not signed) header.d=none;lists.launchpad.net; dmarc=bestguesspass action=none header.from=cern.ch;
-
In-reply-to:
<487aaae7-b75e-629c-2607-bb67a3c6b30b@cern.ch>
-
Spamdiagnosticmetadata:
NSPM
-
Spamdiagnosticoutput:
1:99
-
User-agent:
Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.5.2
I pushed both patches to the master branch. Jon, thank you for your
contribution to KiCad.
Cheers,
Orson
On 01/17/2018 05:54 PM, Maciej Sumiński wrote:
> I have just tested the patch on a Windows virtual machine where KiCad
> normally crashed and it seems that the fallback mechanism works as it
> should.
>
> Chris, if you have time, please check it on you hardware too, if not - I
> will just push the patch to the master branch. I vaguely recall that
> Linux without OpenGL support might crash even before KiCad has a chance
> to throw an exception (X.Org error). Perhaps this case could be resolved
> with Jon's patch.
>
> Cheers,
> Orson
>
> On 01/17/2018 04:30 PM, Maciej Sumiński wrote:
>> An alternative approach attached. Perhaps the two patches could be
>> combined for extra safety.
>>
>> Cheers,
>> Orson
>>
>> On 01/16/2018 08:15 PM, Maciej Suminski wrote:
>>> For me - not entirely, but definitely it is better than the current
>>> implementation. If you run KiCad for the first time and select 'Enable
>>> hardware acceleration' on a machine that does not support OpenGL, it
>>> switches to Cairo (correctly). Then, when you turn off KiCad, it will
>>> save OpenGL as the default canvas and you will not be able to run pcbnew
>>> again until you manually modify the configuration files (or remove them).
>>>
>>> I will have another trial at fixing the problem, finally I have an
>>> environment where I can test different solutions.
>>>
>>> Cheers,
>>> Orson
>>>
>>> On 01/16/2018 07:04 PM, Chris Pavlina wrote:
>>>> Yes! You got it, works for me :)
>>>>
>>>> On Tue, Jan 16, 2018 at 05:16:57PM +0000, Chris Pavlina wrote:
>>>>> Sorry I disappeared for a while. I'm building on the machine where I
>>>>> experienced the bug right now.
>>>>>
>>>>> On Tue, Jan 16, 2018 at 04:26:16PM +0000, Jon Evans wrote:
>>>>>> Wayne/Orson, I'm not sure Chris will have time to review this, could you
>>>>>> please take a look?
>>>>>>
>>>>>> Thanks,
>>>>>> Jon
>>>>>>
>>>>>> On Thu, Jan 11, 2018 at 11:02 PM, Jon Evans <jon@xxxxxxxxxxxxx> wrote:
>>>>>>
>>>>>>> Updated patch attached (missed initializers in the original)
>>>>>>>
>>>>>>> On Wed, Jan 10, 2018 at 11:40 PM, Jon Evans <jon@xxxxxxxxxxxxx> wrote:
>>>>>>>
>>>>>>>> Hopefully this fixes:
>>>>>>>> https://bugs.launchpad.net/kicad/+bug/1741787
>>>>>>>> although I don't currently have a good way of reproducing the crash.
>>>>>>>>
>>>>>>>> Chris, if you can give it a try on your crashing machine that would be
>>>>>>>> greatly appreciated.
>>>>>>>>
>>>>>>>> Thanks
>>>>>>>> -Jon
>>>>>>>>
>>>>>>>
>>>>>>>
>>>>>
>>>
>>> _______________________________________________
>>> 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
>>
>
>
>
>
> _______________________________________________
> 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
>
Attachment:
signature.asc
Description: OpenPGP digital signature
References