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>