Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-13 Thread Kai Tietz
2011/10/13 Paolo Carlini : >> >> Ping, did this go in trunk already? > > I would be surprised to see this happening if nobody like you or Kai actually > does the commit ;) > > P I will take care to apply it. Kai

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-13 Thread NightStrike
On Thu, Oct 13, 2011 at 9:47 AM, Paolo Carlini wrote: >> >> Ping, did this go in trunk already? > > I would be surprised to see this happening if nobody like you or Kai actually > does the commit ;) > > P > Does Jon have commit access?

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-13 Thread Paolo Carlini
> > Ping, did this go in trunk already? I would be surprised to see this happening if nobody like you or Kai actually does the commit ;) P

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-13 Thread JonY
On 10/8/2011 23:50, Kai Tietz wrote: > 2011/10/8 Paolo Carlini: >> Hi, >> >>> Ok, fixed it, I made a very dumb mistake in configure.host, new patch >>> attached. >> >> Patch is still ok with me, if Kai is ok with it (remember for next time: >> regenerated files are not posted, are just a distracti

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-08 Thread Kai Tietz
2011/10/8 Paolo Carlini : > Hi, > >> Ok, fixed it, I made a very dumb mistake in configure.host, new patch >> attached. > > Patch is still ok with me, if Kai is ok with it (remember for next time: > regenerated files are not posted, are just a distraction) > > Paolo Ok, by me, too. Thanks, Kai

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-08 Thread Paolo Carlini
Hi, > Ok, fixed it, I made a very dumb mistake in configure.host, new patch > attached. Patch is still ok with me, if Kai is ok with it (remember for next time: regenerated files are not posted, are just a distraction) Paolo >

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-08 Thread JonY
Ok, fixed it, I made a very dumb mistake in configure.host, new patch attached. Changelog: 2011-10-08 Jonathan Yong * configure.host: Use config/os/mingw32-w64 instead of config/os/mingw32 if vendor key is "w64". * config/os/mingw32-w64: Duplicate from config/os/mingw32

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-06 Thread JonY
On 10/1/2011 22:31, JonY wrote: > On 10/1/2011 19:16, Paolo Carlini wrote: >> Hi, >> >>> Thanks, but I am having problems sending a proper diff with the >>> regenerated files, they have a lot of unrelated, even if I made sure I >>> am using autoconf 2.64 and automake 1.11.1. >> >> To be clear, rege

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Kai Tietz
2011/10/1 Pedro Alves : > On Saturday 01 October 2011 12:15:42, JonY wrote: >> On 10/1/2011 18:33, Pedro Alves wrote: >> > On Saturday 01 October 2011 07:03:35, JonY wrote: >> >> Hi, >> >> >> >> I followed Paolo's suggestion with the os_defines.h trick. I duplicated >> >> os/mingw32/ to os/mingw32-

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread JonY
On 10/1/2011 19:16, Paolo Carlini wrote: > Hi, > >> Thanks, but I am having problems sending a proper diff with the >> regenerated files, they have a lot of unrelated, even if I made sure I >> am using autoconf 2.64 and automake 1.11.1. > > To be clear, regenerated files should **not** be part of

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Pedro Alves
On Saturday 01 October 2011 12:15:42, JonY wrote: > On 10/1/2011 18:33, Pedro Alves wrote: > > On Saturday 01 October 2011 07:03:35, JonY wrote: > >> Hi, > >> > >> I followed Paolo's suggestion with the os_defines.h trick. I duplicated > >> os/mingw32/ to os/mingw32-w64/ for this to work, since the

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Paolo Carlini
Hi, > Thanks, but I am having problems sending a proper diff with the > regenerated files, they have a lot of unrelated, even if I made sure I > am using autoconf 2.64 and automake 1.11.1. To be clear, regenerated files should **not** be part of the patch submitted for review, but should definit

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread JonY
On 10/1/2011 18:33, Pedro Alves wrote: > On Saturday 01 October 2011 07:03:35, JonY wrote: >> Hi, >> >> I followed Paolo's suggestion with the os_defines.h trick. I duplicated >> os/mingw32/ to os/mingw32-w64/ for this to work, since there aren't any >> built-in defines to tell the 2 apart unless y

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread JonY
On 10/1/2011 18:08, Paolo Carlini wrote: > On 10/01/2011 11:48 AM, JonY wrote: >> On 10/1/2011 17:15, Paolo Carlini wrote: >>> On 10/01/2011 08:03 AM, JonY wrote: Hi, I followed Paolo's suggestion with the os_defines.h trick. I duplicated os/mingw32/ to os/mingw32-w64/ for this

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Pedro Alves
On Saturday 01 October 2011 07:03:35, JonY wrote: > Hi, > > I followed Paolo's suggestion with the os_defines.h trick. I duplicated > os/mingw32/ to os/mingw32-w64/ for this to work, since there aren't any > built-in defines to tell the 2 apart unless you include some headers > like _mingw.h. Are

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Paolo Carlini
On 10/01/2011 11:48 AM, JonY wrote: On 10/1/2011 17:15, Paolo Carlini wrote: On 10/01/2011 08:03 AM, JonY wrote: Hi, I followed Paolo's suggestion with the os_defines.h trick. I duplicated os/mingw32/ to os/mingw32-w64/ for this to work, since there aren't any built-in defines to tell the 2 ap

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread JonY
On 10/1/2011 17:15, Paolo Carlini wrote: > On 10/01/2011 08:03 AM, JonY wrote: >> Hi, >> >> I followed Paolo's suggestion with the os_defines.h trick. I duplicated >> os/mingw32/ to os/mingw32-w64/ for this to work, since there aren't any >> built-in defines to tell the 2 apart unless you include s

Re: [patch] --enable-dynamic-string default for mingw-w64 v2

2011-10-01 Thread Paolo Carlini
On 10/01/2011 08:03 AM, JonY wrote: Hi, I followed Paolo's suggestion with the os_defines.h trick. I duplicated os/mingw32/ to os/mingw32-w64/ for this to work, since there aren't any built-in defines to tell the 2 apart unless you include some headers like _mingw.h. Patch attached, comments?

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread Charles Wilson
On 9/20/2011 10:12 AM, Kai Tietz wrote: > 2011/9/20 Charles Wilson : http://cygwin.com/acronyms/index.html#PCYMTNQREAIYR >> So, this would be a change in current mingw.org behavior. I *was* >> under the impression that this workaround for the old "can't pass >> empty strings across DLL boundary"

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread Pedro Alves
On Tuesday 20 September 2011 15:12:30, Kai Tietz wrote: > Yes. If you read last comment of the thread you are citing, then you > would notice that for static-libstdc++ version the issue is still > present. So to allow users to use also static-libstdc++ variant, this > option is still necessary.

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread JonY
On 9/20/2011 22:12, Kai Tietz wrote: >> I'm not really opposed to making this change for i*86-pc-mingw -- and >> now's the time to do it, as the recently released 4.6.1 mingw.org gcc >> broke the C++ abi anyway, thanks to thiscall. > > Here I am a bit curious? How is 4.6.1 affected by new thiscal

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread Kai Tietz
2011/9/20 Charles Wilson : > On 9/20/2011 9:20 AM, JonY wrote: >> On 9/20/2011 13:59, Kai Tietz wrote: >>> 2011/9/20 JonY: Its been used in the automated toolchain builds for sometime, seems like a good idea to enable it by default. It can be easily changed to match for all mingw as

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread xunxun
于 2011/9/20 21:52, Charles Wilson 写道: On 9/20/2011 9:20 AM, JonY wrote: On 9/20/2011 13:59, Kai Tietz wrote: 2011/9/20 JonY: Its been used in the automated toolchain builds for sometime, seems like a good idea to enable it by default. It can be easily changed to match for all mingw as well if

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread Charles Wilson
On 9/20/2011 9:20 AM, JonY wrote: > On 9/20/2011 13:59, Kai Tietz wrote: >> 2011/9/20 JonY: >>> Its been used in the automated toolchain builds for sometime, >>> seems like a good idea to enable it by default. It can be >>> easily changed to match for all mingw as well if needed. >> >> This patch

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-20 Thread JonY
On 9/20/2011 13:59, Kai Tietz wrote: > 2011/9/20 JonY: >> Hi, >> >> Its been used in the automated toolchain builds for sometime, seems like >> a good idea to enable it by default. It can be easily changed to match >> for all mingw as well if needed. >> >> OK for trunk? >> >> Index: libstdc++-v3/co

Re: [patch] --enable-dynamic-string default for mingw-w64

2011-09-19 Thread Kai Tietz
2011/9/20 JonY : > Hi, > > Its been used in the automated toolchain builds for sometime, seems like > a good idea to enable it by default. It can be easily changed to match > for all mingw as well if needed. > > OK for trunk? > > Index: libstdc++-v3/configure.ac > ==