← Back to team overview

kicad-developers team mailing list archive

Re: OpenMP patch

 

Hi Wayne

I hear that there is still an issue for OSX+homebrew users. We are
missing the patch from Cirilio,
https://lists.launchpad.net/kicad-developers/msg21790.html

Except he made a typo. Is should be GLM_INCLUDE_DIR insted of GLEW_INCLUDE_DIR.

2015-12-10 17:59 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
> On 12/10/2015 8:45 AM, Maciej Sumiński wrote:
>> OpenMP is optional, it boosts the ratsnest calculation in GAL canvases.
>> It was in the section specific to both gcc and clang, what are other
>> compilers that we support?
>
> Technically we don't but hiding FindOpenMP in the gcc and clang compiler
> config probably isn't the best choice.  OpenMP is likely to be supported
> on other compilers.
>
> I removed the offending line that inadvertently made OpenMP a
> requirement so everything should be fine.
>
>>
>> Anyway, if it works for other compilers, I do not see a reason that
>> prevents enabling it.
>>
>> Regards,
>> Orson
>>
>> On 12/10/2015 02:30 PM, Wayne Stambaugh wrote:
>>> He didn't add it, he just moved it out of the GCC specific
>>> configuration.  I'm not sure why openmp would only be required when
>>> using gcc and not other compilers (which technically are not supported).
>>>  Do we even need openmp?  Where is used in KiCad?  If we do need it, is
>>> it only required when building kicad with gcc?
>>>
>>> On 12/10/2015 8:26 AM, Nick Østergaard wrote:
>>>> It was added in the patch by the other Simon.
>>>>
>>>> https://github.com/KiCad/kicad-source-mirror/commit/e87258510567cb95800178dec30b058c885c326c#diff-af3b638bc2a3e6c650974192a53c7291
>>>>
>>>> 2015-12-10 14:22 GMT+01:00 Wayne Stambaugh <stambaughw@xxxxxxxxx>:
>>>>> I'm not sure why openmp would be a dependency that is compiler
>>>>> dependent.  I didn't add the openmp dependency so I cannot comment on
>>>>> why this would be specific to gcc.  Anyone else know why openmp was gcc
>>>>> specific?
>>>>>
>>>>> On 12/9/2015 7:15 PM, Simon Wells wrote:
>>>>>> In BZR-6363 a patch was made so that OpenMP worked on non-gcc compilers
>>>>>>
>>>>>>
>>>>>> check_find_package_result( OPENMP_FOUND "OpenMP" )
>>>>>>
>>>>>> Was also added, This appears to make OpenMP a dependency for building kicad?
>>>>>>
>>>>>> Is this meant to be the case or was this line added in error
>>>>>>
>>>>>> thanks
>>>>>>
>>>>>> Simon
>>>>>>
>>>>>>
>>>>>> _______________________________________________
>>>>>> 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
>>>
>>
>>


Follow ups

References