* [bug#42376] [PATCH] Add raylib.
@ 2020-07-15 14:25 trymonv
2020-07-18 12:02 ` Jakub Kądziołka
0 siblings, 1 reply; 4+ messages in thread
From: trymonv @ 2020-07-15 14:25 UTC (permalink / raw)
To: 42376
gnu/packages/game-development.scm | 34 +++++++++++++++++++++++++++++++
1 file changed, 34 insertions(+)
diff --git a/gnu/packages/game-development.scm
b/gnu/packages/game-development.scm
index e45980e7a0..cba9b5bd45 100644
--- a/gnu/packages/game-development.scm
+++ b/gnu/packages/game-development.scm
@@ -2489,3 +2489,37 @@ fully dynamic omnidirectional shadows, global
illumination, HDR lighting,
deferred shading, morphological / temporal / multisample anti-aliasing,
and
much more.")
(license license:zlib))))
+
+(define-public raylib
+ (package
+ (name "raylib")
+ (version "3.0.0")
+ (source (origin
+ (method git-fetch)
+ (uri (git-reference
+ (url "https://github.com/raysan5/raylib/")
+ (commit version)))
+ (file-name (git-file-name name version))
+ (sha256
+ (base32
+
"1chj7sril4l2dxh7flp5ndddydbbf1fhnqlydaysm4m6waxidxmr"))))
+ (build-system cmake-build-system)
+ (arguments '(#:configure-flags `("-DSHARED=OFF" "-DSTATIC=ON")
+ #:tests? #f))
+ (inputs `())
+ (native-inputs `(("alsa-lib", alsa-lib)
+ ("glu", glu)
+ ("libx11", libx11)
+ ("libxrandr", libxrandr)
+ ("libxi", libxi)
+ ("libxinerama", libxinerama)
+ ("libxcursor", libxcursor)
+ ("mesa", mesa)
+ ("pkg-config", pkg-config)))
+ (synopsis "C library for videogame programming")
+ (description "raylib is a library which defines many data
structures and
+methods useful for programming small video games or video game
prototypes in
+plain C. It tries to provide simple bindings for OpenGL in an attempt
to
+conceal technical complexities behind an easy-to-use interface.")
+ (home-page "https://www.raylib.com/")
+ (license license:zlib)))
--
2.27.0
^ permalink raw reply related [flat|nested] 4+ messages in thread* [bug#42376] [PATCH] Add raylib.
2020-07-15 14:25 [bug#42376] [PATCH] Add raylib trymonv
@ 2020-07-18 12:02 ` Jakub Kądziołka
2020-09-24 15:45 ` Ludovic Courtès
0 siblings, 1 reply; 4+ messages in thread
From: Jakub Kądziołka @ 2020-07-18 12:02 UTC (permalink / raw)
To: trymonv; +Cc: 42376
[-- Attachment #1: Type: text/plain, Size: 3664 bytes --]
On Wed, Jul 15, 2020 at 04:25:21PM +0200, trymonv@cock.li wrote:
> gnu/packages/game-development.scm | 34 +++++++++++++++++++++++++++++++
> 1 file changed, 34 insertions(+)
Thanks for the patch! Note that this blob got included as the commit
message, which isn't really desirable. Instead of `git diff', I'd
recommend `git commit', followed by `git format-patch' and/or `git
send-email' (the latter is available as git:send-email - the send-email
output of the git package).
> diff --git a/gnu/packages/game-development.scm
> b/gnu/packages/game-development.scm
> index e45980e7a0..cba9b5bd45 100644
> --- a/gnu/packages/game-development.scm
> +++ b/gnu/packages/game-development.scm
Could you add a Copyright line for yourself?
> +(define-public raylib
> + (package
> + (name "raylib")
> + (version "3.0.0")
> + (source (origin
> + (method git-fetch)
> + (uri (git-reference
> + (url "https://github.com/raysan5/raylib/")
> + (commit version)))
> + (file-name (git-file-name name version))
> + (sha256
> + (base32
> + "1chj7sril4l2dxh7flp5ndddydbbf1fhnqlydaysm4m6waxidxmr"))))
> + (build-system cmake-build-system)
> + (arguments '(#:configure-flags `("-DSHARED=OFF" "-DSTATIC=ON")
> + #:tests? #f))
If you have to disable tests, put a small explanation in a comment.
However, in this case I managed to get the test suite to pass like this:
(arguments
`(#:configure-flags `("-DSHARED=OFF" "-DSTATIC=ON")
#:phases
(modify-phases %standard-phases
;; check needs to run after install
(delete 'check)
(add-after 'install 'check (assoc-ref %standard-phases 'check))
(add-before 'check 'set-CC
(lambda _
(setenv "CC" ,(cc-for-target))
#t)))))
> + (inputs `())
> + (native-inputs `(("alsa-lib", alsa-lib)
> + ("glu", glu)
> + ("libx11", libx11)
> + ("libxrandr", libxrandr)
> + ("libxi", libxi)
> + ("libxinerama", libxinerama)
> + ("libxcursor", libxcursor)
> + ("mesa", mesa)
> + ("pkg-config", pkg-config)))
I don't think these should be native-inputs. native-inputs are for
things that are ran during compilation, like the compiler itself or
pkg-config. The rest is clearly going to run at runtime, so they should
go in inputs. This distinction is necessary for cross-compilation. If
all your dependencies supported it, you could test it with
guix build --target=aarch64-linux-gnu
but unfortunately mesa's buildsystem refuses to cross-compile. It's
still a good idea to put the inputs in their proper categories, though.
One good heuristic is to run 'guix size raylib' and see which packages
are referenced. Speaking of which - alsa-lib doesn't show up in 'guix
size', and removing it from the inputs doesn't trigger any errors. Are
you sure it's being used? Maybe you're missing some configuration flag?
Also, the source for raylib includes its own copies of many libraries
which are already packaged separately in Guix, such as glfw or stb.
Could you configure raylib to use Guix-provided versions of these
libraries? A good way to make sure it works is to remove the bundled
sources. The preferred way of doing so is a 'snippet', you can find an
example in the rust-curl-sys-0.4 package in gnu/packages/crates-io.scm.
Regards,
Jakub Kądziołka
[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 833 bytes --]
^ permalink raw reply [flat|nested] 4+ messages in thread* [bug#42376] [PATCH] Add raylib.
2020-07-18 12:02 ` Jakub Kądziołka
@ 2020-09-24 15:45 ` Ludovic Courtès
2023-05-28 21:54 ` bug#42376: " Jelle Licht
0 siblings, 1 reply; 4+ messages in thread
From: Ludovic Courtès @ 2020-09-24 15:45 UTC (permalink / raw)
To: Jakub Kądziołka; +Cc: 42376, trymonv
Ping! :-)
Jakub Kądziołka <kuba@kadziolka.net> skribis:
> On Wed, Jul 15, 2020 at 04:25:21PM +0200, trymonv@cock.li wrote:
>> gnu/packages/game-development.scm | 34 +++++++++++++++++++++++++++++++
>> 1 file changed, 34 insertions(+)
>
> Thanks for the patch! Note that this blob got included as the commit
> message, which isn't really desirable. Instead of `git diff', I'd
> recommend `git commit', followed by `git format-patch' and/or `git
> send-email' (the latter is available as git:send-email - the send-email
> output of the git package).
>
>> diff --git a/gnu/packages/game-development.scm
>> b/gnu/packages/game-development.scm
>> index e45980e7a0..cba9b5bd45 100644
>> --- a/gnu/packages/game-development.scm
>> +++ b/gnu/packages/game-development.scm
>
> Could you add a Copyright line for yourself?
>
>> +(define-public raylib
>> + (package
>> + (name "raylib")
>> + (version "3.0.0")
>> + (source (origin
>> + (method git-fetch)
>> + (uri (git-reference
>> + (url "https://github.com/raysan5/raylib/")
>> + (commit version)))
>> + (file-name (git-file-name name version))
>> + (sha256
>> + (base32
>> + "1chj7sril4l2dxh7flp5ndddydbbf1fhnqlydaysm4m6waxidxmr"))))
>> + (build-system cmake-build-system)
>> + (arguments '(#:configure-flags `("-DSHARED=OFF" "-DSTATIC=ON")
>> + #:tests? #f))
>
> If you have to disable tests, put a small explanation in a comment.
> However, in this case I managed to get the test suite to pass like this:
>
> (arguments
> `(#:configure-flags `("-DSHARED=OFF" "-DSTATIC=ON")
> #:phases
> (modify-phases %standard-phases
> ;; check needs to run after install
> (delete 'check)
> (add-after 'install 'check (assoc-ref %standard-phases 'check))
> (add-before 'check 'set-CC
> (lambda _
> (setenv "CC" ,(cc-for-target))
> #t)))))
>
>> + (inputs `())
>> + (native-inputs `(("alsa-lib", alsa-lib)
>> + ("glu", glu)
>> + ("libx11", libx11)
>> + ("libxrandr", libxrandr)
>> + ("libxi", libxi)
>> + ("libxinerama", libxinerama)
>> + ("libxcursor", libxcursor)
>> + ("mesa", mesa)
>> + ("pkg-config", pkg-config)))
>
> I don't think these should be native-inputs. native-inputs are for
> things that are ran during compilation, like the compiler itself or
> pkg-config. The rest is clearly going to run at runtime, so they should
> go in inputs. This distinction is necessary for cross-compilation. If
> all your dependencies supported it, you could test it with
>
> guix build --target=aarch64-linux-gnu
>
> but unfortunately mesa's buildsystem refuses to cross-compile. It's
> still a good idea to put the inputs in their proper categories, though.
>
> One good heuristic is to run 'guix size raylib' and see which packages
> are referenced. Speaking of which - alsa-lib doesn't show up in 'guix
> size', and removing it from the inputs doesn't trigger any errors. Are
> you sure it's being used? Maybe you're missing some configuration flag?
>
> Also, the source for raylib includes its own copies of many libraries
> which are already packaged separately in Guix, such as glfw or stb.
> Could you configure raylib to use Guix-provided versions of these
> libraries? A good way to make sure it works is to remove the bundled
> sources. The preferred way of doing so is a 'snippet', you can find an
> example in the rust-curl-sys-0.4 package in gnu/packages/crates-io.scm.
>
> Regards,
> Jakub Kądziołka
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-05-28 21:55 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-15 14:25 [bug#42376] [PATCH] Add raylib trymonv
2020-07-18 12:02 ` Jakub Kądziołka
2020-09-24 15:45 ` Ludovic Courtès
2023-05-28 21:54 ` bug#42376: " Jelle Licht
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).