← Back to team overview

kicad-developers team mailing list archive

Re: [PATCH] Genericize Windows import/export

 

I'm pretty sure I've discussed this with Simon before but maybe it was
someone else.  We removed all of the MSVC foo from KiCad years ago
because it just made the code horrid and it was all over the place.
Whenever I see a check for MSVC, this says to me that this code is
specific to MSVC.  Devs who want to use MSVC should keep the MSVC
specific code in their personal repos.  Maybe some time I will be able
to embrace MSVC.  Today is not that day.  We have much bigger issues to
resolve than getting KiCad to build on MSVC.

On 1/16/2016 2:39 PM, Chris Pavlina wrote:
> Hmm. From my perspective, I didn't really think it was a policy 
> violation as such, as that is definitely not MSVC-specific. It's just 
> "portable", which is generally a good thing - if we were ever to change 
> the current policy in the future, the more portable the code is, the 
> easier that will be. And who knows, it's possible we could have to do 
> that in the future, if current toolchains are abandoned or something.
> 
> Personally I have absolutely no problem with making things more portable 
> when there is no added complexity and nothing compiler-specific - I'd 
> probably have committed this if you hadn't replied before I could get 
> around to testing it.
> 
> Fair enuogh about not wanting to directly support MSVC, of course, I 
> don't want that either. :)
> 
> 
> On Sat, Jan 16, 2016 at 02:34:18PM -0500, Wayne Stambaugh wrote:
>> Please do not commit this patch.  The current policy is no MSVC specific
>> code in KiCad and I'm not interested in changing that policy any time soon.
>>
>> Thanks,
>>
>> Wayne
>>
>> On 1/15/2016 11:41 PM, Simon Richter wrote:
>>>
>>> Windows-style dllimport/dllexport should be used whenever targetting
>>> Windows directly, not just for MINGW.
>>>
>>> ---
>>>  include/import_export.h | 2 +-
>>>  1 file changed, 1 insertion(+), 1 deletion(-)
>>>
>>>
>>>
>>> _______________________________________________
>>> 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