bug#11829: Build failure with --enable-gcc-warnings

[Pádraig Brady]

On 06/30/2012 11:22 PM, Stefano Lattarini wrote:
> Trying to build the latest coreutils from master:
> 
>   $ make all
>   ...
>     CC       stty.o
>   stty.c: In function 'main':
>   stty.c:740:8: error: variable 'speed_was_set' set but not used 
> [-Werror=unused-but-set-variable]
>   cc1: all warnings being treated as errors
>   make[3]: *** [stty.o] Error 1
> 
>   $ uname -rmos
>   Linux 3.3.1-3.fc16.ppc64 ppc64 GNU/Linux
> 
>   $ gcc --version
>   gcc (GCC) 4.6.3 20120306 (Red Hat 4.6.3-2)
>   Copyright (C) 2011 Free Software Foundation, Inc.
>   This is free software; see the source for copying conditions.  There is NO
>   warranty; not even for MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.
> 
> Let me know if you need more information.

This should address it:

commit 8f4dcad477da1a952a04478ae125d5453f29dcc9
Author: Pádraig Brady <address@hidden>
Date:   Sun Jul 1 00:04:37 2012 +0100

    maint: avoid a -Wunsed-but-set warning on some systems

    * src/stty.c (main): Mark speed_was_set as unused when
    CIBAUD undefined (like on ppc64 GNU/Linux for example).
    Reported-by: Stefano Lattarini

diff --git a/src/stty.c b/src/stty.c
index 83b502c..b2dd849 100644
--- a/src/stty.c
+++ b/src/stty.c
@@ -737,7 +737,11 @@ main (int argc, char **argv)
   int argi = 0;
   int opti = 1;
   bool require_set_attr;
+#ifdef CIBAUD
   bool speed_was_set;
+#else
+  bool speed_was_set ATTRIBUTE_UNUSED;
+#endif
   bool verbose_output;
   bool recoverable_output;
   int k;