On Thu, 13 Jun 2019 22:56:41 +0200
Nicolas Goaziou <mail@nicolasgoaziou.fr> wrote:

> Jesse Gibbons <jgibbons2357@gmail.com> writes:
> 
> > On Wed, 12 Jun 2019 21:48:52 +0200
> > Nicolas Goaziou <mail@nicolasgoaziou.fr> 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 will look into this later.
> 
> > 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.
Ok.
> 
> >> > +    (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.
No, i just forgot. It's fixed now, but 'guix lint' does not like it.
> 
> > 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.
I got my information from github. I'll update.
> 
> 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.
Fixed.
> 
> > +             (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?
Done.
> 
> > +    (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.
I did that when I added gnurobots.
> 
> Can you send an updated patch? Hopefully I should be able to apply it
> then.
Attached.
> 
> Thank you!