Re: [update] fonts/nerd-fonts

Peter Hessler wrote (2023-08-29 08:31 CEST):
> On 2023 Aug 29 (Tue) at 08:22:20 +0200 (+0200), Stefan Hagen wrote:
> :Peter Hessler wrote (2023-08-29 08:08 CEST):
> :> I just noticed that the distfiles for these fonts isn't versioned, which
> :> is creating problems for the bulk build systems.
> :>
> :> Here's a simple patch that'll create ugly versioned filenames. (I
> :> wasn't able to easily trick it into FONT-VER-SUFFIX order.)
> :>
> :> OK?
> :
> :Is there a benefit versioning the files instead of DIST_SUBDIR?
> :
>
> Yes, when you update the fonts then the distfile has a unique filename,
> so it won't conflict with an older version of the font.
>
> In my mind, DIST_SUBDIR is more for organization, rather than
> versioning.

DIST_SUBDIR=nerd-fonts-${VERSION} has the same effect, doesn't it?
An older/newer version would never end up in the same directory.

I see it used that way a couple of times:
/usr/ports/fonts/ibm-plex/Makefile:DIST_SUBDIR = ibm-plex-$V
/usr/ports/fonts/inconsolata-new/Makefile:DIST_SUBDIR = inconsolata-$V
/usr/ports/fonts/literata/Makefile:DIST_SUBDIR = literata-$V
/usr/ports/fonts/noto/cjk/Makefile:DIST_SUBDIR = noto-cjk-$V
/usr/ports/fonts/pkfonts/Makefile:DIST_SUBDIR= pkfonts-${VERSION}
/usr/ports/fonts/roboto-fonts/Makefile:DIST_SUBDIR = roboto-$V
/usr/ports/fonts/intel-one-mono/Makefile:DIST_SUBDIR = intel-one-mono-$V

For me, both solutions are fine. Please go ahead and commit yours.

> :Either way, OK sdk@
> :
>
> --
> Why isn't there a special name for the tops of your feet?
> -- Lily Tomlin
>