Re: pkg/45949: bash build on IRIX fails due to missing -c99 flag in builtins

[Staffan Thomén <duck+important%shangtai.net@localhost>]

The following reply was made to PR pkg/45949; it has been noted by GNATS.

From: Staffan =?iso-8859-1?Q?Thom=E9n?= <duck+important%shangtai.net@localhost>
To: gnats-bugs%NetBSD.org@localhost
Cc: 
Subject: Re: pkg/45949: bash build on IRIX fails due to missing -c99 flag in
 builtins
Date: Wed, 15 Feb 2012 18:38:50 +0200

 On Wed, Feb 15, 2012 at 01:40:06PM +0000, Thomas Klausner wrote:
 > The following reply was made to PR pkg/45949; it has been noted by GNATS.
 > 
 > From: Thomas Klausner <wiz%NetBSD.org@localhost>
 > To: NetBSD bugtracking <gnats-bugs%NetBSD.org@localhost>
 > Cc: 
 > Subject: Re: pkg/45949: bash build on IRIX fails due to missing -c99 flag in
 >  builtins
 > Date: Wed, 15 Feb 2012 14:36:32 +0100
 > 
 >  Forward misdirected mail.
 >  
 >  ----- Forwarded message from Aleksej Saushev <asau%inbox.ru@localhost> -----
 >  
 >  Date: Thu, 09 Feb 2012 08:43:06 +0400
 >  From: Aleksej Saushev <asau%inbox.ru@localhost>
 >  To: pkg-bug-handler%NetBSD.org@localhost
 >  Subject: Re: pkg/45949: bash build on IRIX fails due to missing -c99 flag 
 > in builtins
 >  User-Agent: Gnus/5.13 (Gnus v5.13) Emacs/23.3 (berkeley-unix)
 >  
 >  duck+important%shangtai.net@localhost writes:
 >  
 >  > Make a new patch, like this. It takes care of the missing compiler flag. 
 > There
 >  > is a potential here for other systems to fail because of extraenous 
 > compiler
 >  > flags (I don't know, maybe they actually had a reason for not making it 
 > use
 >  > CFLAGS?)
 >  
 >  Yes, they do make it right, CFLAGS_FOR_BUILD and CFLAGS are flags for
 >  different compilers. Could you send values for these variables without
 >  your patch?
 
 CFLAGS = -c99 -n32 -I/usr/include -I/usr/pkg/include
 CFLAGS_FOR_BUILD = -g 
 
 >  > I tested this and it works on IRIX, NetBSD 5.1/alpha, NetBSD 4.0/i386
 >  
 >  It works because you're not cross-compiling.
 
 Does anyone in pkgsrc? I guess the other way to deal with it would be to fill
 out the value from the pkgsrc Makefile, 
 
 .if !empty(OPSYS:MIRIX)
 CONFIGURE_ENV+=CFLAGS_FOR_BUILD="$(CFLAGS)"
 .endif
 
 or perhaps
 
 CFLAGS_FOR_BUILD?=$(CFLAGS)
 CONFIGURE_ENV+=CFLAGS_FOR_BUILD="$(CFLAGS_FOR_BUILD)"
 
 Is this better?
 
 -- 
 Staffan Thomén - ADB3 455F 10D5 86D1 78D6  048D 11BB D66E 7C7E 2EF8