[prev in list] [next in list] [prev in thread] [next in thread]
List: cfe-dev
Subject: Re: [cfe-dev] Merging MinGW toolchain patches to 3.7
From: Richard Smith <richard () metafoo ! co ! uk>
Date: 2015-07-23 21:09:41
Message-ID: CAOfiQqk7Sx=fBqVtLrGe+7SNHW2FJnqNo0fqP35amKtW7-o40g () mail ! gmail ! com
[Download RAW message or body]
[Attachment #2 (multipart/alternative)]
On Thu, Jul 23, 2015 at 12:30 PM, Richard Smith <metafoo@gmail.com> wrote:
> Yes, we should get these changes into 3.7. I'll look over the patches
> later today.
>
I'm not very happy about applying patches with no tests to the branch, but,
reluctantly, approved.
Yaron: please add some tests for r242660 and r242667 to trunk. (I also find
it very weird that the MinGW::Arch member has a trailing slash. Is there a
good reason for that?)
> On 23 Jul 2015 9:56 am, "Hans Wennborg" <hans@chromium.org> wrote:
>
>> +cfe-dev to keep it on the list, and Richard
>>
>> This sounds like a reasonable thing to merge.
>>
>> Richard, what's your opinion as an owner?
>>
>>
>> On Thu, Jul 23, 2015 at 9:48 AM, Martell Malone <martellmalone@gmail.com>
>> wrote:
>> > I almost forgot
>> >
>> > Yaron also fixed all the different combination of linux host/distros
>> > targetting mingw with theses 3.
>> > I have a feeling we will have people hitting the mailing list about
>> come the
>> > 3.7 release if their distro isn't working.
>> >
>> > clang
>> >
>> > r242660 Support mingw toolchain include and lib directories on Arch
>> Linux.
>> > r242667 Remove erroneous space in "lib64" string constant.
>> > r242766 Fix mingw toolchain to honor sysroot on Linux and add tests.
>> >
>> > Again as i said I've no idea what the process is or what we can do here.
>> > Hoping you can fill me in on the details
>> >
>> > Kind Regards
>> > Martell
>> >
>> >
>> >
>> > On Thu, Jul 23, 2015 at 5:44 PM, Martell Malone <
>> martellmalone@gmail.com>
>>
>> > wrote:
>> >>
>> >> Hi Hans :)
>> >>
>> >> rnk and I were talking to me about the possibility of getting the mingw
>> >> driver work back ported into the 3.7 branch.
>> >>
>> >> I read the thread where you branched the 3.7 release you said that if
>> >> anyone has a request to have patches merged they should contact you
>> directly
>> >>
>> >> When the branch was made we were in the middle of crushing the final
>> bugs
>> >> in the MINGW Driver.
>> >> I had a total of about 9 patches merged but 2 missed the window
>> >> unfortunately.
>> >> 1 was for compiler-rt and 1 was for clang
>> >>
>> >> clang
>> >> r242905 Add support for -rtlib option and -stdlib option to the mingw
>> >> driver
>> >>
>> >> compiler-rt
>> >> r242539 compiler-rt: add support for mingw-w64 in builtins
>> >> r242540 Add missing chkstk.S files from r242539
>> >>
>> >> (r242540 is here because only half the patch was committed initially)
>> >>
>> >> These 2(now 3) patches complete my series and fixes everything to have
>> a
>> >> clang toolchain with mingw-w64 without needing gcc.
>> >>
>> >> I don't know what the process is for back porting or how you feel about
>> >> these patches but I said I would email you about it an see where we
>> can go
>> >> from here :)
>> >>
>> >> Kind Regards
>> >> Martell
>> >
>> >
>>
>
[Attachment #5 (text/html)]
<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Thu, Jul 23, 2015 \
at 12:30 PM, Richard Smith <span dir="ltr"><<a href="mailto:metafoo@gmail.com" \
target="_blank">metafoo@gmail.com</a>></span> wrote:<br><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><p dir="ltr">Yes, we should get these changes into 3.7. \
I'll look over the patches later today.</p></blockquote><div>I'm not very \
happy about applying patches with no tests to the branch, but, reluctantly, \
approved.</div><div><br></div><div>Yaron: please add some tests for r242660 and \
r242667 to trunk. (I also find it very weird that the MinGW::Arch member has a \
trailing slash. Is there a good reason for that?)<br></div><blockquote \
class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc \
solid;padding-left:1ex"><p dir="ltr">On 23 Jul 2015 9:56 am, "Hans \
Wennborg" <<a href="mailto:hans@chromium.org" \
target="_blank">hans@chromium.org</a>> wrote:</p><div \
class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 \
.8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">+cfe-dev to keep it \
on the list, and Richard<br> <br>
This sounds like a reasonable thing to merge.<br>
<br>
Richard, what's your opinion as an owner?<br>
<br>
<br></span><span class="">
On Thu, Jul 23, 2015 at 9:48 AM, Martell Malone <<a \
href="mailto:martellmalone@gmail.com" target="_blank">martellmalone@gmail.com</a>> \
wrote:<br> > I almost forgot<br>
><br>
> Yaron also fixed all the different combination of linux host/distros<br>
> targetting mingw with theses 3.<br>
> I have a feeling we will have people hitting the mailing list about come the<br>
> 3.7 release if their distro isn't working.<br>
><br>
> clang<br>
><br>
> r242660 Support mingw toolchain include and lib directories on Arch Linux.<br>
> r242667 Remove erroneous space in "lib64" string constant.<br>
> r242766 Fix mingw toolchain to honor sysroot on Linux and add tests.<br>
><br>
> Again as i said I've no idea what the process is or what we can do here.<br>
> Hoping you can fill me in on the details<br>
><br>
> Kind Regards<br>
> Martell<br>
><br>
><br>
><br></span>
> On Thu, Jul 23, 2015 at 5:44 PM, Martell Malone <<a \
href="mailto:martellmalone@gmail.com" \
target="_blank">martellmalone@gmail.com</a>><div><div class="h5"><br> > \
wrote:<br> >><br>
>> Hi Hans :)<br>
>><br>
>> rnk and I were talking to me about the possibility of getting the mingw<br>
>> driver work back ported into the 3.7 branch.<br>
>><br>
>> I read the thread where you branched the 3.7 release you said that if<br>
>> anyone has a request to have patches merged they should contact you \
directly<br> >><br>
>> When the branch was made we were in the middle of crushing the final \
bugs<br> >> in the MINGW Driver.<br>
>> I had a total of about 9 patches merged but 2 missed the window<br>
>> unfortunately.<br>
>> 1 was for compiler-rt and 1 was for clang<br>
>><br>
>> clang<br>
>> r242905 Add support for -rtlib option and -stdlib option to the mingw<br>
>> driver<br>
>><br>
>> compiler-rt<br>
>> r242539 compiler-rt: add support for mingw-w64 in builtins<br>
>> r242540 Add missing chkstk.S files from r242539<br>
>><br>
>> (r242540 is here because only half the patch was committed initially)<br>
>><br>
>> These 2(now 3) patches complete my series and fixes everything to have a<br>
>> clang toolchain with mingw-w64 without needing gcc.<br>
>><br>
>> I don't know what the process is for back porting or how you feel \
about<br> >> these patches but I said I would email you about it an see where \
we can go<br> >> from here :)<br>
>><br>
>> Kind Regards<br>
>> Martell<br>
><br>
><br>
</div></div></blockquote></div>
</blockquote></div><br></div></div>
_______________________________________________
cfe-dev mailing list
cfe-dev@cs.uiuc.edu
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic