Feedback on clang change (#53194)

Marcus Calhoun-Lopez mcalhoun at macports.org
Wed Feb 1 15:19:53 UTC 2017


It is hard to argue with your position.
For my own personal work, I have no need of my own proposals.
My main motivation has been to support Qt and GNU Octave on the widest possible set of systems (and close a couple of tickets).
I am hoping that the proposals will cause minimal disruption at least.
Thank you for the feedback.

-Marcus

> On Jan 31, 2017, at 1:35 PM, Chris Jones <jonesc at hep.phy.cam.ac.uk> wrote:
> 
> Hi,
> 
> To be blunt, i am not in favour of any of your proposals. I do not think we should have any update that allows builds to default to using macports stdlibc++. Old (ancient in my view) systems where the system c++ runtime is stdlibc++ should be allowed to die the graceful death they deserve. On other systems, when the system clang compiler does not support the features a specific port requires, then those ports can blacklist those compilers such that the macports clang compilers, using libc++, are preferred. Thats it, nothing else.
> 
> Chris
> 
>> On 31 Jan 2017, at 6:48 pm, Marcus Calhoun-Lopez <mcalhoun at macports.org> wrote:
>> 
>> In the recent mailing list discussion on C++11, there were no comments on the specific aspects of the proposal
>> (https://lists.macports.org/pipermail/macports-dev/2017-January/thread.html#35173).
>> There is one aspect in particular that I would like to ask about: the proposed changes to clang (https://trac.macports.org/ticket/53194).
>> Any comments would be greatly appreciated.
>> 
>> There are three ways to allow clang to refer to MacPorts libstdc++.
>> 
>> 1) Create a *default* variant to have -stdlib=libstdc++ refer to MacPorts libstdc++.
>>   This was quite rightly rejected because clang should be “as compatible with the host as possible.”
>>   I only bring it up because the required patch is simpler.
>> 
>> 2) Create a new command switch -stdlib=macports-libstdc++ to refer to MacPorts libstdc++.
>> 
>> 3) Create a new subport clang-libstdcxx for which -stdlib=libstdc++ refers to MacPorts libstdc++.
>>   Again, a reason to do this is because the required patch is simpler.
>> 
>> #2 is already in the development versions of clang
>> (https://github.com/macports/macports-ports/commit/c68d69b898fece7fea96b61ae0dd33650f628289
>> and
>> https://github.com/macports/macports-ports/commit/49bd2ee7c1e54f290cb6fe07a084de8a2abddb66),
>> but for my purposes, the variant would have to be the default.
>> 
>> Patches for all three proposal are attached to the ticket.
>> 
>> Thank you,
>> Marcus
> 



More information about the macports-dev mailing list