GNU bug report logs - #70708
[PATCH] gnu: hyperrogue: correct font path

Previous Next

Package: guix-patches;

Reported by: Tadhg McDonald-Jensen <tadhgmister <at> gmail.com>

Date: Thu, 2 May 2024 04:41:01 UTC

Severity: normal

Tags: patch

Full log


Message #12 received at 70708-quiet <at> debbugs.gnu.org (full text, mbox):

From: tadhgmister <at> gmail.com
To: Steve George <steve <at> futurile.net>, 70708-quiet <at> debbugs.gnu.org
Subject: Re: bug#70708: hyperrogue: correct font path
Date: Thu, 20 Feb 2025 10:39:38 -0500
[Message part 1 (text/plain, inline)]
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
> 
> 
[OpenPGP_0x85C0C8DCBBF44EE7.asc (application/pgp-keys, attachment)]
[OpenPGP_signature.asc (application/pgp-signature, attachment)]

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.