GNU bug report logs - #74485
[PATCH 0/1] gnu: emulators: Add libretro-beetle-gba.

Previous Next

Package: guix-patches;

Reported by: Noé Lopez <noe <at> xn--no-cja.eu>

Date: Fri, 22 Nov 2024 21:15:02 UTC

Severity: normal

Tags: patch

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

Bug is archived. No further changes may be made.

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

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

View this report as an mbox folder, status mbox, maintainer mbox


Report forwarded to adam.faiz <at> disroot.org, liliana.prikler <at> gmail.com, iyzsong <at> envs.net, guix-patches <at> gnu.org:
bug#74485; Package guix-patches. (Fri, 22 Nov 2024 21:15:02 GMT) Full text and rfc822 format available.

Acknowledgement sent to Noé Lopez <noe <at> xn--no-cja.eu>:
New bug report received and forwarded. Copy sent to adam.faiz <at> disroot.org, liliana.prikler <at> gmail.com, iyzsong <at> envs.net, guix-patches <at> gnu.org. (Fri, 22 Nov 2024 21:15:02 GMT) Full text and rfc822 format available.

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

From: Noé Lopez <noe <at> xn--no-cja.eu>
To: guix-patches <at> gnu.org
Cc: Noé Lopez <noelopez <at> free.fr>
Subject: [PATCH 0/1] gnu: emulators: Add libretro-beetle-gba.
Date: Fri, 22 Nov 2024 22:15:29 +0100
From: Noé Lopez <noelopez <at> free.fr>

Hi,

Here is a patch to add Beetle GBA’s libretro port, to use with RetroArch.

You can test it with “./pre-inst-env guix shell libretro-beetle-gba retroarch -- retroarch“.  It will appear in the cores list and can be used to play GBA games.

I checked that BSD code was not included like in the original Beetle GBA from
Mednafen (the fork only contains a subset of it).

Have a great day,
Noé Lopez

Noé Lopez (1):
  gnu: emulators: Add libretro-beetle-gba.

 gnu/packages/emulators.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)


base-commit: 1affd2b5aa7f5467a44cf757c4fc0c6956d3f3c9
-- 
2.46.0





Information forwarded to adam.faiz <at> disroot.org, liliana.prikler <at> gmail.com, iyzsong <at> envs.net, guix-patches <at> gnu.org:
bug#74485; Package guix-patches. (Fri, 22 Nov 2024 21:21:02 GMT) Full text and rfc822 format available.

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

From: Noé Lopez <noe <at> xn--no-cja.eu>
To: 74485 <at> debbugs.gnu.org
Cc: Noé Lopez <noe <at> xn--no-cja.eu>
Subject: [PATCH 1/1] gnu: emulators: Add libretro-beetle-gba.
Date: Fri, 22 Nov 2024 22:19:13 +0100
From: Noé Lopez <noelopez <at> free.fr>

* gnu/packages/emulators.scm (libretro-beetle-gba): New variable.

Change-Id: I3f58972ae9065e7e59b2d9fd5b31d528f7690da2
---
 gnu/packages/emulators.scm | 37 +++++++++++++++++++++++++++++++++++++
 1 file changed, 37 insertions(+)

diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
index 948e588c4c..4d757048a4 100644
--- a/gnu/packages/emulators.scm
+++ b/gnu/packages/emulators.scm
@@ -743,6 +743,43 @@ (define-public mednafen
     ;; Main license is GPL2+.  Some parts are BSD-3.
     (license (list license:gpl2+ license:bsd-3))))
 
+(define-public libretro-beetle-gba
+  ;; There are no releases.  Use the latest commit.
+  (let ((commit "6cee80685f735ea6c2373db2622a1f1ee9f39d39")
+        (revision "0"))
+    (package
+      (name "libretro-beetle-gba")
+      ;; Use Mednafen core version as base.  Defined in libretro.cpp:73.
+      (version (git-version "0.9.36" revision commit))
+      (source
+       (origin
+         (method git-fetch)
+         (uri (git-reference
+               (url "https://github.com/libretro/beetle-gba-libretro")
+               (commit commit)))
+         (file-name (git-file-name name version))
+         (sha256
+          (base32 "14fm2g3hrsvvd57d6m9apzc30ypa4m0m5hk2viq422fm2l9y0xbb"))))
+      (build-system gnu-build-system)
+      (arguments
+       (list
+        #:make-flags #~(list "CC=gcc"
+                             (string-append "GIT_VERSION="
+                                            #$commit)
+                             (string-append "DESTDIR="
+                                            #$output) "prefix=/")
+        ;; No tests.
+        #:tests? #f
+        #:phases #~(modify-phases %standard-phases
+                     (delete 'configure))))
+      (home-page "https://github.com/libretro/beetle-gba-libretro")
+      (synopsis "Standalone port of Mednafen GBA to libretro")
+      (description
+       "A standalone port of Mednafen’s GameBoy Advance emulator called
+Beetle GBA to libretro.  Beetle GBA is based on VBA-M, itself a fork of Visual Boy
+Advance.  This is an old fork, and is not recommended over newer alternatives.")
+      (license license:gpl2+))))
+
 (define-public mgba
   (package
     (name "mgba")
-- 
2.46.0





Reply sent to Maxim Cournoyer <maxim.cournoyer <at> gmail.com>:
You have taken responsibility. (Thu, 19 Dec 2024 06:42:02 GMT) Full text and rfc822 format available.

Notification sent to Noé Lopez <noe <at> xn--no-cja.eu>:
bug acknowledged by developer. (Thu, 19 Dec 2024 06:42:02 GMT) Full text and rfc822 format available.

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

From: Maxim Cournoyer <maxim.cournoyer <at> gmail.com>
To: Noé Lopez <noe <at> xn--no-cja.eu>
Cc: 宋文武 <iyzsong <at> envs.net>,
 Liliana Marie Prikler <liliana.prikler <at> gmail.com>,
 Adam Faiz <adam.faiz <at> disroot.org>, 74485-done <at> debbugs.gnu.org
Subject: Re: [bug#74485] [PATCH 1/1] gnu: emulators: Add libretro-beetle-gba.
Date: Thu, 19 Dec 2024 15:40:29 +0900
Hi Noé,

Noé Lopez <noe <at> noé.eu> writes:

> From: Noé Lopez <noelopez <at> free.fr>
>
> * gnu/packages/emulators.scm (libretro-beetle-gba): New variable.

Cool!

> Change-Id: I3f58972ae9065e7e59b2d9fd5b31d528f7690da2
> ---
>  gnu/packages/emulators.scm | 37 +++++++++++++++++++++++++++++++++++++
>  1 file changed, 37 insertions(+)
>
> diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
> index 948e588c4c..4d757048a4 100644
> --- a/gnu/packages/emulators.scm
> +++ b/gnu/packages/emulators.scm
> @@ -743,6 +743,43 @@ (define-public mednafen
>      ;; Main license is GPL2+.  Some parts are BSD-3.
>      (license (list license:gpl2+ license:bsd-3))))
>  
> +(define-public libretro-beetle-gba
> +  ;; There are no releases.  Use the latest commit.
> +  (let ((commit "6cee80685f735ea6c2373db2622a1f1ee9f39d39")
> +        (revision "0"))
> +    (package
> +      (name "libretro-beetle-gba")
> +      ;; Use Mednafen core version as base.  Defined in libretro.cpp:73.
> +      (version (git-version "0.9.36" revision commit))
> +      (source
> +       (origin
> +         (method git-fetch)
> +         (uri (git-reference
> +               (url "https://github.com/libretro/beetle-gba-libretro")
> +               (commit commit)))
> +         (file-name (git-file-name name version))
> +         (sha256
> +          (base32 "14fm2g3hrsvvd57d6m9apzc30ypa4m0m5hk2viq422fm2l9y0xbb"))))
> +      (build-system gnu-build-system)
> +      (arguments
> +       (list
> +        #:make-flags #~(list "CC=gcc"

You should prefer to use (string-append "CC=" #$(cc-for-target)), which
gives the right gcc command name when cross-compiling.

> +                             (string-append "GIT_VERSION="
> +                                            #$commit)
> +                             (string-append "DESTDIR="
> +                                            #$output) "prefix=/")
> +        ;; No tests.
> +        #:tests? #f
> +        #:phases #~(modify-phases %standard-phases
> +                     (delete 'configure))))
> +      (home-page "https://github.com/libretro/beetle-gba-libretro")
> +      (synopsis "Standalone port of Mednafen GBA to libretro")
> +      (description
> +       "A standalone port of Mednafen’s GameBoy Advance emulator called
> +Beetle GBA to libretro.  Beetle GBA is based on VBA-M, itself a fork of Visual Boy
> +Advance.  This is an old fork, and is not recommended over newer alternatives.")

Is the last sentence about VBA-M, or Visual Boy Advance?  I'm guessing
the later, but it could confuse so I'd drop it.

I've done the two above modifications myself:

--8<---------------cut here---------------start------------->8---
modified   gnu/packages/emulators.scm
@@ -762,22 +762,18 @@ (define-public libretro-beetle-gba
           (base32 "14fm2g3hrsvvd57d6m9apzc30ypa4m0m5hk2viq422fm2l9y0xbb"))))
       (build-system gnu-build-system)
       (arguments
-       (list
-        #:make-flags #~(list "CC=gcc"
-                             (string-append "GIT_VERSION="
-                                            #$commit)
-                             (string-append "DESTDIR="
-                                            #$output) "prefix=/")
-        ;; No tests.
-        #:tests? #f
-        #:phases #~(modify-phases %standard-phases
-                     (delete 'configure))))
+       (list #:make-flags #~(list (string-append "CC=" #$(cc-for-target))
+                                  (string-append "GIT_VERSION=" #$commit)
+                                  (string-append "prefix=" #$output))
+             #:tests? #f                ;no tests
+             #:phases #~(modify-phases %standard-phases
+                          (delete 'configure))))
       (home-page "https://github.com/libretro/beetle-gba-libretro")
       (synopsis "Standalone port of Mednafen GBA to libretro")
       (description
-       "A standalone port of Mednafen’s GameBoy Advance emulator called
-Beetle GBA to libretro.  Beetle GBA is based on VBA-M, itself a fork of Visual Boy
-Advance.  This is an old fork, and is not recommended over newer alternatives.")
+       "A standalone port of Mednafen’s GameBoy Advance emulator called Beetle
+GBA to libretro.  Beetle GBA is based on VBA-M, itself a fork of Visual Boy
+Advance.")
       (license license:gpl2+))))
 
 (define-public mgba
--8<---------------cut here---------------end--------------->8---

I've also ordered this package before libretro-bsnes-jg (it's nice to
keep packages sorted in alphabetic order, as much as possible).

Pushed as commit f73588e3b0.

-- 
Thanks,
Maxim




bug archived. Request was from Debbugs Internal Request <help-debbugs <at> gnu.org> to internal_control <at> debbugs.gnu.org. (Thu, 16 Jan 2025 12:24:16 GMT) Full text and rfc822 format available.

This bug report was last modified 149 days ago.

Previous Next


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