GNU bug report logs -
#70708
[PATCH] gnu: hyperrogue: correct font path
Previous Next
Full log
Message #13 received at 70708-quiet <at> debbugs.gnu.org (full text, mbox):
Hi,
Ah OK, I understand.
In that case would you be up for updating the patch to remove the font
files using a snippet please?
Thanks,
Steve / Futurile
On 20/02/2025 15:39, tadhgmister <at> gmail.com wrote:
> The game source code includes a copy of the dejavusans.ttf
> https://github.com/zenorogue/hyperrogue/blob/master/DejaVuSans.ttf
>
> Since the source is considered the first input to the package the
> function `search-input-file` finds it in the games source code and thus
> the checkout of the code becomes a runtime dependency. This means that
> the music and sounds folders which are copied to the package build must
> be downloaded twice (one in checkout and one in built package) to use
> the game.
>
> If you run `guix graph hyperrogue --type=references -M 1 | grep
> checkout` you will see the runtime dependency on the code checkout and
> if you remove font-dejavu from the dependencies without my patch you
> will see it still renders text fine in game. With my patch it correctly
> uses the font from the dejavu package and no longer depends on the checkout.
>
> If this is intended then at least we should remove the dependency on
> font-dejavu since it is not used at all and reference the music and
> sound from the checkout instead of copying it over and then also needing
> the original.
>
>
> On 2025-02-20 8:03 a.m., Steve George wrote:
>> Hi Tadhg,
>>
>> Your patch in #70708 embeds the font in hyperrogue to be font-dejavu [0]. Can you explain why this is necessary please? I'm not familiar with this game, but I just quickly ran it and the font appeared to be fine in the tutorial.
>>
>> Thanks,
>>
>> Steve / Futurile
>>
>> [0] https://debbugs.gnu.org/cgi/bugreport.cgi?bug=70708
>>
This bug report was last modified 44 days ago.
Previous Next
GNU bug tracking system
Copyright (C) 1999 Darren O. Benham,
1997,2003 nCipher Corporation Ltd,
1994-97 Ian Jackson.