Subject: Re: CVS commit: pkgsrc/textproc/nbsed/files
To: None <tech-pkg@NetBSD.org>
From: Johnny C. Lam <jlam@NetBSD.org>
List: tech-pkg
Date: 09/13/2004 00:52:56
On Sun, Sep 12, 2004 at 12:49:33PM -0400, Jan Schaumann wrote:
> "Johnny C. Lam" <jlam@NetBSD.org> wrote:
> > 
> > I wonder why this change was necessary at all.  Within pkgsrc and for
> > the bootstrap process, nbsed is configured with -lnbcompat passed to
> > the compiler/linker, so those function checks for fgetln() and
> > {get,set}progname() should show that they exist (because they exist
> > in libnbcompat.a).  Was this not working on your machine?
> 
> It wasn't working on my IRIX system.
> 
> During configure phase, we compile with -lnbcompat, so configure will
> determine that we have getprogname, setprogname and fgetln and set
> HAVE_GETPROGNAME etc. accordingly.
> 
> During the build phase, we include nbcompat.h, which in turn includes
> nbcompat/stdio.h.  That file includes the lines:
> 
> #if !HAVE_FGETLN
> char    *fgetln(FILE *, size_t *);
> #endif
> 
> Since during configure phase HAVE_FGETLN was set to 1, the function
> protoype is now not defined in our program, causing the build to fail.
> 
> Similarly for nbcompat/stdlib.h and HAVE_SETPROGNAME.

Ah, I understand now.  We should look at changing this post-branch,
since I think we want to have a chance to build those packages without
libnbcompat if possible.

	Cheers,

	-- Johnny Lam <jlam@NetBSD.org>