Re: [Bug-indent] Bug#286435: indent: FTBFS (amd64/gcc-4.0): (fwd)

[Eric Deplagne]

  Looks like the right thing to do, if 'output' is really used somewhere else.
  What if you instead remove the declaration of 'output' in indent.h  ?

On lun, 20 déc 2004 12:41:23 +0100, Santiago Vila wrote:
> I received this from the Debian bug system.
> It seems the indent package fails to build from source when using gcc 4.0.
> [ Please keep the Cc: lines when replying. Thanks ].
> 
> ---------- Forwarded message ----------
> From: Andreas Jochens <address@hidden>
> To: Debian Bug Tracking System <address@hidden>
> Date: Mon, 20 Dec 2004 11:10:58 +0100
> Subject: Bug#286435: indent: FTBFS (amd64/gcc-4.0): 
> Resent-Sender: Santiago Vila <address@hidden>
> 
> Package: indent
> Severity: normal
> Tags: patch
> 
> When building 'indent' on amd64 with gcc-4.0,
> I get the following error:
> 
> gcc -DHAVE_CONFIG_H -DLOCALEDIR=\"/usr/share/locale\" -I. -I. -I.. -I. -I. 
> -I../intl    -g -Wall -O2 -c `test -f 'output.c' || echo './'`output.c
> In file included from output.c:22:
> output.h:51: warning: type defaults to 'int' in declaration of 
> 'inhibit_indenting'
> output.c:26: error: static declaration of 'output' follows non-static 
> declaration
> indent.h:100: error: previous declaration of 'output' was here
> 
> With the attached patch 'indent' can be compiled
> on amd64 using gcc-4.0.
> 
> Regards
> Andreas Jochens
> 
> diff -urN ../tmp-orig/indent-2.2.9/src/output.c ./src/output.c
> --- ../tmp-orig/indent-2.2.9/src/output.c     2002-12-12 18:36:49.000000000 
> +0100
> +++ ./src/output.c    2004-12-20 10:30:47.547161480 +0100
> @@ -23,7 +23,7 @@
>  
>  RCSTAG_CC ("$Id: output.c,v 1.5 2002/12/12 17:36:49 david Exp $");
>  
> -static FILE            * output       = NULL;
> +FILE* output = NULL;
>  static BOOLEAN           inhibited    = 0;
>  static buf_break_st_ty * buf_break_list = NULL;

-- 
  Eric Deplagne