[PATCH 2/5] build: Use CSafeLoader if available

Sebastian Huber sebastian.huber at embedded-brains.de
Tue May 2 05:33:35 UTC 2023


On 27.04.23 23:59, Chris Johns wrote:
> On 28/4/2023 4:27 am, Gedare Bloom wrote:
>> On Wed, Apr 26, 2023 at 11:46 PM Sebastian Huber
>> <sebastian.huber at embedded-brains.de>  wrote:
>>> On 27.04.23 02:11, Chris Johns wrote:
>>>> On 26/4/2023 6:04 pm, Sebastian Huber wrote:
>>>>> The CSafeLoader uses the C libyaml libary to considerably speed up the
>>>>> loading of YAML files.
>>>> No from me.
>>> What do you mean with not for me? You have the CSafeLoader available and
>>> it is slow? Do you have some timings before and after the patch set for
>>> a "./waf configure" and "./waf build"? On my systems the configure needs
>>> less than a second with the CSafeLoader and the waf build setup time is
>>> less than 100ms.
> I do not agree this changing being merged as is. A move to csafeloader for all
> users is a good idea but not conditionally.

Ok, good. I have currently no time to dig into the specifics of 
deploying a Python module with C extensions, so I will simply use this 
patch set internally.

-- 
embedded brains GmbH
Herr Sebastian HUBER
Dornierstr. 4
82178 Puchheim
Germany
email: sebastian.huber at embedded-brains.de
phone: +49-89-18 94 741 - 16
fax:   +49-89-18 94 741 - 08

Registergericht: Amtsgericht München
Registernummer: HRB 157899
Vertretungsberechtigte Geschäftsführer: Peter Rasmussen, Thomas Dörfler
Unsere Datenschutzerklärung finden Sie hier:
https://embedded-brains.de/datenschutzerklaerung/


More information about the devel mailing list