Nicholas Vinson <nvinson...@gmail.com> writes: > On 12/7/21 15:04, Robbie Harwood wrote: >> Nicholas Vinson <nvinson...@gmail.com> writes: >> >>> Wouldn't it be better to modify configure.ac so the location could >>> be passed in instead of having one-off patches for alternate >>> locations? Something like >>> --with-dejavu-font=/usr/share/fonts/dejavu-sans-fonts? >> >> That would mean that anyone using grub2 that wants the dejavu fonts >> will have to figure out that the option exists and pass it to the >> buildscripts. That's a pretty bad user experience for anyone >> building their own grub. > > On the contrary, it doesn't have to mean that at all. It could mean > that when a path is given, configure would use the provided path; > otherwise, it falls back to its default list to find the font.
But then we still have to keep the default list... and I'd still be here as a distro maintainer wanting my distro's path in the default list. I don't see how this is any better. > Such an approach would mimic configure's current behavior for values > such as bootdir and grubdir. But those are *install* paths, not *detection* paths. We're not installing the font - we're figuring out where it is on the system. >> I get that maintaining a list of where everyone has put their fonts is >> somewhat obnoxious, but in the absence of a standard way of doing it, >> I'm not sure there's anything better. The only other alternative I > > See above and configure.ac's solutions to similar issues for better > approaches. configure.ac is 2189 lines of code. You're going to have to be more specific :) Be well, --Robbie
signature.asc
Description: PGP signature
_______________________________________________ Grub-devel mailing list Grub-devel@gnu.org https://lists.gnu.org/mailman/listinfo/grub-devel