[prev in list] [next in list] [prev in thread] [next in thread] 

List:       freebsd-ports
Subject:    Re: [PATCH] unbreak imake build when clang set as base compiler
From:       Oliver Pinter <oliver.pntr () gmail ! com>
Date:       2012-09-26 22:31:13
Message-ID: CAPjTQNGtQ7onod=ojd6mM_nMVaHKWfOTNrPK0jPr4koueuXbWA () mail ! gmail ! com
[Download RAW message or body]

On 9/27/12, Oliver Pinter <oliver.pntr@gmail.com> wrote:
> On 9/26/12, Garrett Cooper <yanegomi@gmail.com> wrote:
>> On Wed, Sep 26, 2012 at 2:18 PM, Oliver Pinter <oliver.pntr@gmail.com>
>> wrote:
>>> On 9/26/12, Garrett Cooper <yanegomi@gmail.com> wrote:
>>>> On Tue, Sep 25, 2012 at 3:37 PM, Oliver Pinter <oliver.pntr@gmail.com>
>>>> wrote:
>>>>> Hi all!
>>>>>
>>>>> This patch fixed the problem, when buildig imake on a machine where
>>>>> clang is the base the compiler (WITH_CLANG_IS_CC).
>>>>
>>>> (Picking a random message to reply to) Why not create PRs and CC the
>>>> relevant parties?
>>>
>>> because I hope, there become this issue a mote attention ;)
>>
>> Or gets lost in someone's mailbox :(...
>>
>>> of course, I create in near future a PR, but before that, I send an
>>> updated patch, and yes, I know, that this is not a clean solution, but
>>> better than nothing.
>>
>> Having a PR is better than a random email that will most likely get
>> lost. I would definitely respond to:
>>
>> "Hey $dev! I filed $PR -- can you please take a look at it? It solves
>> '$reason'. Thanks!"
>
> ports/172103 ports/172104 ports/172102 ports/172101 ports/172100 ;)

and the patches are here too:
http://oliverp.teteny.bme.hu/freebsd/patches/ports-clang/

>
>>
>> You might get better results this way too and less duplicated effort
>> by multiple individuals.. just sayin'...
>>
>> Thanks!
>> -Garrett
>>
>
_______________________________________________
freebsd-ports@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/freebsd-ports
To unsubscribe, send any mail to "freebsd-ports-unsubscribe@freebsd.org"
[prev in list] [next in list] [prev in thread] [next in thread] 

Configure | About | News | Add a list | Sponsored by KoreLogic