Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-15 Thread Janne Blomqvist
On Tue, Mar 14, 2017 at 10:05 AM, Janne Blomqvist wrote: > On Tue, Mar 14, 2017 at 6:32 AM, NightStrike wrote: >> On Mon, Mar 13, 2017 at 5:01 AM, Janne Blomqvist >> wrote: >>> On Sun, Mar 12, 2017 at 10:46 PM, Janne Blomqvist >>> wrote: On Sun, Mar 12, 2017 at 7:26 PM, NightStrike wrote:

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-14 Thread Janne Blomqvist
On Tue, Mar 14, 2017 at 6:32 AM, NightStrike wrote: > On Mon, Mar 13, 2017 at 5:01 AM, Janne Blomqvist > wrote: >> On Sun, Mar 12, 2017 at 10:46 PM, Janne Blomqvist >> wrote: >>> On Sun, Mar 12, 2017 at 7:26 PM, NightStrike wrote: On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist wrot

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-13 Thread NightStrike
On Mon, Mar 13, 2017 at 5:01 AM, Janne Blomqvist wrote: > On Sun, Mar 12, 2017 at 10:46 PM, Janne Blomqvist > wrote: >> On Sun, Mar 12, 2017 at 7:26 PM, NightStrike wrote: >>> On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist >>> wrote: Don't try to use rand_s on CYGWIN CYGWIN see

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-13 Thread Janne Blomqvist
On Sun, Mar 12, 2017 at 10:46 PM, Janne Blomqvist wrote: > On Sun, Mar 12, 2017 at 7:26 PM, NightStrike wrote: >> On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist >> wrote: >>> Don't try to use rand_s on CYGWIN >>> >>> CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is >>> defi

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-12 Thread Janne Blomqvist
On Sun, Mar 12, 2017 at 7:26 PM, NightStrike wrote: > On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist > wrote: >> Don't try to use rand_s on CYGWIN >> >> CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is >> defined even though rand_s is not available. Thus add an extra check >

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-12 Thread NightStrike
On Sun, Mar 12, 2017 at 1:26 PM, NightStrike wrote: > On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist > wrote: >> Don't try to use rand_s on CYGWIN >> >> CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is >> defined even though rand_s is not available. Thus add an extra check >

Re: [Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-03-12 Thread NightStrike
On Mon, Feb 27, 2017 at 6:15 AM, Janne Blomqvist wrote: > Don't try to use rand_s on CYGWIN > > CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is > defined even though rand_s is not available. Thus add an extra check > for __CYGWIN__. > > Thanks to Tim Prince and Nightstrike for

[Patch, libgfortran, committed] Don't use rand_s on CYGWIN

2017-02-27 Thread Janne Blomqvist
Don't try to use rand_s on CYGWIN CYGWIN seems to include _mingw.h and thus __MINGW64_VERSION_MAJOR is defined even though rand_s is not available. Thus add an extra check for __CYGWIN__. Thanks to Tim Prince and Nightstrike for bringing this issue to my attention. Committed as r245755. 2017-02