Re: [ptxdist] [PATCH] stress: add new package: workload generator for POSIX systems

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

On Mon, May 15, 2017 at 10:47:01AM +0200, Roland Hieber wrote:
> ---
>  patches/stress-1.0.4/0001-dont-build-docs.patch | 28 +++++++++++++
>  patches/stress-1.0.4/series                     |  1 +
>  rules/stress.in                                 |  9 +++++
>  rules/stress.make                               | 54 +++++++++++++++++++++++++
>  4 files changed, 92 insertions(+)
>  create mode 100644 patches/stress-1.0.4/0001-dont-build-docs.patch
>  create mode 100644 patches/stress-1.0.4/series
>  create mode 100644 rules/stress.in
>  create mode 100644 rules/stress.make
> 
> diff --git a/patches/stress-1.0.4/0001-dont-build-docs.patch b/patches/stress-1.0.4/0001-dont-build-docs.patch
> new file mode 100644
> index 0000000..bf73d16
> --- /dev/null
> +++ b/patches/stress-1.0.4/0001-dont-build-docs.patch
> @@ -0,0 +1,28 @@
> +Subject: disable building docs
> +From: Roland Hieber <rhi@pengutronix.de>
> +Forwarded: not-needed
> +
> +Building docs currently fails with errors in doc/stress.texi:
> +
> +	stress.texi:68: @itemx must follow @item
> +	stress.texi:87: @itemx must follow @item
> +	stress.texi:102: @itemx must follow @item
> +	stress.texi:105: @itemx must follow @item
> +	stress.texi:120: @itemx must follow @item
> +	stress.texi:123: @itemx must follow @item
> +	Makefile:191: recipe for target 'stress.info' failed
> +
> +
> +diff --git a/Makefile.in b/Makefile.in
> +index 64e5a39..2396f1c 100644
> +--- a/Makefile.in
> ++++ b/Makefile.in
> +@@ -155,7 +155,7 @@ MAINTAINERCLEANFILES = install-sh mkinstalldirs missing Makefile.in \
> +                        aclocal.m4 configure COPYING Log.cvs ChangeLog depcomp
> + 
> + EXTRA_DIST = ChangeLog
> +-SUBDIRS = . src doc test
> ++SUBDIRS = . src test
> + all: all-recursive
> + 
> + .SUFFIXES:
> diff --git a/patches/stress-1.0.4/series b/patches/stress-1.0.4/series
> new file mode 100644
> index 0000000..24d2f45
> --- /dev/null
> +++ b/patches/stress-1.0.4/series
> @@ -0,0 +1 @@
> +0001-dont-build-docs.patch

Please regenerate the patches with git:
- extract the packages with 'ptxdist --git extract stress'
- go to the package source dir and amend the commit if needed
- regenerate the patches with 'git ptx-patches'

> diff --git a/rules/stress.in b/rules/stress.in
> new file mode 100644
> index 0000000..8aabc44
> --- /dev/null
> +++ b/rules/stress.in
> @@ -0,0 +1,9 @@
> +## SECTION=test_suites
> +
> +config STRESS
> +	tristate
> +	prompt "stress"
> +	help
> +		stress is a deliberately simple workload generator for POSIX
> +		systems. It imposes a configurable amount of CPU, memory, I/O, and
> +		disk stress on the system.

Intend the help text with '<tab><space><space>'

> diff --git a/rules/stress.make b/rules/stress.make
> new file mode 100644
> index 0000000..a9e0213
> --- /dev/null
> +++ b/rules/stress.make
> @@ -0,0 +1,54 @@
> +# -*-makefile-*-
> +#
> +# Copyright (C) 2017 by Roland Hieber <r.hieber@pengutronix.de>
> +#
> +# 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_STRESS) += stress
> +
> +#
> +# Paths and names
> +#
> +STRESS_VERSION	:= 1.0.4
> +STRESS_MD5	:= 890a4236dd1656792f3ef9a190cf99ef
> +STRESS		:= stress-$(STRESS_VERSION)
> +STRESS_SUFFIX	:= tar.gz
> +STRESS_URL	:= https://people.seas.harvard.edu/~apw/stress/$(STRESS).$(STRESS_SUFFIX)
> +STRESS_SOURCE	:= $(SRCDIR)/$(STRESS).$(STRESS_SUFFIX)
> +STRESS_DIR	:= $(BUILDDIR)/$(STRESS)
> +STRESS_LICENSE	:= GPL-2

Use SPDX license identifiers and check the code. This should be GPL-2.0+

> +
> +#
> +# autoconf
> +#
> +STRESS_CONF_TOOL:= autoconf

STRESS_CONF_TOOL	:= autoconf

Indent to align with the line below.

> +STRESS_CONF_OPT	:= \

break.

	$(CROSS_AUTOCONF_USR) \
> +	--disable-dependency-tracking
> +
> +# ----------------------------------------------------------------------------
> +# Target-Install
> +# ----------------------------------------------------------------------------
> +
> +$(STATEDIR)/stress.targetinstall:
> +	@$(call targetinfo)
> +
> +	@$(call install_init, stress)
> +	@$(call install_fixup, stress,PRIORITY,optional)
> +	@$(call install_fixup, stress,SECTION,base)
> +	@$(call install_fixup, stress,AUTHOR,"Roland Hieber <r.hieber@pengutronix.de>")
> +	@$(call install_fixup, stress,DESCRIPTION,missing)
> +
> +	@$(call install_copy, stress, 0, 0, 0755, $(STRESS_DIR)/src/stress, /usr/bin/stress)

	@$(call install_copy, stress, 0, 0, 0755, -, /usr/bin/stress)

Michael

> +
> +	@$(call install_finish, stress)
> +
> +	@$(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