Re: [ptxdist] [PATCH 1/3] hiredis: new package

[Michael Olbrich <m.olbrich@pengutronix.de>]

On Fri, Feb 04, 2022 at 05:28:07PM +0100, Clemens Gruber wrote:
> Adds a new package for hiredis, a minimalistic C client library for
> Redis.
> 
> Signed-off-by: Clemens Gruber <clemens.gruber@pqgruber.com>
> ---
>  rules/hiredis.in   |  8 ++++++
>  rules/hiredis.make | 61 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 69 insertions(+)
>  create mode 100644 rules/hiredis.in
>  create mode 100644 rules/hiredis.make
> 
> diff --git a/rules/hiredis.in b/rules/hiredis.in
> new file mode 100644
> index 000000000..d416d69c4
> --- /dev/null
> +++ b/rules/hiredis.in
> @@ -0,0 +1,8 @@
> +## SECTION=system_libraries
> +
> +config HIREDIS
> +	tristate
> +	select HOST_CMAKE
> +	prompt "hiredis                       "
> +	help
> +	  Hiredis is a minimalistic C client library for the Redis database.
> diff --git a/rules/hiredis.make b/rules/hiredis.make
> new file mode 100644
> index 000000000..10e6d81d2
> --- /dev/null
> +++ b/rules/hiredis.make
> @@ -0,0 +1,61 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2022 by Clemens Gruber <clemens.gruber@pqgruber.com>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_HIREDIS) += hiredis
> +
> +#
> +# Paths and names
> +#
> +HIREDIS_VERSION	:= 1.0.2
> +HIREDIS_MD5	:= 58e8313188f66ed1be1c220d14a7752e
> +HIREDIS		:= hiredis-$(HIREDIS_VERSION)
> +HIREDIS_SUFFIX	:= tar.gz
> +HIREDIS_URL	:= https://github.com/redis/hiredis/archive/v$(HIREDIS_VERSION).$(HIREDIS_SUFFIX)
> +HIREDIS_SOURCE	:= $(SRCDIR)/$(HIREDIS).$(HIREDIS_SUFFIX)
> +HIREDIS_DIR	:= $(BUILDDIR)/$(HIREDIS)
> +HIREDIS_LICENSE	:= BSD-3-Clause

Please add _LICENSE_FILES as well.

> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +#
> +# cmake
> +#
> +HIREDIS_CONF_TOOL	:= cmake
> +
> +HIREDIS_CONF_OPT	:= \
> +	$(CROSS_CMAKE_USR) \
> +	-DDISABLE_TESTS=ON \
> +	-DENABLE_SSL=OFF
> +
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/hiredis.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, hiredis)
> +	@$(call install_fixup, hiredis,PRIORITY,optional)
> +	@$(call install_fixup, hiredis,SECTION,base)
> +	@$(call install_fixup, hiredis,AUTHOR,"Clemens Gruber <clemens.gruber@pqgruber.com>")
> +	@$(call install_fixup, hiredis,DESCRIPTION,"Minimalistic C client for Redis")
> +
> +	@$(call install_lib, hiredis, 0, 0, 0644, libhiredis)
> +	@$(call install_link, hiredis, libhiredis.so.1.0.0, /usr/lib/libhiredis.so)

In general, this link should not be needed. If it really is needed then
please add a comment why.

Michael

> +
> +	@$(call install_finish, hiredis)
> +
> +	@$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.35.1
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de
> To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de
> 

-- 
Pengutronix e.K.                           |                             |
Steuerwalder Str. 21                       | http://www.pengutronix.de/  |
31137 Hildesheim, Germany                  | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de
To unsubscribe, send a mail with subject "unsubscribe" to ptxdist-request@pengutronix.de