Subject: Re: CVS commit: pkgsrc
To: Johnny C. Lam <jlam@NetBSD.org>
From: Krister Walfridsson <cato@df.lth.se>
List: pkgsrc-changes
Date: 07/18/2005 17:05:03
On Mon, 18 Jul 2005, Johnny C. Lam wrote:

> Module Name:	pkgsrc
> Committed By:	jlam
> Date:		Mon Jul 18 02:33:27 UTC 2005
>
> Modified Files:
> 	pkgsrc/games/falcons-eye: Makefile
> 	pkgsrc/mk/tools: replace.mk
> 	pkgsrc/x11/qt3-libs: Makefile.common
>
> Log Message:
> Export a new variable TOOLS_CMDLINE_<TOOL> that holds the full command
> line (path and arguments) needed to run the real tool.
>
> Modify TOOLS_<TOOL> to hold only the path to the real tool.
>
> Modify falcons-eye/Makefile and qt3-libs/Makefile.common to use
> TOOLS_CMDLINE_YACC instead of TOOLS_YACC to that they'll use "bison -y".

The removal of arguments breaks some packages (e.g. net/libIDL) in
a way that cannot be fixed in a sane way.

The pkgsrc infrastructure does now add YACC=/usr/pkg/bin/bison to
the CONFIGURE_ENV, and that cannot be overridden by
    CONFIGURE_ENV+=         YACC=${TOOLS_CMDLINE_YACC:Q}
in the package's Makefile.

I think that the infrastructure either should add
    YACC=${TOOLS_CMDLINE_YACC:Q}
or not define YACC at all...

    /Krister