[Buildroot] [PATCH 1/1] dacapo: new package
Thomas Petazzoni
thomas.petazzoni at bootlin.com
Sun Apr 14 13:42:25 UTC 2019
Hello Daniel,
On Mon, 18 Feb 2019 21:55:03 +0000
"Leach, Daniel J." <dleach at belcan.com> wrote:
> The DaCapo benchmark suite is intended as a tool for Java benchmarking
> by the programming language, memory management and computer architecture
> communities.
>
> Depends on the OpenJDK package for Java runtime environment.
>
> Signed-off-by: Daniel J. Leach <dleach at belcan.com>
Thanks for your contribution, I have applied after doing a few changes.
First of all, your patch didn't apply as-is, it was probably mangled by
your e-mail client. Please try to use "git send-email" to send patches
in the future.
> diff --git a/DEVELOPERS b/DEVELOPERS
> index baa325eead..a64a671b56 100644
> --- a/DEVELOPERS
> +++ b/DEVELOPERS
> @@ -1405,6 +1405,7 @@ F: package/checksec/
> F: package/cgroupfs-mount/
> F: package/crda/
> F: package/cunit/
> +F: package/dacapo
Why was this added in the middle of Matt Weber entries ? It should have
been associated to your name, since you are the one contributing
dacapo. I fixed that up.
> diff --git a/package/dacapo/dacapo.mk b/package/dacapo/dacapo.mk
> new file mode 100644
> index 0000000000..32d0cc0701
> --- /dev/null
> +++ b/package/dacapo/dacapo.mk
> @@ -0,0 +1,21 @@
> +################################################################################
> +#
> +# dacapo
> +#
> +################################################################################
> +
> +DACAPO_VERSION = 9.12-MR1-bach
> +DACAPO_SOURCE = dacapo-$(DACAPO_VERSION).jar
> +DACAPO_SITE = http://sourceforge.net/projects/dacapobench/files/9.12-bach-MR1
> +DACAPO_LICENSE = Apache-2.0
> +DACAPO_LICENSE_FILES = LICENSE
> +
> +define DACAPO_EXTRACT_CMDS
> + unzip $(DACAPO_DL_DIR)/$(DACAPO_SOURCE) LICENSE -d $(@D)
> +endef
> +
> +define DACAPO_INSTALL_TARGET_CMDS
> + $(INSTALL) -D -m 755 $(DACAPO_DL_DIR)/$(DACAPO_SOURCE) $(TARGET_DIR)/usr/bin/
The destination path should include the filename, so I also fixed that.
Applied with those two issues fixed. Thanks!
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com
More information about the buildroot
mailing list