diff mbox series

[1/1] package/ptm2human: new package

Message ID 20200124182447.17295-1-juju@cotds.org
State Accepted
Headers show
Series [1/1] package/ptm2human: new package | expand

Commit Message

Julien Olivain Jan. 24, 2020, 6:24 p.m. UTC 
ptm2human is a decoder for trace data outputted by Program
Trace Macrocell (PTM) and Embedded Trace Macrocell (ETMv4).
It deconstructs ID packets and data packets from the
formatter of ARM Coresight ETB, and then translates the
trace data to a human-readable format.

./utils/test-pkg --package ptm2human --all
44 builds, 29 skipped, 0 build failed, 0 legal-info failed

Signed-off-by: Julien Olivain <juju@cotds.org>
---
 DEVELOPERS                                    |  1 +
 package/Config.in                             |  1 +
 package/Config.in.host                        |  1 +
 ...remove-unneeded-check-for-c-compiler.patch | 29 +++++++++++++++++++
 package/ptm2human/Config.in                   | 12 ++++++++
 package/ptm2human/Config.in.host              | 12 ++++++++
 package/ptm2human/ptm2human.hash              |  3 ++
 package/ptm2human/ptm2human.mk                | 14 +++++++++
 8 files changed, 73 insertions(+)
 create mode 100644 package/ptm2human/0001-configure.ac-remove-unneeded-check-for-c-compiler.patch
 create mode 100644 package/ptm2human/Config.in
 create mode 100644 package/ptm2human/Config.in.host
 create mode 100644 package/ptm2human/ptm2human.hash
 create mode 100644 package/ptm2human/ptm2human.mk

Comments

Yann E. MORIN Jan. 26, 2020, 3:08 p.m. UTC | #1 
Julien, All,

On 2020-01-24 19:24 +0100, Julien Olivain spake thusly:
> ptm2human is a decoder for trace data outputted by Program
> Trace Macrocell (PTM) and Embedded Trace Macrocell (ETMv4).
> It deconstructs ID packets and data packets from the
> formatter of ARM Coresight ETB, and then translates the
> trace data to a human-readable format.
> 
> ./utils/test-pkg --package ptm2human --all
> 44 builds, 29 skipped, 0 build failed, 0 legal-info failed
> 
> Signed-off-by: Julien Olivain <juju@cotds.org>

I've applied to msater, with the folowing chamges:

  - drop the host variant
  - introduce _ARCH_SUPPORTS
  - comment why we need autoreconf
  - drop spurious empty line

For the host variant, I dropped it because I don't see how this would be
usefull as a build-time tool. If you believe you have a good case for
it, please resend a patch adding the host variant with a commit log that
explains why it is nice to have it.

Regards,
Yann E. MORIN.
diff mbox series

Patch

diff --git a/DEVELOPERS b/DEVELOPERS
index a1eb052652..1c97b2564f 100644
--- a/DEVELOPERS
+++ b/DEVELOPERS
@@ -1384,6 +1384,7 @@  F:	configs/imx8mmpico_defconfig
 F:	configs/imx8mpico_defconfig
 F:	package/fluid-soundfont/
 F:	package/fluidsynth/
+F:	package/ptm2human/
 
 N:	Julien Viard de Galbert <julien@vdg.name>
 F:	package/dieharder/
diff --git a/package/Config.in b/package/Config.in
index 4c94914680..4c62841ffa 100644
--- a/package/Config.in
+++ b/package/Config.in
@@ -124,6 +124,7 @@  menu "Debugging, profiling and benchmark"
 	source "package/pax-utils/Config.in"
 	source "package/pcm-tools/Config.in"
 	source "package/piglit/Config.in"
+	source "package/ptm2human/Config.in"
 	source "package/pv/Config.in"
 	source "package/racehound/Config.in"
 	source "package/ramsmp/Config.in"
diff --git a/package/Config.in.host b/package/Config.in.host
index 62e860d7c3..f5d0c3daf9 100644
--- a/package/Config.in.host
+++ b/package/Config.in.host
@@ -52,6 +52,7 @@  menu "Host utilities"
 	source "package/patchelf/Config.in.host"
 	source "package/pkgconf/Config.in.host"
 	source "package/pru-software-support/Config.in.host"
+	source "package/ptm2human/Config.in.host"
 	source "package/pwgen/Config.in.host"
 	source "package/python/Config.in.host"
 	source "package/python-cython/Config.in.host"
diff --git a/package/ptm2human/0001-configure.ac-remove-unneeded-check-for-c-compiler.patch b/package/ptm2human/0001-configure.ac-remove-unneeded-check-for-c-compiler.patch
new file mode 100644
index 0000000000..b5c835a246
--- /dev/null
+++ b/package/ptm2human/0001-configure.ac-remove-unneeded-check-for-c-compiler.patch
@@ -0,0 +1,29 @@ 
+From 57b32b3785da1e48bd77c4f13ef41125a6abb83d Mon Sep 17 00:00:00 2001
+From: Julien Olivain <juju@cotds.org>
+Date: Fri, 24 Jan 2020 19:12:37 +0100
+Subject: [PATCH] configure.ac: remove unneeded check for c++ compiler
+
+If the c++ compiler is not present, the build configuration
+will fail. This patch remove the check for the c++ compiler
+because all the ptm2human code is written in C.
+
+Signed-off-by: Julien Olivain <juju@cotds.org>
+---
+ configure.ac | 1 -
+ 1 file changed, 1 deletion(-)
+
+diff --git a/configure.ac b/configure.ac
+index 3b1f337..924dc77 100644
+--- a/configure.ac
++++ b/configure.ac
+@@ -7,7 +7,6 @@ AM_INIT_AUTOMAKE([foreign -Wall])
+ AC_CONFIG_HEADERS([config.h])
+ 
+ # Checks for programs.
+-AC_PROG_CXX
+ AC_PROG_CC
+ 
+ # Checks for libraries.
+-- 
+2.24.1
+
diff --git a/package/ptm2human/Config.in b/package/ptm2human/Config.in
new file mode 100644
index 0000000000..4136082f3c
--- /dev/null
+++ b/package/ptm2human/Config.in
@@ -0,0 +1,12 @@ 
+config BR2_PACKAGE_PTM2HUMAN
+	bool "ptm2human"
+	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_aarch64_be
+	help
+
+	  ptm2human is a decoder for trace data outputted by Program
+	  Trace Macrocell (PTM) and Embedded Trace Macrocell
+	  (ETMv4). It deconstructs ID packets and data packets
+	  from the formatter of ARM Coresight ETB, and then
+	  translates the trace data to a human-readable format.
+
+	  https://github.com/hwangcc23/ptm2human
diff --git a/package/ptm2human/Config.in.host b/package/ptm2human/Config.in.host
new file mode 100644
index 0000000000..f5e73c2d9b
--- /dev/null
+++ b/package/ptm2human/Config.in.host
@@ -0,0 +1,12 @@ 
+config BR2_PACKAGE_HOST_PTM2HUMAN
+	bool "ptm2human"
+	depends on BR2_arm || BR2_armeb || BR2_aarch64 || BR2_aarch64_be
+	help
+
+	  ptm2human is a decoder for trace data outputted by Program
+	  Trace Macrocell (PTM) and Embedded Trace Macrocell
+	  (ETMv4). It deconstructs ID packets and data packets
+	  from the formatter of ARM Coresight ETB, and then
+	  translates the trace data to a human-readable format.
+
+	  https://github.com/hwangcc23/ptm2human
diff --git a/package/ptm2human/ptm2human.hash b/package/ptm2human/ptm2human.hash
new file mode 100644
index 0000000000..b4372f77f6
--- /dev/null
+++ b/package/ptm2human/ptm2human.hash
@@ -0,0 +1,3 @@ 
+# Computed locally
+sha256 c3fd505949db82fde4f058cad0b78c8e96ef620c193dfff8acdbbd020d51fa1a  ptm2human-d7dd68ea6495daef50e00ef2d65c99810e0a594f.tar.gz
+sha256 ab15fd526bd8dd18a9e77ebc139656bf4d33e97fc7238cd11bf60e2b9b8666c6  LICENSE
diff --git a/package/ptm2human/ptm2human.mk b/package/ptm2human/ptm2human.mk
new file mode 100644
index 0000000000..086a64338f
--- /dev/null
+++ b/package/ptm2human/ptm2human.mk
@@ -0,0 +1,14 @@ 
+################################################################################
+#
+# ptm2human
+#
+################################################################################
+
+PTM2HUMAN_VERSION = d7dd68ea6495daef50e00ef2d65c99810e0a594f
+PTM2HUMAN_SITE = $(call github,hwangcc23,ptm2human,$(PTM2HUMAN_VERSION))
+PTM2HUMAN_LICENSE = GPL-2.0
+PTM2HUMAN_LICENSE_FILES = LICENSE
+PTM2HUMAN_AUTORECONF = YES
+
+$(eval $(autotools-package))
+$(eval $(host-autotools-package))