all messages for Guix-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
From: Tobias Geerinckx-Rice via Guix-patches via <guix-patches@gnu.org>
To: Nicolas Goaziou <mail@nicolasgoaziou.fr>
Cc: albertoefg@posteo.mx, 40035@debbugs.gnu.org
Subject: [bug#40035] Add widelands game
Date: Fri, 19 Jun 2020 14:09:42 +0200	[thread overview]
Message-ID: <871rmbw92x.fsf@nckx> (raw)
In-Reply-To: <87lfkk83ws.fsf@nicolasgoaziou.fr>

[-- Attachment #1: Type: text/plain, Size: 3142 bytes --]

Alberto, Nicolas,

Nicolas Goaziou 写道:
> I integrated the previous suggestions. I am not sure about the 
> version
> string. Upstream seems to use "build-20", and most distributions 
> use
> "20", some use "build20" or "b20". I used "build20" because it 
> makes it
> easier to build the download URL.

Does ‘build’ mean beta or (stable) preview here?  Then we should 
append ‘0.0.0-’ or similar, like we do for git snapshots, to the 
front in case upstream releases a 1.x at some point.

Otherwise, ‘20’'s fine, really.

> I'm sending the updated patch for review.

Thank you both!  I'm building it now.

> +       (patches
> +        ;; Use system Minizip.  This is discussed updstream at
> +        ;; <https://github.com/widelands/widelands/issues/399>.
> +        (search-patches 
> "widelands-system-wide_minizip.patch"))))

Minor typo (‘embeeded’).  Are you the ‘I’ in the patch?  Is it 
pending upstream application?  If so, a link to the upstream bug 
report/commit would be nice.

> +         (list "-DCMAKE_BUILD_TYPE=Release"

Why not the default RelWithDebInfo?  Please note in a comment.

> +       ("DejaVu" ,font-dejavu)          ;case matters in name
[…]
> +       ("MicroHei" ,font-wqy-microhei)  ;case matters in name

Mkay I guess.  No point in a codebase chock-full of repetitive 
alists if we're not going to take advantage of it once in a while 
:-)

I'd put both at the end with a single

  ;; Fonts for the ‘unbundle-fonts’ phase.  Case matters in name!

comment.

> +       ("minizip" ,minizip)
> +       ("sdl" ,(sdl-union (list sdl2 sdl2-image sdl2-mixer 
> sdl2-ttf)))
> +       ("zlib" ,zlib)))
> +    (home-page "https://www.widelands.org/")
> +    (synopsis "Fantasy real-time strategy game")
> +    (description
> +     "Widelands is a real-time strategy game with singleplayer 
> campaigns and
> +multiplayer mode, aiming for gameplay similar to Settlers II by 
> BlueByte.
> +
> +In this game, you start out on a small piece of land with 
> nothing more than
> +a few of useful resources.  Using those, you can build yourself 
> an empire with
> +many thousands of inhabitants.  On your way towards this goal, 
> you will have
> +to build up an economic infrastructure, explore the lands 
> around you and face
> +enemies who are trying to rule the world just like you do.")

I think Debian(-derived) descriptions are OK but I prefer 
yours/upstream's anyway.

> +    ;; Game is released as GPL2+.  Some parts, e.g., art, are 
> released under
> +    ;; different licenses.

At least Culmus and Sinhala are GPL2-only.  The title screen also 
says ‘GNU General Public License V2.0’.

> +    (license (list license:gpl2+
> +                   license:gpl3+

Which parts are gpl3+?  That sounds problematic.

> +                   license:expat
> +                   license:zlib
> +                   license:silofl1.1
> +                   license:asl2.0
> +                   license:cc-by-sa3.0))))

It's built now, I will test it extensively on the bus…

Thanks!

T G-R

[-- Attachment #2: signature.asc --]
[-- Type: application/pgp-signature, Size: 227 bytes --]

  parent reply	other threads:[~2020-06-19 12:10 UTC|newest]

Thread overview: 15+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2020-03-12  4:29 [bug#40035] Add widelands game Alberto EFG
2020-03-12 14:31 ` Nicolas Goaziou
2020-03-13  3:14   ` Alberto EFG
2020-03-13  8:16     ` Nicolas Goaziou
2020-06-18 21:22       ` Nicolas Goaziou
2020-06-18 22:58         ` Jonathan Brielmaier
2020-06-19  7:01           ` Nicolas Goaziou
2020-06-21  6:33           ` Efraim Flashner
2020-06-19 12:09         ` Tobias Geerinckx-Rice via Guix-patches via [this message]
2020-06-19 18:59           ` Nicolas Goaziou
2020-06-19 20:46             ` Nicolas Goaziou
2020-06-19 21:14             ` Tobias Geerinckx-Rice via Guix-patches via
2020-06-19 21:55               ` Nicolas Goaziou
2020-06-30 15:50                 ` bug#40035: " Nicolas Goaziou
2020-06-19 12:30 ` [bug#40035] " Tobias Geerinckx-Rice via Guix-patches via

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=871rmbw92x.fsf@nckx \
    --to=guix-patches@gnu.org \
    --cc=40035@debbugs.gnu.org \
    --cc=albertoefg@posteo.mx \
    --cc=mail@nicolasgoaziou.fr \
    --cc=me@tobias.gr \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
Code repositories for project(s) associated with this external index

	https://git.savannah.gnu.org/cgit/guix.git

This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.