Re: Updated [test]: coreutils-8.24-2

2015-08-27 Thread Sam Edge
On 27/08/2015 21:33, Helmut Karlowski wrote: > Did you mean: "The compiler-warning would inform you that the value is > not set in a code-path"? I'd agree if I was always in the position of starting projects from scratch. All compiler warnings should be errors and disabling a compiler warning has

Re: Updated [test]: coreutils-8.24-2

2015-08-27 Thread Helmut Karlowski
Am 27.08.2015, 21:26 Uhr, schrieb Peter Rosin: Seems like a fast way to not get a compiler warning if you do happen to use a variable that would be used uninitialized if it were not for an ugly pointless initializer. Sorry if I added the wrong number of negations in there, but you get my poin

Re: Updated [test]: coreutils-8.24-2

2015-08-27 Thread Peter Rosin
On 2015-08-27 21:47, Sam Edge wrote: > One might add, "Always, always initialize automatic variables. This > ensures deterministic behaviour. The compiler will optimise out the > redundant ones." Seems like a fast way to not get a compiler warning if you do happen to use a variable that would be u

Re: Updated [test]: coreutils-8.24-2

2015-08-27 Thread Sam Edge
On 27/08/2015 00:28, Eric Blake wrote: > On 08/26/2015 04:50 PM, Fergus wrote: >>> .. No, you're the first to report it (so 8.24-2 has the same issue), but >>> now that I know about it, it will get fixed soon. > Spot the bugs: > > int > cygwin_spelling (char const *path) > { > ... > int len; > ..

Re: Updated [test]: coreutils-8.24-2

2015-08-26 Thread Eric Blake
On 08/26/2015 04:50 PM, Fergus wrote: >> .. No, you're the first to report it (so 8.24-2 has the same issue), but >> now that I know about it, it will get fixed soon. Spot the bugs: int cygwin_spelling (char const *path) { ... int len; ... if (! path || ! *path || len > PATH_MAX) /* PATH

Re: Updated [test]: coreutils-8.24-2

2015-08-26 Thread Fergus
> .. No, you're the first to report it (so 8.24-2 has the same issue), but > now that I know about it, it will get fixed soon. Fantastic, thank you. Fergus -- Problem reports: http://cygwin.com/problems.html FAQ: http://cygwin.com/faq/ Documentation: http://cygwin

Re: Updated [test]: coreutils-8.24-2

2015-08-26 Thread Eric Blake
On 08/26/2015 04:16 PM, Fergus wrote: > Yesterday's update from coreutils-8.23-4 to 8.24-1 has done something > horrible to cp. Previously > cp -r {sourcepath}/subdirectory {targetpath} > is allowed even when {targetpath}/subdirectory already exists - which, very > often or even usually, it does.

Re: Updated [test]: coreutils-8.24-2

2015-08-26 Thread Fergus
Yesterday's update from coreutils-8.23-4 to 8.24-1 has done something horrible to cp. Previously cp -r {sourcepath}/subdirectory {targetpath} is allowed even when {targetpath}/subdirectory already exists - which, very often or even usually, it does. (A very common context is where a Cygwin user mai

[ANNOUNCEMENT] Updated [test]: coreutils-8.24-2

2015-08-26 Thread Eric Blake (cygwin)
A new test release of coreutils, 8.24-2, has been uploaded, and will be available soon from your favorite mirror. This leaves 8.24-1 as current, and 8.23-4 as the previous version. NEWS: = This is a rebuild that enables the 'stdbuf' utility (Thanks to Yaakov for helping with patches to hack i