Re: [ptxdist] [PATCH v2] platsch: new package

[Roland Hieber <rhi@pengutronix.de>]

On Wed, Jan 22, 2025 at 11:45:52AM +0100, Rouven Czerwinski wrote:
> Add platsch, a simple splash screen application which uses libdrm during
> early boot to display a splash image and than hands over execution to
> /sbin/init. For advanced usecases liplatsch is installed as well.
> 
> Signed-off-by: Rouven Czerwinski <r.czerwinski@pengutronix.de>
> ---
> v2:
> - fix license information
>  rules/platsch.in   |  9 ++++++
>  rules/platsch.make | 68 ++++++++++++++++++++++++++++++++++++++++++++++
>  2 files changed, 77 insertions(+)
>  create mode 100644 rules/platsch.in
>  create mode 100644 rules/platsch.make
> 
> diff --git a/rules/platsch.in b/rules/platsch.in
> new file mode 100644
> index 000000000..afb169998
> --- /dev/null
> +++ b/rules/platsch.in
> @@ -0,0 +1,9 @@
> +## SECTION=multimedia_tools
> +
> +config PLATSCH
> +	tristate
> +	select HOST_MESON
> +	select LIBDRM
> +	prompt "platsch"
> +	help
> +	  Splash Screen Application using libdrm.
> diff --git a/rules/platsch.make b/rules/platsch.make
> new file mode 100644
> index 000000000..2bfc9ff20
> --- /dev/null
> +++ b/rules/platsch.make
> @@ -0,0 +1,68 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2025 by Rouven Czerwinski <r.czerwinski@pengutronix.de>
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_PLATSCH) += platsch
> +
> +#
> +# Paths and names
> +#
> +PLATSCH_VERSION	:= v2024.08.0
> +PLATSCH_MD5		:= f71c2c60acb8ef221f3cc5612e6a920e
> +PLATSCH		:= platsch-$(PLATSCH_VERSION)
> +PLATSCH_SUFFIX	:= tar.gz
> +PLATSCH_URL		:= https://github.com/pengutronix/platsch/archive/refs/tags//$(PLATSCH_VERSION).$(PLATSCH_SUFFIX)
> +PLATSCH_SOURCE	:= $(SRCDIR)/$(PLATSCH).$(PLATSCH_SUFFIX)
> +PLATSCH_DIR		:= $(BUILDDIR)/$(PLATSCH)
> +PLATSCH_LICENSE	:= 0BSD
> +PLATSCH_LICENSE_FILES	:= file://LICENSE;md5=1a4c8b8d288b4fc32c3c4830d7a5e169
> +
> +# ----------------------------------------------------------------------------
> +# Prepare
> +# ----------------------------------------------------------------------------
> +
> +PLATSCH_CONF_ENV	:= $(CROSS_ENV)
> +
> +#
> +# meson
> +#
> +PLATSCH_CONF_TOOL	:= meson
> +PLATSCH_CONF_OPT	:=  \

                           ^- superfluous space… :)

Other than that, looks good to me.

 - Roland

> +	$(CROSS_MESON_USR)
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/platsch.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, platsch)
> +	@$(call install_fixup, platsch,PRIORITY,optional)
> +	@$(call install_fixup, platsch,SECTION,base)
> +	@$(call install_fixup, platsch,AUTHOR,"Rouven Czerwinski <r.czerwinski@pengutronix.de>")
> +	@$(call install_fixup, platsch,DESCRIPTION,missing)
> +
> +	@$(call install_lib, platsch, 0, 0, 0755, libplatsch)
> +	@$(call install_copy, platsch, 0, 0, 0755, -, /usr/sbin/platsch)
> +
> +	@$(call install_finish, platsch)
> +
> +	@$(call touch)
> +
> +# ----------------------------------------------------------------------------
> +# Clean
> +# ----------------------------------------------------------------------------
> +
> +#$(STATEDIR)/platsch.clean:
> +#	@$(call targetinfo)
> +#	@$(call clean_pkg, PLATSCH)
> +
> +# vim: syntax=make
> -- 
> 2.39.5
> 
> 
> 

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