* [bug#58499] [PATCH] gnu: add citra (stable version)
@ 2022-10-13 14:59 florhizome
2022-10-17 8:37 ` Liliana Marie Prikler
` (2 more replies)
0 siblings, 3 replies; 9+ messages in thread
From: florhizome @ 2022-10-13 14:59 UTC (permalink / raw)
To: 58499; +Cc: florhizome
From: florhizome <florhizome@posteo.net>
This adds the citra emulator for nintendo 3ds games - the nightly repo marks the stable version. NixOs does some things for compatibility checks that we don't but it works fine so far. :)
---
gnu/packages/emulators.scm | 60 ++++++++++++++++++++++++++++++++++++++
1 file changed, 60 insertions(+)
diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
index 38795b703c..3121a71484 100644
--- a/gnu/packages/emulators.scm
+++ b/gnu/packages/emulators.scm
@@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages cross-base)
#:use-module (gnu packages curl)
#:use-module (gnu packages digest)
+ #:use-module (gnu packages documentation)
#:use-module (gnu packages elf)
#:use-module (gnu packages flex)
#:use-module (gnu packages fltk)
@@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages gl)
#:use-module (gnu packages glib)
#:use-module (gnu packages gnome)
+ #:use-module (gnu packages gstreamer)
#:use-module (gnu packages gtk)
#:use-module (gnu packages image)
#:use-module (gnu packages libedit)
@@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages xorg)
#:use-module (gnu packages web)
#:use-module (guix build-system cmake)
+ #:use-module (guix build-system qt)
#:use-module (guix build-system glib-or-gtk)
#:use-module (guix build-system gnu)
#:use-module (guix build-system meson)
@@ -245,9 +248,66 @@ (define-public desmume
"DeSmuME is an emulator for the Nintendo DS handheld gaming console.")
(license license:gpl2)))
+(define-public citra
+ (package
+ (name "citra")
+ (version "1785")
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/citra-emu/citra-nightly")
+ (commit
+ (string-append "nightly-" version))
+ (recursive? #t)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+ "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))))
+ (build-system qt-build-system)
+ (arguments
+ (list
+ #:configure-flags
+ #~(list "-DUSE_SYSTEM_BOOST=ON"
+ "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
+ "-DCITRA_USE_BUNDLED_QT=OFF"
+ "-DCITRA_USE_BUNDLED_SDL2=OFF"
+ "-DCMAKE_BUILD_TYPE=Release"
+ "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
+ "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
+ "-DENABLE_QT_TRANSLATION=ON"
+ "-DENABLE_WEB_SERVICE=ON"
+ "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
+ "-DUSE_DISCORD_PRESENCE=ON")
+ #:tests? #f
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-before 'configure 'delete-check
+ (lambda _
+ (substitute* "CMakeLists.txt"
+ (("check_submodules_present\\(\\)")"")))))))
+ (native-inputs
+ (list pkg-config doxygen))
+ (inputs
+ (list qtbase-5 qtmultimedia-5 qttools-5
+ qtwayland libpng libxkbcommon
+ openssl
+ ffmpeg pulseaudio curl rapidjson
+ sdl2 libusb boost))
+ ;;gstreamer-plugins-bad needed for camera support
+ (propagated-inputs (list gst-plugins-bad xdg-utils shared-mime-info))
+ (home-page "https://citra-emu.org")
+ (synopsis "Nintendo 3DS Emulator")
+ (description "Citra is an experimental open-source Nintendo 3DS
+ emulator/debugger written in C++. It is written with portability in
+ mind, with builds actively maintained for Windows, Linux and macOS.
+ The \"nightly\" version marks the stable release.")
+ (license license:gpl3+)))
+
;; Building from recent Git because the official 5.0 release no longer builds.
;; Following commits and revision numbers of beta versions listed at
;; https://dolphin-emu.org/download/.
+
(define-public dolphin-emu
(let ((commit "a34823df61df65168aa40ef5e82e44defd4a0138")
(revision "13178"))
base-commit: 86ec52f66735b122b9035eba56516fd16f3be958
prerequisite-patch-id: 253e1cc8278ab9981294f4e483c7b29a466672d9
--
2.38.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH] gnu: add citra (stable version)
2022-10-13 14:59 [bug#58499] [PATCH] gnu: add citra (stable version) florhizome
@ 2022-10-17 8:37 ` Liliana Marie Prikler
2022-10-20 21:49 ` florhizome
2022-10-20 21:55 ` [bug#58499] [PATCH v2] gnu: Add citra florhizome
2022-11-01 17:58 ` [bug#58499] [PATCH v3] " florhizome
2 siblings, 1 reply; 9+ messages in thread
From: Liliana Marie Prikler @ 2022-10-17 8:37 UTC (permalink / raw)
To: florhizome, 58499
Am Donnerstag, dem 13.10.2022 um 14:59 +0000 schrieb
florhizome@posteo.net:
> From: florhizome <florhizome@posteo.net>
>
> This adds the citra emulator for nintendo 3ds games - the nightly
> repo marks the stable version. NixOs does some things for
> compatibility checks that we don't but it works fine so far. :)
Missing ChangeLog. Title should be: "gnu: Add citra."
> ---
Notes not meant for the commit log go below this line.
> gnu/packages/emulators.scm | 60
> ++++++++++++++++++++++++++++++++++++++
> 1 file changed, 60 insertions(+)
>
> diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
> index 38795b703c..3121a71484 100644
> --- a/gnu/packages/emulators.scm
> +++ b/gnu/packages/emulators.scm
> @@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages cross-base)
> #:use-module (gnu packages curl)
> #:use-module (gnu packages digest)
> + #:use-module (gnu packages documentation)
> #:use-module (gnu packages elf)
> #:use-module (gnu packages flex)
> #:use-module (gnu packages fltk)
> @@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages gl)
> #:use-module (gnu packages glib)
> #:use-module (gnu packages gnome)
> + #:use-module (gnu packages gstreamer)
> #:use-module (gnu packages gtk)
> #:use-module (gnu packages image)
> #:use-module (gnu packages libedit)
> @@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages xorg)
> #:use-module (gnu packages web)
> #:use-module (guix build-system cmake)
> + #:use-module (guix build-system qt)
> #:use-module (guix build-system glib-or-gtk)
> #:use-module (guix build-system gnu)
> #:use-module (guix build-system meson)
> @@ -245,9 +248,66 @@ (define-public desmume
> "DeSmuME is an emulator for the Nintendo DS handheld gaming
> console.")
> (license license:gpl2)))
>
> +(define-public citra
> + (package
> + (name "citra")
> + (version "1785")
I'd prefer a let-bound revision/commit pair, since this is very much
experimental software. Citra seems to be using a build version base of
"0" [cf. 1], so we can do the same.
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/citra-emu/citra-nightly")
Even if using nightly revisions, I think we might want to prefer the
main repository.
> + (commit
> + (string-append "nightly-" version))
> + (recursive? #t)))
Recursive checkouts are discouraged. Wherefore do you need one?
> + (file-name (git-file-name name version))
> + (sha256
> + (base32
> + "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))))
> + (build-system qt-build-system)
> + (arguments
> + (list
> + #:configure-flags
> + #~(list "-DUSE_SYSTEM_BOOST=ON"
> + "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
> + "-DCITRA_USE_BUNDLED_QT=OFF"
> + "-DCITRA_USE_BUNDLED_SDL2=OFF"
Don't forget to actually unbundle all of the above.
> + "-DCMAKE_BUILD_TYPE=Release"
> + "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
> + "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
> + "-DENABLE_QT_TRANSLATION=ON"
> + "-DENABLE_WEB_SERVICE=ON"
> + "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
> + "-DUSE_DISCORD_PRESENCE=ON")
Don't enable features for the profit of proprietary software.
> + #:tests? #f
Why?
> + #:phases
> + #~(modify-phases %standard-phases
> + (add-before 'configure 'delete-check
> + (lambda _
> + (substitute* "CMakeLists.txt"
> + (("check_submodules_present\\(\\)")"")))))))
> + (native-inputs
> + (list pkg-config doxygen))
> + (inputs
> + (list qtbase-5 qtmultimedia-5 qttools-5
> + qtwayland libpng libxkbcommon
> + openssl
> + ffmpeg pulseaudio curl rapidjson
> + sdl2 libusb boost))
> + ;;gstreamer-plugins-bad needed for camera support
> + (propagated-inputs (list gst-plugins-bad xdg-utils shared-mime-
> info))
For the gst-plugins, check if you can isolate a minimum set of them.
Also, use wrappers rather than propagation.
> + (home-page "https://citra-emu.org")
> + (synopsis "Nintendo 3DS Emulator")
> + (description "Citra is an experimental open-source Nintendo 3DS
> + emulator/debugger written in C++. It is written with portability
> in
> + mind, with builds actively maintained for Windows, Linux and macOS.
> + The \"nightly\" version marks the stable release.")
"Citra is an experimental emulator/debugger for the Nintendo 3DS
written in C++. It emulates a subset of the Nintendo 3DS' hardware."
> + (license license:gpl3+)))
Should be gpl2+.
Cheers
[1]
https://github.com/citra-emu/citra/blob/47c4f31c8db86b043a1802a65f1df08aeec8e772/CMakeModules/GenerateSCMRev.cmake#L22
^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH] gnu: add citra (stable version)
2022-10-17 8:37 ` Liliana Marie Prikler
@ 2022-10-20 21:49 ` florhizome
2022-10-21 6:30 ` Liliana Marie Prikler
0 siblings, 1 reply; 9+ messages in thread
From: florhizome @ 2022-10-20 21:49 UTC (permalink / raw)
To: Liliana Marie Prikler, 58499
Hi Liliana!
thx for the revision! I also have yuzu in the pipeline,
which is by the same/similar team, which makes this even more rewarding.
The next patch will remove all submodules that I was able to
remove. More could be done by replacing the rest under external/ with.
symlinks, I guess. Also, tests seem to work under x86, and using only
gst-plugins-bad-minimal in a wrapper seems to be enough to make the
camera work.
This leaves us with the repo/versioning and the discord issue.
On 2022-10-17, 10:37 +0200, Liliana Marie Prikler <liliana.prikler@ist.tugraz.at> wrote:
> I'd prefer a let-bound revision/commit pair, since this is very much
> experimental software. Citra seems to be using a build version base of
> "0" [cf. 1], so we can do the same.
>> + (source
>> + (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url "https://github.com/citra-emu/citra-nightly")
> Even if using nightly revisions, I think we might want to prefer the
> main repository.
After reading their repo-policy
The main repo is for development, the nightly one picks out stable
features, the experimental/canary one adds more unstable ones. By going with
"main" we basically skip any stability guarantees, that the devs are trying
to give.
For me personally, the version numbers are very helpful, since they make it easier
to track dependency changes.
The patch I am preparing will use 0 as main version, though.
> Recursive checkouts are discouraged. Wherefore do you need one?
>> + (file-name (git-file-name name version))
>> + (sha256
>> + (base32
>> + "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))))
>> + (build-system qt-build-system)
>> + (arguments
>> + (list
>> + #:configure-flags
>> + #~(list "-DUSE_SYSTEM_BOOST=ON"
>> + "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
>> + "-DCITRA_USE_BUNDLED_QT=OFF"
>> + "-DCITRA_USE_BUNDLED_SDL2=OFF"
> Don't forget to actually unbundle all of the above.
Actually ffmpeg and sdl are not bundled in our version,
but later versions will bundle sdl again.
>> + "-DCMAKE_BUILD_TYPE=Release"
>> + "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
>> + "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
>> + "-DENABLE_QT_TRANSLATION=ON"
>> + "-DENABLE_WEB_SERVICE=ON"
>> + "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
>> + "-DUSE_DISCORD_PRESENCE=ON")
> Don't enable features for the profit of proprietary software.
As far as I understand, there are other programs, and free alternatives
other then discord using its status protocol.
>> + (license license:gpl3+)))
> Should be gpl2+.
>
To me it looks like gpl2 only, am I overlooking something?
Cheers, florhizome
^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH v2] gnu: Add citra
2022-10-13 14:59 [bug#58499] [PATCH] gnu: add citra (stable version) florhizome
2022-10-17 8:37 ` Liliana Marie Prikler
@ 2022-10-20 21:55 ` florhizome
2022-10-22 17:29 ` Liliana Marie Prikler
2022-11-01 17:58 ` [bug#58499] [PATCH v3] " florhizome
2 siblings, 1 reply; 9+ messages in thread
From: florhizome @ 2022-10-20 21:55 UTC (permalink / raw)
To: 58499; +Cc: florhizome
From: florhizome <florhizome@posteo.net>
* gnu: Add citra.
---
gnu/packages/emulators.scm | 103 +++++++++++++++++++++++++++++++++++++
1 file changed, 103 insertions(+)
diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
index 4d7710f8ea..be4e10bb9c 100644
--- a/gnu/packages/emulators.scm
+++ b/gnu/packages/emulators.scm
@@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages cross-base)
#:use-module (gnu packages curl)
#:use-module (gnu packages digest)
+ #:use-module (gnu packages documentation)
#:use-module (gnu packages elf)
#:use-module (gnu packages flex)
#:use-module (gnu packages fltk)
@@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages gl)
#:use-module (gnu packages glib)
#:use-module (gnu packages gnome)
+ #:use-module (gnu packages gstreamer)
#:use-module (gnu packages gtk)
#:use-module (gnu packages image)
#:use-module (gnu packages libedit)
@@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages xorg)
#:use-module (gnu packages web)
#:use-module (guix build-system cmake)
+ #:use-module (guix build-system qt)
#:use-module (guix build-system glib-or-gtk)
#:use-module (guix build-system gnu)
#:use-module (guix build-system meson)
@@ -245,9 +248,109 @@ (define-public desmume
"DeSmuME is an emulator for the Nintendo DS handheld gaming console.")
(license license:gpl2)))
+(define-public citra
+ (let ((version "0")
+ (revision "1785")
+ (commit "baecc18d8c5365af0dddb231bc8c0a9c03850bf6"))
+ (package
+ (name "citra")
+ (version (git-version version
+ (string-append "nightly-" revision) commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/citra-emu/citra-nightly")
+ (commit
+ (string-append "nightly-" revision))
+ ;;some submodules can't be avoided
+ ;;TODO devendor the rest
+ (recursive? #t)))
+ (sha256
+ (base32
+ "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))
+ (file-name (git-file-name name version))
+ (modules '((guix build utils)))
+ (snippet
+ '(begin
+ ;; Remove as much external stuff as we can
+ ;; f.e. some files in boost are still needed
+ (for-each (lambda (dir)
+ (delete-file-recursively
+ (string-append "externals/" dir)))
+ '("android-ifaddrs"
+ "catch"
+ "libyuv"
+ "libressl"
+ "getopt"
+ "libusb"))
+ ;; Clean up source.
+ (for-each delete-file
+ (find-files "." ".*\\.(bin|dsy|exe|jar|rar)$"))
+ #t))))
+ (build-system qt-build-system)
+ (arguments
+ (list
+ #:configure-flags
+ #~(list
+ "-DUSE_SYSTEM_BOOST=ON"
+ "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
+ "-DCITRA_USE_BUNDLED_QT=OFF"
+ "-DCITRA_USE_BUNDLED_SDL2=OFF"
+ "-DCMAKE_BUILD_TYPE=Release"
+ "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
+ "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
+ "-DENABLE_QT_TRANSLATION=ON"
+ "-DENABLE_WEB_SERVICE=ON"
+ "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
+ "-DUSE_DISCORD_PRESENCE=ON")
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-before 'configure 'delete-check
+ (lambda _
+ (substitute* "CMakeLists.txt"
+ (("check_submodules_present\\(\\)")""))))
+ (add-after 'qt-wrap 'wrap-other-dependencies
+ (lambda* (#:key outputs #:allow-other-keys)
+ (for-each
+ (lambda (bin)
+ (wrap-program bin)
+ `("GST_PLUGIN_SYSTEM_PATH" prefix
+ (,(getenv "GST_PLUGIN_SYSTEM_PATH"))))
+ `(,(search-input-file outputs "bin/citra")
+ ,(search-input-file outputs "bin/citra-qt"))))))))
+ (native-inputs
+ (list pkg-config doxygen catch2))
+ (inputs
+ (list qtbase-5
+ qtmultimedia-5
+ qttools-5
+ qtwayland-5
+ libpng
+ libxkbcommon
+ openssl
+ libfdk
+ ffmpeg
+ pulseaudio
+ curl
+ rapidjson
+ sdl2
+ libusb
+ boost
+ gst-plugins-bad-minimal))
+ ;;gstreamer-plugins-bad-minimal needed for camera support
+ (propagated-inputs (list xdg-utils shared-mime-info))
+ (home-page "https://citra-emu.org")
+ (synopsis "Nintendo 3DS Emulator")
+ (description "Citra is an experimental emulator/debugger for the Nintendo 3DS
+ written in C++. It emulates a subset of the Nintendo 3DS' hardware.")
+ (license license:gpl2+))))
+
+
;; Building from recent Git because the official 5.0 release no longer builds.
;; Following commits and revision numbers of beta versions listed at
;; https://dolphin-emu.org/download/.
+
(define-public dolphin-emu
(let ((commit "a34823df61df65168aa40ef5e82e44defd4a0138")
(revision "13178"))
base-commit: 00ff6f7c399670a76efffb91276dea2633cc130c
--
2.38.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH] gnu: add citra (stable version)
2022-10-20 21:49 ` florhizome
@ 2022-10-21 6:30 ` Liliana Marie Prikler
0 siblings, 0 replies; 9+ messages in thread
From: Liliana Marie Prikler @ 2022-10-21 6:30 UTC (permalink / raw)
To: florhizome, 58499
Hi,
Am Donnerstag, dem 20.10.2022 um 21:49 +0000 schrieb florhizome:
> On 2022-10-17, 10:37 +0200, Liliana Marie Prikler
> <liliana.prikler@ist.tugraz.at> wrote:
> > I'd prefer a let-bound revision/commit pair, since this is very
> > much
> > experimental software. Citra seems to be using a build version
> > base of
> > "0" [cf. 1], so we can do the same.
> > > + (source
> > > + (origin
> > > + (method git-fetch)
> > > + (uri (git-reference
> > > + (url "https://github.com/citra-emu/citra-nightly")
> > Even if using nightly revisions, I think we might want to prefer
> > the main repository.
>
> After reading their repo-policy
> The main repo is for development, the nightly one picks out stable
> features, the experimental/canary one adds more unstable ones. By
> going with "main" we basically skip any stability guarantees, that
> the devs are trying to give.
Okay, in that case it makes sense to use nightly.
> For me personally, the version numbers are very helpful, since they
> make it easier to track dependency changes.
> The patch I am preparing will use 0 as main version, though.
Therefore I suggest (git-version "0" revision commit) where revision is
the nightly version number.
> > Recursive checkouts are discouraged. Wherefore do you need one?
> > > + (file-name (git-file-name name version))
> > > + (sha256
> > > + (base32
> > > +
> > > "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))))
> > > + (build-system qt-build-system)
> > > + (arguments
> > > + (list
> > > + #:configure-flags
> > > + #~(list "-DUSE_SYSTEM_BOOST=ON"
> > > + "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
> > > + "-DCITRA_USE_BUNDLED_QT=OFF"
> > > + "-DCITRA_USE_BUNDLED_SDL2=OFF"
> > Don't forget to actually unbundle all of the above.
>
> Actually ffmpeg and sdl are not bundled in our version,
> but later versions will bundle sdl again.
You could try something cheeky like
(when (file-exists? "ffmpeg") (delete-recursively "ffmpeg"))
in the snippet with an appropriate comment.
> > > + "-DCMAKE_BUILD_TYPE=Release"
> > > + "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
> > > + "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
> > > + "-DENABLE_QT_TRANSLATION=ON"
> > > + "-DENABLE_WEB_SERVICE=ON"
> > > + "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
> > > + "-DUSE_DISCORD_PRESENCE=ON")
> > Don't enable features for the profit of proprietary software.
>
> As far as I understand, there are other programs, and free
> alternatives other then discord using its status protocol.
Which ones?
As far as I'm aware, XMPP is a completely free alternative to the
entire protocol ;)
Cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH v2] gnu: Add citra
2022-10-20 21:55 ` [bug#58499] [PATCH v2] gnu: Add citra florhizome
@ 2022-10-22 17:29 ` Liliana Marie Prikler
2022-11-01 15:56 ` Florian
0 siblings, 1 reply; 9+ messages in thread
From: Liliana Marie Prikler @ 2022-10-22 17:29 UTC (permalink / raw)
To: florhizome, 58499
Am Donnerstag, dem 20.10.2022 um 21:55 +0000 schrieb
florhizome@posteo.net:
> From: florhizome <florhizome@posteo.net>
>
> * gnu: Add citra.
Missing ChangeLog.
> ---
> gnu/packages/emulators.scm | 103
> +++++++++++++++++++++++++++++++++++++
> 1 file changed, 103 insertions(+)
>
> diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
> index 4d7710f8ea..be4e10bb9c 100644
> --- a/gnu/packages/emulators.scm
> +++ b/gnu/packages/emulators.scm
> @@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages cross-base)
> #:use-module (gnu packages curl)
> #:use-module (gnu packages digest)
> + #:use-module (gnu packages documentation)
> #:use-module (gnu packages elf)
> #:use-module (gnu packages flex)
> #:use-module (gnu packages fltk)
> @@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages gl)
> #:use-module (gnu packages glib)
> #:use-module (gnu packages gnome)
> + #:use-module (gnu packages gstreamer)
> #:use-module (gnu packages gtk)
> #:use-module (gnu packages image)
> #:use-module (gnu packages libedit)
> @@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages xorg)
> #:use-module (gnu packages web)
> #:use-module (guix build-system cmake)
> + #:use-module (guix build-system qt)
> #:use-module (guix build-system glib-or-gtk)
> #:use-module (guix build-system gnu)
> #:use-module (guix build-system meson)
> @@ -245,9 +248,109 @@ (define-public desmume
> "DeSmuME is an emulator for the Nintendo DS handheld gaming
> console.")
> (license license:gpl2)))
>
> +(define-public citra
> + (let ((version "0")
> + (revision "1785")
> + (commit "baecc18d8c5365af0dddb231bc8c0a9c03850bf6"))
You don't need to let-bind version. Add a comment next to revision
explaining that we mean nightly revisions.
> + (package
> + (name "citra")
> + (version (git-version version
> + (string-append "nightly-" revision)
> commit))
"nightly-" is not needed here.
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/citra-emu/citra-nightly")
> + (commit
> + (string-append "nightly-" revision))
> + ;;some submodules can't be avoided
> + ;;TODO devendor the rest
> + (recursive? #t)))
Why not?
> + (sha256
> + (base32
> + "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))
> + (file-name (git-file-name name version))
> + (modules '((guix build utils)))
> + (snippet
> + '(begin
> + ;; Remove as much external stuff as we can
> + ;; f.e. some files in boost are still needed
We have a boost package.
> + (for-each (lambda (dir)
> + (delete-file-recursively
> + (string-append "externals/" dir)))
> + '("android-ifaddrs"
> + "catch"
> + "libyuv"
> + "libressl"
> + "getopt"
> + "libusb"))
> + ;; Clean up source.
> + (for-each delete-file
> + (find-files "."
> ".*\\.(bin|dsy|exe|jar|rar)$"))
> + #t))))
> + (build-system qt-build-system)
> + (arguments
> + (list
> + #:configure-flags
> + #~(list
> + "-DUSE_SYSTEM_BOOST=ON"
> + "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
> + "-DCITRA_USE_BUNDLED_QT=OFF"
> + "-DCITRA_USE_BUNDLED_SDL2=OFF"
> + "-DCMAKE_BUILD_TYPE=Release"
> + "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
> + "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
> + "-DENABLE_QT_TRANSLATION=ON"
> + "-DENABLE_WEB_SERVICE=ON"
> + "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
> + "-DUSE_DISCORD_PRESENCE=ON")
Again, no cookies for proprietary software.
> + #:phases
> + #~(modify-phases %standard-phases
> + (add-before 'configure 'delete-check
> + (lambda _
> + (substitute* "CMakeLists.txt"
> + (("check_submodules_present\\(\\)")""))))
> + (add-after 'qt-wrap 'wrap-other-dependencies
> + (lambda* (#:key outputs #:allow-other-keys)
> + (for-each
> + (lambda (bin)
> + (wrap-program bin)
> + `("GST_PLUGIN_SYSTEM_PATH" prefix
> + (,(getenv "GST_PLUGIN_SYSTEM_PATH"))))
> + `(,(search-input-file outputs "bin/citra")
> + ,(search-input-file outputs "bin/citra-qt"))))))))
You can use find-files here and it'll probably be more natural.
> + (native-inputs
> + (list pkg-config doxygen catch2))
> + (inputs
> + (list qtbase-5
> + qtmultimedia-5
> + qttools-5
> + qtwayland-5
> + libpng
> + libxkbcommon
> + openssl
> + libfdk
> + ffmpeg
> + pulseaudio
> + curl
> + rapidjson
> + sdl2
> + libusb
> + boost
> + gst-plugins-bad-minimal))
Inputs are best sorted.
> + ;;gstreamer-plugins-bad-minimal needed for camera support
Use a line comment "; camera support".
Emacs adds proper distance with M-;
> + (propagated-inputs (list xdg-utils shared-mime-info))
> + (home-page "https://citra-emu.org")
> + (synopsis "Nintendo 3DS Emulator")
> + (description "Citra is an experimental emulator/debugger for the
> Nintendo 3DS
> + written in C++. It emulates a subset of the Nintendo 3DS'
> hardware.")
> + (license license:gpl2+))))
Cheers
^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH v2] gnu: Add citra
2022-10-22 17:29 ` Liliana Marie Prikler
@ 2022-11-01 15:56 ` Florian
0 siblings, 0 replies; 9+ messages in thread
From: Florian @ 2022-11-01 15:56 UTC (permalink / raw)
To: Liliana Marie Prikler, 58499
>> + (source
>> + (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url "https://github.com/citra-emu/citra-nightly")
>> + (commit
>> + (string-append "nightly-" revision))
>> + ;;some submodules can't be avoided
>> + ;;TODO devendor the rest
>> + (recursive? #t)))
> Why not?
>> + (sha256
>> + (base32
>> + "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))
>> + (file-name (git-file-name name version))
>> + (modules '((guix build utils)))
>> + (snippet
>> + '(begin
>> + ;; Remove as much external stuff as we can
>> + ;; f.e. some files in boost are still needed
> We have a boost package.
What I mean by avoid: the rest of submodules we would need to replace in
place (which I mean by devendor). I have tried to add a build
phase doing so, but wasn't successful by now. It will be a
huge amount of work added anyways, which would take me a bigger amount
of time.
For some reason disabling the bundled version of boost still seems to
leave some links to the bundled boost in the build, so when I remove it,
the build fails.
>> + "-DUSE_DISCORD_PRESENCE=ON")
> Again, no cookies for proprietary software.
I don't see use to discuss this as I would like to get to get this done
and I don't need it.
v3 coming soon, I hope i get the changelog right this time...
Cheers.
^ permalink raw reply [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH v3] gnu: Add citra
2022-10-13 14:59 [bug#58499] [PATCH] gnu: add citra (stable version) florhizome
2022-10-17 8:37 ` Liliana Marie Prikler
2022-10-20 21:55 ` [bug#58499] [PATCH v2] gnu: Add citra florhizome
@ 2022-11-01 17:58 ` florhizome
2022-11-02 8:04 ` Liliana Marie Prikler
2 siblings, 1 reply; 9+ messages in thread
From: florhizome @ 2022-11-01 17:58 UTC (permalink / raw)
To: 58499; +Cc: florhizome
From: florhizome <florhizome@posteo.net>
* gnu/packages/emulators.scm (citra): New variable.
---
gnu/packages/emulators.scm | 100 +++++++++++++++++++++++++++++++++++++
1 file changed, 100 insertions(+)
diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
index 4d7710f8ea..3ea3bfe327 100644
--- a/gnu/packages/emulators.scm
+++ b/gnu/packages/emulators.scm
@@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages cross-base)
#:use-module (gnu packages curl)
#:use-module (gnu packages digest)
+ #:use-module (gnu packages documentation)
#:use-module (gnu packages elf)
#:use-module (gnu packages flex)
#:use-module (gnu packages fltk)
@@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages gl)
#:use-module (gnu packages glib)
#:use-module (gnu packages gnome)
+ #:use-module (gnu packages gstreamer)
#:use-module (gnu packages gtk)
#:use-module (gnu packages image)
#:use-module (gnu packages libedit)
@@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
#:use-module (gnu packages xorg)
#:use-module (gnu packages web)
#:use-module (guix build-system cmake)
+ #:use-module (guix build-system qt)
#:use-module (guix build-system glib-or-gtk)
#:use-module (guix build-system gnu)
#:use-module (guix build-system meson)
@@ -245,9 +248,106 @@ (define-public desmume
"DeSmuME is an emulator for the Nintendo DS handheld gaming console.")
(license license:gpl2)))
+(define-public citra
+ ;; we use the 'nightly' revision, picking stable features
+ (let ((revision "1785")
+ (commit "baecc18d8c5365af0dddb231bc8c0a9c03850bf6"))
+ (package
+ (name "citra")
+ (version (git-version "0" revision commit))
+ (source
+ (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/citra-emu/citra-nightly")
+ (commit
+ (string-append "nightly-" revision))
+ ;;some submodules can't be avoided
+ ;;TODO devendor the rest
+ (recursive? #t)))
+ (sha256
+ (base32
+ "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))
+ (file-name (git-file-name name version))
+ (modules '((guix build utils)))
+ (snippet
+ '(begin
+ ;; Remove as much external stuff as we can
+ ;; f.e. some files in boost are still needed
+ (for-each (lambda (dir)
+ (delete-file-recursively
+ (string-append "externals/" dir)))
+ '("android-ifaddrs"
+ "catch"
+ "discord-rpc"
+ "getopt"
+ "libyuv"
+ "libressl"
+ "libusb"))
+ ;; Clean up source.
+ (for-each delete-file
+ (find-files "." ".*\\.(bin|dsy|exe|jar|rar)$"))
+ #t))))
+ (build-system qt-build-system)
+ (arguments
+ (list
+ #:configure-flags
+ #~(list
+ "-DUSE_SYSTEM_BOOST=ON"
+ "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
+ "-DCITRA_USE_BUNDLED_QT=OFF"
+ "-DCITRA_USE_BUNDLED_SDL2=OFF"
+ "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
+ "-DCMAKE_BUILD_TYPE=Release"
+ "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
+ "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
+ "-DENABLE_QT_TRANSLATION=ON"
+ "-DENABLE_WEB_SERVICE=ON")
+ #:phases
+ #~(modify-phases %standard-phases
+ (add-before 'configure 'delete-check
+ (lambda _
+ (substitute* "CMakeLists.txt"
+ (("check_submodules_present\\(\\)")""))))
+ (add-after 'qt-wrap 'wrap-gst-plugins
+ (lambda* (#:key outputs #:allow-other-keys)
+ (for-each
+ (lambda (bin)
+ (wrap-program bin)
+ `("GST_PLUGIN_SYSTEM_PATH" prefix
+ (,(getenv "GST_PLUGIN_SYSTEM_PATH"))))
+ `(,(search-input-file outputs "bin/citra")
+ ,(search-input-file outputs "bin/citra-qt"))))))))
+ (native-inputs
+ (list catch2 doxygen pkg-config))
+ (inputs
+ (list boost
+ curl
+ ffmpeg
+ gst-plugins-bad-minimal ;camera-support
+ libfdk
+ libpng
+ libusb
+ libxkbcommon
+ openssl
+ pulseaudio
+ qtbase-5
+ qtmultimedia-5
+ qttools-5
+ qtwayland-5
+ sdl2))
+ (propagated-inputs (list xdg-utils shared-mime-info))
+ (home-page "https://citra-emu.org")
+ (synopsis "Nintendo 3DS Emulator")
+ (description "Citra is an experimental emulator/debugger for the Nintendo 3DS
+ written in C++. It emulates a subset of the Nintendo 3DS' hardware.")
+ (license license:gpl2+))))
+
+
;; Building from recent Git because the official 5.0 release no longer builds.
;; Following commits and revision numbers of beta versions listed at
;; https://dolphin-emu.org/download/.
+
(define-public dolphin-emu
(let ((commit "a34823df61df65168aa40ef5e82e44defd4a0138")
(revision "13178"))
--
2.38.0
^ permalink raw reply related [flat|nested] 9+ messages in thread
* [bug#58499] [PATCH v3] gnu: Add citra
2022-11-01 17:58 ` [bug#58499] [PATCH v3] " florhizome
@ 2022-11-02 8:04 ` Liliana Marie Prikler
0 siblings, 0 replies; 9+ messages in thread
From: Liliana Marie Prikler @ 2022-11-02 8:04 UTC (permalink / raw)
To: florhizome, 58499
Am Dienstag, dem 01.11.2022 um 17:58 +0000 schrieb
florhizome@posteo.net:
> From: florhizome <florhizome@posteo.net>
>
> * gnu/packages/emulators.scm (citra): New variable.
> ---
> gnu/packages/emulators.scm | 100
> +++++++++++++++++++++++++++++++++++++
> 1 file changed, 100 insertions(+)
>
> diff --git a/gnu/packages/emulators.scm b/gnu/packages/emulators.scm
> index 4d7710f8ea..3ea3bfe327 100644
> --- a/gnu/packages/emulators.scm
> +++ b/gnu/packages/emulators.scm
> @@ -61,6 +61,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages cross-base)
> #:use-module (gnu packages curl)
> #:use-module (gnu packages digest)
> + #:use-module (gnu packages documentation)
> #:use-module (gnu packages elf)
> #:use-module (gnu packages flex)
> #:use-module (gnu packages fltk)
> @@ -74,6 +75,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages gl)
> #:use-module (gnu packages glib)
> #:use-module (gnu packages gnome)
> + #:use-module (gnu packages gstreamer)
> #:use-module (gnu packages gtk)
> #:use-module (gnu packages image)
> #:use-module (gnu packages libedit)
> @@ -105,6 +107,7 @@ (define-module (gnu packages emulators)
> #:use-module (gnu packages xorg)
> #:use-module (gnu packages web)
> #:use-module (guix build-system cmake)
> + #:use-module (guix build-system qt)
> #:use-module (guix build-system glib-or-gtk)
> #:use-module (guix build-system gnu)
> #:use-module (guix build-system meson)
> @@ -245,9 +248,106 @@ (define-public desmume
> "DeSmuME is an emulator for the Nintendo DS handheld gaming
> console.")
> (license license:gpl2)))
>
> +(define-public citra
> + ;; we use the 'nightly' revision, picking stable features
> + (let ((revision "1785")
> + (commit "baecc18d8c5365af0dddb231bc8c0a9c03850bf6"))
Note that the latest citra revision is now 1802. Feel free to bump it
during review.
> + (package
> + (name "citra")
> + (version (git-version "0" revision commit))
> + (source
> + (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/citra-emu/citra-nightly")
> + (commit
> + (string-append "nightly-" revision))
> + ;;some submodules can't be avoided
> + ;;TODO devendor the rest
> + (recursive? #t)))
For those submodules that can't be avoided, the "unpack after unpack"
pattern is still preferable to recursive checkouts; the latter can pull
in new vendored modules sneakily.
For instance, the first package that fails the build because it's
missing is cryptopp [1]. They vendor version 8.5 while the latest is
8.7. I think packaging cryptopp and trying to unvendor it makes sense.
Even if not completely unvendored (i.e. we still pull in the source
instead of the built package, as is e.g. done for armips in ppsspp),
I'd prefer the origin to be explicit.
> + (sha256
> + (base32
> + "1grkrrxs1497i51spgnwmgfkqgkm7gplylhcrk67agaklx65d5s9"))
> + (file-name (git-file-name name version))
> + (modules '((guix build utils)))
> + (snippet
> + '(begin
> + ;; Remove as much external stuff as we can
> + ;; f.e. some files in boost are still needed
> + (for-each (lambda (dir)
> + (delete-file-recursively
> + (string-append "externals/" dir)))
> + '("android-ifaddrs"
> + "catch"
> + "discord-rpc"
> + "getopt"
> + "libyuv"
> + "libressl"
> + "libusb"))
> + ;; Clean up source.
> + (for-each delete-file
> + (find-files "."
> ".*\\.(bin|dsy|exe|jar|rar)$"))
> + #t))))
> + (build-system qt-build-system)
> + (arguments
> + (list
> + #:configure-flags
> + #~(list
> + "-DUSE_SYSTEM_BOOST=ON"
> + "-DCITRA_USE_BUNDLED_FFMPEG=OFF"
> + "-DCITRA_USE_BUNDLED_QT=OFF"
> + "-DCITRA_USE_BUNDLED_SDL2=OFF"
> + "-DCITRA_ENABLE_COMPATIBILITY_REPORTING=OFF"
> + "-DCMAKE_BUILD_TYPE=Release"
> + "-DENABLE_COMPATIBILITY_LIST_DOWNLOAD=OFF"
> + "-DENABLE_FFMPEG_AUDIO_DECODER=ON"
> + "-DENABLE_QT_TRANSLATION=ON"
> + "-DENABLE_WEB_SERVICE=ON")
The web service should be disabled, we don't let our software phone
home.
> + #:phases
> + #~(modify-phases %standard-phases
> + (add-before 'configure 'delete-check
> + (lambda _
> + (substitute* "CMakeLists.txt"
> + (("check_submodules_present\\(\\)")""))))
Instead of 'delete-check, I'd name this 'patch-cmake. 'delete-check
implies that we're disabling checks, which we don't.
> + (add-after 'qt-wrap 'wrap-gst-plugins
> + (lambda* (#:key outputs #:allow-other-keys)
> + (for-each
> + (lambda (bin)
> + (wrap-program bin)
> + `("GST_PLUGIN_SYSTEM_PATH" prefix
> + (,(getenv "GST_PLUGIN_SYSTEM_PATH"))))
> + `(,(search-input-file outputs "bin/citra")
> + ,(search-input-file outputs "bin/citra-qt"))))))))
> + (native-inputs
> + (list catch2 doxygen pkg-config))
> + (inputs
> + (list boost
> + curl
> + ffmpeg
> + gst-plugins-bad-minimal ;camera-support
> + libfdk
> + libpng
> + libusb
> + libxkbcommon
> + openssl
> + pulseaudio
> + qtbase-5
> + qtmultimedia-5
> + qttools-5
> + qtwayland-5
> + sdl2))
There seem to be more missing inputs, for instance json-modern-cxx
which has nlohmann::json (somewhat related, I do think that it's a
silly package name and we should rename it for searchability's sake).
> + (propagated-inputs (list xdg-utils shared-mime-info))
> + (home-page "https://citra-emu.org")
> + (synopsis "Nintendo 3DS Emulator")
> + (description "Citra is an experimental emulator/debugger for the
> Nintendo 3DS
I'd write out "and" instead of the slash.
> + written in C++. It emulates a subset of the Nintendo 3DS'
> hardware.")
> + (license license:gpl2+))))
> +
> +
Gratuitous newline.
> ;; Building from recent Git because the official 5.0 release no
> longer builds.
> ;; Following commits and revision numbers of beta versions listed at
> ;; https://dolphin-emu.org/download/.
> +
Gratuitous newline.
> (define-public dolphin-emu
> (let ((commit "a34823df61df65168aa40ef5e82e44defd4a0138")
> (revision "13178"))
Cheers
[1] https://github.com/weidai11/cryptopp
^ permalink raw reply [flat|nested] 9+ messages in thread
end of thread, other threads:[~2022-11-02 8:05 UTC | newest]
Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2022-10-13 14:59 [bug#58499] [PATCH] gnu: add citra (stable version) florhizome
2022-10-17 8:37 ` Liliana Marie Prikler
2022-10-20 21:49 ` florhizome
2022-10-21 6:30 ` Liliana Marie Prikler
2022-10-20 21:55 ` [bug#58499] [PATCH v2] gnu: Add citra florhizome
2022-10-22 17:29 ` Liliana Marie Prikler
2022-11-01 15:56 ` Florian
2022-11-01 17:58 ` [bug#58499] [PATCH v3] " florhizome
2022-11-02 8:04 ` Liliana Marie Prikler
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/guix.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).