GNU bug report logs - #73522
[PATCH 0/6] Package and use RetroArch assets, plus other improvements.

Previous Next

Package: guix-patches;

Reported by: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>

Date: Sat, 28 Sep 2024 01:03:02 UTC

Severity: normal

Tags: patch

Done: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>

Bug is archived. No further changes may be made.

Full log


View this message in rfc822 format

From: help-debbugs <at> gnu.org (GNU bug Tracking System)
To: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Cc: tracker <at> debbugs.gnu.org
Subject: bug#73522: closed ([PATCH 0/6] Package and use RetroArch assets,
 plus other improvements.)
Date: Sun, 06 Oct 2024 16:28:02 +0000
[Message part 1 (text/plain, inline)]
Your message dated Mon, 07 Oct 2024 01:26:29 +0900
with message-id <87bjzxci4q.fsf <at> gmail.com>
and subject line Re: [bug#73522] [PATCH games-team 06/14] gnu: retroarch: Clean-up source from extraneous bundled libraries.
has caused the debbugs.gnu.org bug report #73522,
regarding [PATCH 0/6] Package and use RetroArch assets, plus other improvements.
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs <at> gnu.org.)


-- 
73522: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=73522
GNU Bug Tracking System
Contact help-debbugs <at> gnu.org with problems
[Message part 2 (message/rfc822, inline)]
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: guix-patches <at> gnu.org
Cc: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
Subject: [PATCH 0/6] Package and use RetroArch assets, plus other improvements.
Date: Sat, 28 Sep 2024 09:29:26 +0900
Testing RetroArch, I wondered why the UI was so unappealing, with black boxes
where there should have been icons and pixelated text.  It was difficult to
comprehend the interface.

Researching a bit, I've found the 'assets' are actually needed for the various
menu modes of RetroArch to render correctly; I've thus packaged them, and
devised a way to discover them in a way that will be immune to going stale
(RetroArch always saves a configuration file upon exists, and persists any
settings used -- the environment variable mechanism devised thus had to
override a value found in the ~/.config/retroarch/retroarch.cfg file).


Maxim Cournoyer (6):
  gnu: Add retroarch-assets.
  gnu: retroarch: Use gexps.
  gnu: retroarch: Add missing inputs, upgrade Qt to version 6.
  gnu: retroarch: Clean-up source from extraneous bundled libraries.
  gnu: retroarch: Add search-path support for assets discovery.
  gnu: retroarch: Really disable online fetching of cores and resources.

 gnu/local.mk                                  |   1 +
 gnu/packages/emulators.scm                    | 169 +++++++++++++-----
 .../retroarch-improved-search-paths.patch     | 132 ++++++++++++++
 3 files changed, 260 insertions(+), 42 deletions(-)
 create mode 100644 gnu/packages/patches/retroarch-improved-search-paths.patch


base-commit: a4ea332bc219e14560d3a5daaa658425d898ec37
-- 
2.46.0



[Message part 3 (message/rfc822, inline)]
From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Liliana Marie Prikler <liliana.prikler <at> gmail.com>
Cc: 73522-done <at> debbugs.gnu.org, Sergey Trofimov <sarg <at> sarg.org.ru>,
 宋文武 <iyzsong <at> envs.net>, Adam Faiz <adam.faiz <at> disroot.org>
Subject: Re: [bug#73522] [PATCH games-team 06/14] gnu: retroarch: Clean-up
 source from extraneous bundled libraries.
Date: Mon, 07 Oct 2024 01:26:29 +0900
Hi again,

Maxim Cournoyer <maxim.cournoyer <at> gmail.com> writes:

[...]

>> Can we unvendor spirv-cross?
>
> It could be done, but it'd have to be new development as their build
> system doesn't currently support it.  I'll give it another try right
> now, but if it becomes too much work, I'll drop it.

I've spent the evening on it, but in the end it works!

>>> +                            "discord-rpc"   ;used in
>>> network/discord.h
>> Can we drop this discord thing?
>
> Not easily, no.  I had checked and there's no switch to opt out in the
> build system.

Actually I was wrong, after getting more knowledgeable about its
'quickbuild / qb' build configure script. I could pass --disable-discord
and a bunch of others to remove support provided via bundled libraries.

I've now push the series (see commit 3e9ebe7f28 and previous ones) with
these changes; you are welcome to inspect the result and let me know if
something is amiss.

Thanks for the review.

Happy hacking!

-- 
Maxim


This bug report was last modified 223 days ago.

Previous Next


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