kicad-developers team mailing list archive

[Maciej Sumiński <maciej.suminski@xxxxxxx>]

On 09/16/2017 09:32 AM, jp charras wrote:
> Le 16/09/2017 à 07:28, Russell Oliver a écrit :
>> Hi, 
>>
>> Jean-Pierre: can you send me a link to the files you tested it on? 
>>
>> Also for opening either an Eagle schematic or board file, the intended workflow starts within the
>> KiCad launcher. File > Import Project > Eagle Cad. This launches both Eeschema, Pcbnew for the
>> matching files and then synchronises the timestamps across the files. 
>>
>> Regards
>> Russell
>>  
> 
> I found the issue!
> 
> The crash is due to the fact certainly the schematic Eagle plugin is missing a switch to the "C"
> locale (adding a "LOCALE_IO dummy;" declaration before reading the file), so the floating numbers
> create issues in countries that use the comma as floating number separator (like in France).
> 
> However, when parsing a file, the Eagle plugin should show an error and abort the load process,
> without crashing Eeschema.

I rebased the changes on the current master branch and pushed to my
repository [1]. There are also some modification, that hopefully make
the branch ready to merge:
- code formatting
- added LOCALE_IO object (JP, would you confirm the problem you
experienced is fixed?)
- separated open and import functions to resemble the pcbnew approach
- added exception handler for the import function, so any errors should
result in an error message rather than segfault

Regards,
Orson

1. https://code.launchpad.net/~orsonmmz/kicad/+git/kicad/+ref/eagle-import

Attachment: signature.asc
Description: OpenPGP digital signature