[Buildroot] [PATCH 1/1] package/unifdef: add new package

[Peter Korsgaard]

>>>>> "Thomas" == Thomas Devoogdt <thomas at devoogdt.com> writes:

 > A small package that does "Selectively remove C preprocessor conditionals".
 > The package will be needed for the upcoming webkitgtk 2.40.0 release.
 > I personally added it to get started with 2.39.2.

 > See: https://github.com/WebKit/WebKit/commit/f76a7e30e5749af897b83cc75b2534b1afa32552
 > Signed-off-by: Thomas Devoogdt <thomas.devoogdt at barco.com>
 > ---
 >  .../0001-Makefile-fix-error-on-install.patch  | 31 +++++++++++++++++++
 >  package/unifdef/unifdef.hash                  |  3 ++
 >  package/unifdef/unifdef.mk                    | 20 ++++++++++++
 >  3 files changed, 54 insertions(+)
 >  create mode 100644 package/unifdef/0001-Makefile-fix-error-on-install.patch
 >  create mode 100644 package/unifdef/unifdef.hash
 >  create mode 100644 package/unifdef/unifdef.mk

 > +++ b/package/unifdef/unifdef.hash
 > @@ -0,0 +1,3 @@
 > +# Locally computed
 > +sha256  37ed46a39446b15b08af418fda23b75c27a769d5aad236d817ee7522cc9caeee  COPYING
 > +sha256  fba564a24db7b97ebe9329713ac970627b902e5e9e8b14e19e024eb6e278d10b  unifdef-2.12.tar.gz

NIT: We normally list the tarball before the license files.


> diff --git a/package/unifdef/unifdef.mk b/package/unifdef/unifdef.mk
 > new file mode 100644
 > index 0000000000..0107cd7096
 > --- /dev/null
 > +++ b/package/unifdef/unifdef.mk
 > @@ -0,0 +1,20 @@
 > +################################################################################
 > +#
 > +# unifdef
 > +#
 > +################################################################################
 > +
 > +UNIFDEF_VERSION = 2.12
 > +UNIFDEF_SITE = https://dotat.at/prog/unifdef
 > +UNIFDEF_LICENSE = BSD-2-Clause
 > +UNIFDEF_LICENSE_FILES = COPYING
 > +
 > +define HOST_UNIFDEF_BUILD_CMDS
 > +	$(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D)
 > +endef
 > +
 > +define HOST_UNIFDEF_INSTALL_CMDS
 > +	$(HOST_CONFIGURE_OPTS) $(MAKE) -C $(@D) HOME=$(HOST_DIR) install

HOME= sounds a bit odd. The Makefile just does prefix=${HOME} and the
INSTALL file only mentions setting prefix=, so I've changed it to use
that.

Committed with these minor changes, thanks.

-- 
Bye, Peter Korsgaard