Subject: [bug#36182] [PATCH] add freeorion

From: Nicolas Goaziou <[email-masked]>
Date: Thu, 13 Jun 2019 22:56:41 +0200

Jesse Gibbons <[email-masked]> writes:

> On Wed, 12 Jun 2019 21:48:52 +0200
> Nicolas Goaziou <[email-masked]> wrote:

>> If you use Emacs, there is a template to automatically fill this.
> I did not know this. How do I use the template?

See "The Perfect Setup" section of the Guix manual, in particular the
part about Yasnippet.

> I added a comment to explain why I specify the most recent commit. I
> also added a note that it should be updated when the next stable
> release is available (the maintainers seem to prefer announcing new
> releases every September). I will personally update this in
> mid-September or October if version 0.4.8.1 or 0.4.9 or 0.5 or 1.0 is
> available and no volunteer beats me to it. If you want I can also
> request release 0.4.8.1 ASAP so we can specify a release rather than a
> commit. I personally prefer to keep the commit and wait three or four
> months for the release.

Fair enough. Thank you for the explanation.

I think we also need a

  (file-name (git-file-name name version))

in the source.

>> > +    (home-page
>> > +     "https://www.freeorion.org/index.php/Main_Page")  
>> 
>> I think "https://www.freeorion.org" is enough, since it points to the
>> page above.

You probably overlooked that remark.

> The link says that 4X refers to a genre of strategy game centered
> around conquest. I think the current context is a sufficient
> explanation, but I did expand the description. I think it is best to
> keep the 4X in the description in case someone tries a search for that
> genre.

OK.

> I added comments to explain these licenses.

According to
<https://www.freeorion.org/index.php/FreeOrionWiki:Copyrights>, the
source code is released under gpl2, artwork, music and sounds, and
in-game text are released under cc-by-sa3.0, and the game content
scripts are released under both gpl2 and cc-by-sa3.0.

So, the comments should include the description above, for completeness.

> Updated patch is attached. Is there anything else you want me to fix?

Great. I'm adding some small remarks, but overall the patch LGTM.

> +;;;Most recent stable release uses boost_signals (v1)
> +;;;which was later replaced with boost-signals2 and
> +;;;no longer exists. This commit builds and runs.
> +;;;TODO: Update this when the next stable release when
> +;;;it is available.

In Scheme, such comments use ";; " as a prefix.

> +             (commit "470d0711537804df3c2ca25532f674ab4bec58af")))
> +       (sha256
> +        (base32
> +         "1wsw632l1cj17px6i88nqjzs0dngp5rsr67n6qkkjlfjfxi69j0f"))))
> +    (arguments
> +     '(#:tests? #f))                    ;no test
> +    (build-system cmake-build-system)
> +    (inputs `(("boost" ,boost)
> +              ("boost_signals" ,boost-signals2)
> +              ("freetype2" ,freetype)
> +              ("python2" ,python-2.7)
> +              ("openal" ,openal)
> +              ("libogg" ,libogg)
> +              ("libvorbis" ,libvorbis)
> +              ("zlib" ,zlib)
> +              ("libpng" ,libpng)
> +              ("sdl2" ,sdl2)
> +              ("glu" ,glu)
> +              ("glew" ,glew)))

Could you re-order inputs alphabetically?

> +    (home-page
> +     "https://www.freeorion.org/index.php/Main_Page")

See above.

Also, don't forget to add yourself to the Hall of Fame at the top of the
"games.scm" file.

Can you send an updated patch? Hopefully I should be able to apply it
then.

Thank you!