[1/3] package/xorcurses: new package

From: Thomas Huth <thuth@redhat.com>

XorCurses is a remake of the 8-bit game 'Xor' by Astral Software.
Your task is to roam around a series of mazes where you have to collect
all blue masks before finding the exit. You have two 'shields' (players)
and you can use either one at any time and switch between them. While
the first level is simply a matter of navigation, the following levels
introduce further objects like bombs and teleports, which have to be
used right to solve the puzzles.

Signed-off-by: Thomas Huth <huth@tuxfamily.org>
---
 package/Config.in                |  1 +
 package/xorcurses/Config.in      | 13 +++++++++++++
 package/xorcurses/xorcurses.hash |  2 ++
 package/xorcurses/xorcurses.mk   | 25 +++++++++++++++++++++++++
 4 files changed, 41 insertions(+)
 create mode 100644 package/xorcurses/Config.in
 create mode 100644 package/xorcurses/xorcurses.hash
 create mode 100644 package/xorcurses/xorcurses.mk

Hello Thomas,

Thanks for your contribution. Please find some comments below.

Le sam. 23 févr. 2019 à 21:57, Thomas Huth <huth@tuxfamily.org> a écrit :
>
> From: Thomas Huth <thuth@redhat.com>
>
> XorCurses is a remake of the 8-bit game 'Xor' by Astral Software.
> Your task is to roam around a series of mazes where you have to collect
> all blue masks before finding the exit. You have two 'shields' (players)
> and you can use either one at any time and switch between them. While
> the first level is simply a matter of navigation, the following levels
> introduce further objects like bombs and teleports, which have to be
> used right to solve the puzzles.
>
> Signed-off-by: Thomas Huth <huth@tuxfamily.org>
> ---
>  package/Config.in                |  1 +
>  package/xorcurses/Config.in      | 13 +++++++++++++
>  package/xorcurses/xorcurses.hash |  2 ++
>  package/xorcurses/xorcurses.mk   | 25 +++++++++++++++++++++++++
You also need to create an entry in the DEVELOPERS file for your name
and this package.

>  4 files changed, 41 insertions(+)
>  create mode 100644 package/xorcurses/Config.in
>  create mode 100644 package/xorcurses/xorcurses.hash
>  create mode 100644 package/xorcurses/xorcurses.mk
>
> diff --git a/package/Config.in b/package/Config.in
> index cc232b9fba..490cbdb216 100644
> --- a/package/Config.in
> +++ b/package/Config.in
> @@ -258,6 +258,7 @@ menu "Games"
>         source "package/stella/Config.in"
>         source "package/supertux/Config.in"
>         source "package/supertuxkart/Config.in"
> +       source "package/xorcurses/Config.in"
>  endmenu
>
>  menu "Graphic libraries and applications (graphic/text)"
> diff --git a/package/xorcurses/Config.in b/package/xorcurses/Config.in
> new file mode 100644
> index 0000000000..0af161fec9
> --- /dev/null
> +++ b/package/xorcurses/Config.in
> @@ -0,0 +1,13 @@
> +config BR2_PACKAGE_XORCURSES
> +    bool "xorcurses"
> +    select BR2_PACKAGE_NCURSES
> +    help
> +      XorCurses is a remake of the 8-bit game 'Xor' by Astral Software.
> +      Your task is to roam around a series of mazes where you have to collect
> +      all blue masks before finding the exit. You have two 'shields' (players)
> +      and you can use either one at any time and switch between them. While
> +      the first level is simply a matter of navigation, the following levels
> +      introduce further objects like bombs and teleports, which have to be
> +      used right to solve the puzzles.
> +      Note that the game needs a terminal with colors, i.e. make sure that
> +      the TERM environment variable is set to "linux" ("vt100" won't work).
There are some coding style issues in this file. Please run the
following command:
./utils/check-package package/xorcurses/*
to fix the warnings.

You also need to mention the upstream URL of this package as part of
the help text.
You might not need to have a help text that long. Just the first line
should be enough.

> diff --git a/package/xorcurses/xorcurses.hash b/package/xorcurses/xorcurses.hash
> new file mode 100644
> index 0000000000..49d3a0c3b9
> --- /dev/null
> +++ b/package/xorcurses/xorcurses.hash
> @@ -0,0 +1,2 @@
> +# Locally calculated
> +sha512 1782ebf6ae8798b32e0a8e5e20e6d0b44eb619f6e7f015735c6e86205bd545da108c36bff1c9cfc9e9b44dee8f90bacb9df70436991f57f51c2c87e94b3a4725  XorCurses-0.2.2.tar.bz2
You also need to provide the hash for the license file.


> diff --git a/package/xorcurses/xorcurses.mk b/package/xorcurses/xorcurses.mk
> new file mode 100644
> index 0000000000..4bc132c5e8
> --- /dev/null
> +++ b/package/xorcurses/xorcurses.mk
> @@ -0,0 +1,25 @@
> +################################################################################
> +#
> +# XorCurses
> +#
> +################################################################################
> +
> +XORCURSES_VERSION = 0.2.2
> +XORCURSES_LICENSE = GPL-3.0
> +XORCURSES_SOURCE = XorCurses-$(XORCURSES_VERSION).tar.bz2
> +XORCURSES_SITE = http://www.jwm-art.net/art/archive
> +XORCURSES_INSTALL_TARGET = YES
this is the default value, so not needed.

> +XORCURSES_DEPENDENCIES = ncurses
> +
> +define XORCURSES_BUILD_CMDS
> +       $(MAKE) -C $(@D) $(TARGET_CONFIGURE_OPTS) \
> +               CFLAGS="-O2 -DDATADIR='\"/usr/share/xorcurses\"' -DVERSION='\"$(XORCURSES_VERSION)\"'"
You mght want to keep the default CFLAGS by doing this
CFLAGS="$(TARGET_CFLAGS) -O2 ...."

> +endef
> +
> +define XORCURSES_INSTALL_TARGET_CMDS
> +       $(INSTALL) -D -m 0755 $(@D)/xorcurses $(TARGET_DIR)/usr/bin/
It should be $(INSTALL) -D -m 0755 $(@D)/xorcurses
$(TARGET_DIR)/usr/bin/xorcurses

> +       mkdir -p $(TARGET_DIR)/usr/share/xorcurses/maps
> +       $(INSTALL) -D -m 0755 $(@D)/maps/*.xcm $(TARGET_DIR)/usr/share/xorcurses/maps/
If these are not executable files, it would be better to change their
access permission mode.

> +endef
> +
> +$(eval $(generic-package))
> --
> 2.17.2
>
> _______________________________________________
> buildroot mailing list
> buildroot@busybox.net
> http://lists.busybox.net/mailman/listinfo/buildroot

As a rule of thumb, it is recommended to test your package with the
test-pkg tool. This will allow you to spot some issues when building
in different configurations.
Please look into the buildroot user manual, under the "How to test
your package" for more details.

Thanks!
Gilles.

Am Sat, 16 Mar 2019 17:57:28 +0100
schrieb Gilles Talis <gilles.talis@gmail.com>:

> Hello Thomas,
> 
> Thanks for your contribution. Please find some comments below.

 Hi Gilles,

thanks a lot for your review! I finally found some spare time to look
into this again, but I have some questions below...

> Le sam. 23 févr. 2019 à 21:57, Thomas Huth <huth@tuxfamily.org> a
> écrit :
[...]
> > diff --git a/package/xorcurses/xorcurses.hash
> > b/package/xorcurses/xorcurses.hash new file mode 100644
> > index 0000000000..49d3a0c3b9
> > --- /dev/null
> > +++ b/package/xorcurses/xorcurses.hash
> > @@ -0,0 +1,2 @@
> > +# Locally calculated
> > +sha512
> > 1782ebf6ae8798b32e0a8e5e20e6d0b44eb619f6e7f015735c6e86205bd545da108c36bff1c9cfc9e9b44dee8f90bacb9df70436991f57f51c2c87e94b3a4725
> > XorCurses-0.2.2.tar.bz2  
> You also need to provide the hash for the license file.

There does not seem to be a license file in the tarball here, they just
mention in the README that the game is licensed under the GPL. I guess
I can skip the hash for the license file in this case?

 Thomas

Hello Thomas,


Le lun. 22 avr. 2019 à 07:56, Thomas Huth <huth@tuxfamily.org> a écrit :
>
> Am Sat, 16 Mar 2019 17:57:28 +0100
> schrieb Gilles Talis <gilles.talis@gmail.com>:
>
> > Hello Thomas,
> >
> > Thanks for your contribution. Please find some comments below.
>
>  Hi Gilles,
>
> thanks a lot for your review! I finally found some spare time to look
> into this again, but I have some questions below...
All apologies for not coming back sooner on your patch.


> There does not seem to be a license file in the tarball here, they just
> mention in the README that the game is licensed under the GPL. I guess
> I can skip the hash for the license file in this case?
This is correct. If there's no actual license file, you can skip the
hash for it.

thanks
Gilles.