Subject: Re: pkg/36929 (net/tacacs: fails to build with gcc4)
To: None <obache@NetBSD.org, gnats-admin@netbsd.org,>
From: Yakovetsky Vladimir <yx@x.ua>
List: pkgsrc-bugs
Date: 09/23/2007 16:15:10
The following reply was made to PR pkg/36929; it has been noted by GNATS.
From: Yakovetsky Vladimir <yx@x.ua>
To: gnats-bugs@NetBSD.org
Cc: obache@NetBSD.org, pkgsrc-bugs@NetBSD.org, gnats-admin@NetBSD.org
Subject: Re: pkg/36929 (net/tacacs: fails to build with gcc4)
Date: Sun, 23 Sep 2007 19:14:22 +0300
On Sun, Sep 23, 2007 at 08:48:44AM +0000, obache@NetBSD.org wrote:
> Synopsis: net/tacacs: fails to build with gcc4
>
> State-Changed-From-To: open->feedback
> State-Changed-By: obache@netbsd.org
> State-Changed-When: Sun, 23 Sep 2007 08:48:42 +0000
> State-Changed-Why:
> Simply removing the 'extern' line seems OK.
> Why do you want to keep the line if using gcc<4.0 ?
>
As forward declarations to static variables are valid gcc<4
that we can keep this line to not change an author's original code
(and a case for gcc4 having preprocessing).
p.s. and really, as in the pkgsrc version of tacacs
forward declaration to nopasswd_str is not used
is possible (I am agree with you) to clean simply this line.
--
Yakovetsky Vladimir