Re: CVS commit: pkgsrc/fonts/dejavu-ttf

[David Brownlee <abs%absd.org@localhost>]

On Sun, 25 Jul 2021 at 19:51, Greg Troxel <gdt%lexort.com@localhost> wrote:
> Hauke Fath <hf%spg.tu-darmstadt.de@localhost> writes:
>
> > On Sun, 25 Jul 2021 14:22:29 -0400, Greg Troxel wrote:
> >>> Essentially, just like with switching X11 from native to modular, you'd
> >>> probably have to rebuild all when you switch from
> >>> PKG_DEFAULT_OPTIONS=-x11 to a setup based on X11.
> >>
> >> Yes, but I thought we were talking about making it optional to skip
> >> things all the time, rather than adding an x11 option to every font
> >> package.
> >
> > You mean, like a global switch that optionally disables registering
> > installed fonts?
>
> No, I thought you were proposing (or maybe it was abs@) changing the
> install logic to "run mkfontdir if it is isstalled, otherwise just
> dont'".

That would be me (with a little nuance). I much prefer a Just Works
option to additional option flags, providing it doesn't add too much
complexity.

So that would be:
- Change the all font packages's mkfontdir logic to remove dependency
and add a per package file with a list of font dirs used by the
package, plus run mkfontdir iff present on install/uninstall
- Adjust fonts/mkfontscale to run mkfontdir using the font dirs files
on install & cleanup on uninstall

So the cases would be:

1) X11_TYPE:native and X sets installed, fonts installed - behaviour unchanged
2) X11_TYPE:native no X sets installed, fonts installed - no longer
errors,no font indexes. Apps that just want fonts should be happy
3) X11_TYPE:native no X sets installed, then fonts installed, then X
sets extracted - font indexes missing - "Don't do that"

4) X11_TYPE:modular and mkfontscale installed before fonts - behaviour unchanged
5) X11_TYPE:modular and fonts installed without mkfontscale - no font indexes
6) X11_TYPE:modular and fonts installed then mkfontscale - all font
indexes generated when mkfontscale installed

Providing 4 & 6 can give the same font index data, we have no
regressions and have gained 2) and 5) as valid states