On Mon, Oct 09, 2017 at 07:02:36PM +0200, Bastian Stender wrote:
> Extended beep utility that does pretty much the same as busybox' beep,
> but allows reading stdin and specifying an alternative console device.
> 
> This is especially useful to test the kernel drivers "pwm-beeper" and
> "gpio-beeper". To do so enable CONFIG_INPUT_EVDEV and specify the input
> device with -e.
> 
> Signed-off-by: Bastian Stender <b...@pengutronix.de>
> ---
>  patches/beep-1.3/0001-Makefile-fix.patch | 33 +++++++++++++++++++
>  rules/beep.in                            |  7 ++++
>  rules/beep.make                          | 56 
> ++++++++++++++++++++++++++++++++
>  3 files changed, 96 insertions(+)
>  create mode 100644 patches/beep-1.3/0001-Makefile-fix.patch
>  create mode 100644 rules/beep.in
>  create mode 100644 rules/beep.make
> 
> diff --git a/patches/beep-1.3/0001-Makefile-fix.patch 
> b/patches/beep-1.3/0001-Makefile-fix.patch
> new file mode 100644
> index 000000000..5802a29ab
> --- /dev/null
> +++ b/patches/beep-1.3/0001-Makefile-fix.patch
> @@ -0,0 +1,33 @@
> +From: Bastian Stender <b...@pengutronix.de>
> +Date: Mon, 9 Oct 2017 18:30:48 +0200
> +Subject: [PATCH] Makefile: fix
> +
> +Signed-off-by: Bastian Stender <b...@pengutronix.de>
> +---
> + Makefile | 11 ++++-------
> + 1 file changed, 4 insertions(+), 7 deletions(-)
> +
> +diff --git a/Makefile b/Makefile
> +index cc359c4843e4..fa8a7e3fde04 100644
> +--- a/Makefile
> ++++ b/Makefile
> +@@ -5,15 +5,12 @@ INSTALL_DIR=/usr/bin
> + MAN_FILE=beep.1.gz
> + MAN_DIR=/usr/man/man1
> + 
> +-default : beep
> ++all :
> ++    ${CC} ${FLAGS} -o ${EXEC_NAME} beep.c
> + 
> + clean : 
> +     rm ${EXEC_NAME}
> + 
> +-beep : beep.c
> +-    ${CC} ${FLAGS} -o ${EXEC_NAME} beep.c
> +-

Why is this necessary? 'make' without target argument builds the first
target, so that should do what we need.

> + install : 
> +-    cp ${EXEC_NAME} ${INSTALL_DIR}
> +-    # rm -f /usr/man/man1/beep.1.bz2
> +-    cp ${MAN_FILE} ${MAN_DIR}
> ++    cp ${EXEC_NAME} ${DESTDIR}/${INSTALL_DIR}
> ++    cp ${MAN_FILE} ${DESTDIR}/${MAN_DIR}

You could just overwrite INSTALL_DIR and MAN_DIR in the arguments, but I
don't care either way.

> diff --git a/rules/beep.in b/rules/beep.in
> new file mode 100644
> index 000000000..4b41c613a
> --- /dev/null
> +++ b/rules/beep.in
> @@ -0,0 +1,7 @@
> +## SECTION=project_specific
> +
> +config BEEP
> +     tristate
> +     prompt "beep"
> +     help
> +       Extended beep utility
> diff --git a/rules/beep.make b/rules/beep.make
> new file mode 100644
> index 000000000..74931ca4c
> --- /dev/null
> +++ b/rules/beep.make
> @@ -0,0 +1,56 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2017 by Johnathan Nightingale <john...@johnath.com>
> +#
> +# See CREDITS for details about who has contributed to this project.
> +#
> +# For further information about the PTXdist project and license conditions
> +# see the README file.
> +#
> +
> +#
> +# We provide this package
> +#
> +PACKAGES-$(PTXCONF_BEEP) += beep
> +
> +#
> +# Paths and names
> +#
> +BEEP_VERSION := 1.3
> +BEEP_MD5     := 49c340ceb95dbda3f97b2daafac7892a
> +BEEP         := beep-$(BEEP_VERSION)
> +BEEP_SUFFIX  := tar.gz
> +BEEP_URL     := http://www.johnath.com/beep/$(BEEP).$(BEEP_SUFFIX)
> +BEEP_SOURCE  := $(SRCDIR)/$(BEEP).$(BEEP_SUFFIX)
> +BEEP_DIR     := $(BUILDDIR)/$(BEEP)
> +BEEP_LICENSE := GPL-2.0
> +
> +# 
> ----------------------------------------------------------------------------
> +# Prepare
> +# 
> ----------------------------------------------------------------------------
> +
> +BEEP_CONF_TOOL       := NO

> +BEEP_PATH    := PATH=$(CROSS_PATH)
> +BEEP_ENV     := $(CROSS_ENV)

not needed

> +BEEP_MAKEVARS        := CC=$(CROSS_CC)

<PKG>_MAKEVARS is deprecated use <PKG>_MAKE_OPT / <PKG>_INSTALL_OPT

> +
> +# 
> ----------------------------------------------------------------------------
> +# Target-Install
> +# 
> ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/beep.targetinstall:
> +     @$(call targetinfo)
> +
> +     @$(call install_init, beep)
> +     @$(call install_fixup, beep,PRIORITY,optional)
> +     @$(call install_fixup, beep,SECTION,base)
> +     @$(call install_fixup, beep,AUTHOR,"Johnathan Nightingale 
> <john...@johnath.com>")

As Alexander said, you are the ptxdist package author.


Michael

> +     @$(call install_fixup, beep,DESCRIPTION,missing)
> +
> +     @$(call install_copy, beep, 0, 0, 0755, -, /usr/bin/beep)
> +
> +     @$(call install_finish, beep)
> +
> +     @$(call touch)
> +
> +# vim: syntax=make
> -- 
> 2.11.0
> 
> 
> _______________________________________________
> ptxdist mailing list
> ptxdist@pengutronix.de

-- 
Pengutronix e.K.                           |                             |
Industrial Linux Solutions                 | http://www.pengutronix.de/  |
Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0    |
Amtsgericht Hildesheim, HRA 2686           | Fax:   +49-5121-206917-5555 |

_______________________________________________
ptxdist mailing list
ptxdist@pengutronix.de

Reply via email to