GNU bug report logs - #58451
cool-retro-term's fonts are broken

Previous Next

To add a comment to this bug, you must first unarchive it, by sending
a message to control AT debbugs.gnu.org, with unarchive 58451 in the body.
You can then email your comments to 58451 AT debbugs.gnu.org in the normal way.

Toggle the display of automated, internal messages from the tracker.

Message #5 received at submit <at> debbugs.gnu.org (full text, mbox):

From: Christine Lemmer-Webber <cwebber <at> dustycloud.org>
To: bug-guix <at> gnu.org
Subject: cool-retro-term's fonts are broken
Date: Tue, 11 Oct 2022 15:08:29 -0400

I realize we've made some modifications to cool-retro-term's font stuff
back in Bavier's original patches adding cool-retro-term.  Particularly,
some mentions of fonts are emoved.  However, something has gone wrong,
you can no longer increase font sizes, and there aren't really any fonts
listed.  The culprit seems hinted by looking at the output from running
cool-retro-term from another terminal:

qrc:/Fonts.qml:206:1: Expected token `}'
qrc:/Fonts.qml:206:1: Expected token `}'
qrc:/ApplicationSettings.qml:173: TypeError: Cannot read property 'count' of undefined
qrc:/ApplicationSettings.qml:126: TypeError: Cannot read property 'fontlist' of null

I'm guessing the code that's trying to remove mentions of forbidden
fonts is accidentally dropping a closing brace somehow.

Message #10 received at 58451-done <at> debbugs.gnu.org (full text, mbox):

From: Eric Bavier <bavier <at> posteo.net>
To: 58451-done <at> debbugs.gnu.org
Subject: Re: bug#58451: cool-retro-term's fonts are broken
Date: Wed, 12 Oct 2022 15:25:58 +0000

[Message part 1 (text/plain, inline)]

Hi Christine,

Thanks for the report. This should be fixed in https://git.savannah.gnu.org/cgit/guix.git/commit/?id=86ec52f66735b122b9035eba56516fd16f3be958

The "Cannot read property 'fontlist' of null" is still present, but was/is also present in the earlier 1.1.1 release, and seems to be mostly harmless.

Cheers,
`~Eric

On October 11, 2022 7:08:29 PM UTC, Christine Lemmer-Webber <cwebber <at> dustycloud.org> wrote:
>I realize we've made some modifications to cool-retro-term's font stuff
>back in Bavier's original patches adding cool-retro-term.  Particularly,
>some mentions of fonts are emoved.  However, something has gone wrong,
>you can no longer increase font sizes, and there aren't really any fonts
>listed.  The culprit seems hinted by looking at the output from running
>cool-retro-term from another terminal:
>
>qrc:/Fonts.qml:206:1: Expected token `}'
>qrc:/Fonts.qml:206:1: Expected token `}'
>qrc:/ApplicationSettings.qml:173: TypeError: Cannot read property 'count' of undefined
>qrc:/ApplicationSettings.qml:126: TypeError: Cannot read property 'fontlist' of null
>
>I'm guessing the code that's trying to remove mentions of forbidden
>fonts is accidentally dropping a closing brace somehow.
>
>
>
>

-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.
-- 
Sent from my Android device with K-9 Mail. Please excuse my brevity.

[Message part 2 (text/html, inline)]

Previous Next

GNU bug tracking system
Copyright (C) 1999 Darren O. Benham, 1997,2003 nCipher Corporation Ltd, 1994-97 Ian Jackson.