Re: CVS commit: pkgsrc/meta-pkgs/bulk-large

["David H. Gutteridge" <david%gutteridge.ca@localhost>]

On 2023-09-17 18:34, Greg Troxel wrote:
> "David H. Gutteridge" <gutteridge%netbsd.org@localhost> writes:
>
> > +.include "../../graphics/librsvg/available.mk"
> > .if ${LIBRSVG_TYPE} == "rust"
> > DEPENDS+=    librsvg-[0-9]*:../../graphics/librsvg
> > .else
>
> Totally reasonable, but it does make me wonder if this availble/include
> switch for things that depend on librsvg is the right thing, vs
> including some graphics/librsvg/librsvg.mk

Yes, it would probably make sense to consolidate that. (I don't know
the exact details of the choices that were made.)

(A different, slightly related topic is naming inconsistencies. E.g.,
various REPLACE_ variables for different languages are located in quite
varied paths and files, e.g., REPLACE_R is found in
mk/configure/replace-interpreter.mk, vs. REPLACE_PYTHON being placed in
lang/python/application.mk, vs. REPLACE_PHP in lang/php/replace.mk,
etc.)

Dave