guix-patches
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

[bug#67134] [PATCH] gnu: Add camelia-girls


From: hrn
Subject: [bug#67134] [PATCH] gnu: Add camelia-girls
Date: Wed, 15 Nov 2023 20:07:21 +0000

Thanks for reviewing!

   +       (snippet '(begin

Use #~ instead of ': The former also allows you to pull in other
packages, should you ever need to.

Ok I'll test.

   +     `(#:phases (modify-phases %standard-phases

Same as above, prefer (list #:phases #~(...))

Ok will try, I thought it was the other way around.

+ (font-path-bi (string-append (assoc-ref
   inputs
   +                                                           "font-
   dejavu")
+ "/share/fonts/truetype/"
   +                                            font-name-bi))

Prefer search-input-file.

Ok, sounds better.

   +                        (symlink font-path-bi target-bi))))

Instead of symlinking, hard-code the file name where it's used?

Symlinking avoids changing the source files. Isn't that better?

We have a dedicated procedure for generating .desktop files.  Use that.

I would have used that if it wasn't for the builder already having created the desktop file. I didn't want to change the builder so I ended up with this solution. In case the builder can do more automatically in the future we can remove the extra code. Or do you want me to overwrite the desktop file created by the builder entirely?


The credits are, as far as I can see, a little unclear about some
included assets – I only see some Wikimedia links, and that's it.  I'd
be happy to be proven wrong, though, we desperately need a package to
make use of renpy-build-system :)

I checked the licenses to the best of my best ability from the information in license.txt. The in game credits with more detailed license information can be found in the
games/scripts/credits?.txt.





reply via email to

[Prev in Thread] Current Thread [Next in Thread]