Re: CVS commit: pkgsrc/mk/tools

[Thomas Klausner <wiz%NetBSD.org@localhost>]

On Sun, Jul 24, 2022 at 02:56:44PM +0200, Tobias Nygren wrote:
> On Sun, 24 Jul 2022 14:48:22 +0200
> Tobias Nygren <tnn%NetBSD.org@localhost> wrote:
> 
> > > Log Message:
> > > mk: if pkg-config is not in USE_TOOLS, create a false(1) wrapper for it
> > 
> > Hi,
> > 
> > I think there might be a problem with this change.
> > Many packages now report this message during configure:
> > 
> > checking pkg-config is at least version 0.9.0... /tmp/pkgsrc-obj/fonts/encodings
> > /work/.tools/bin/pkg-config: Cannot vfork (Resource temporarily unavailable)
> > no
> > 
> > Also the configure test for pkg-config takes a very long time.
> > Looking at .work.log it seems the wrapper script keep re-forking
> > itself until it runs out of processes and then fails.
> 
> Should it be TOOLS_CMD.false instead of TOOLS_CMD.pkg-config?

I've removed the TOOLS_CMD.pkg-config override, the default is
false(1).  fonts/encodings now gives a couple weird warnings, but no
vfork error.

nia, can you please explain why you override TOOLS_CMD.itstool in
itstool.mk instead of using the default false(1)?

Thanks,
 Thomas