diff mbox series

arm/patrace: Add new package

Message ID 20240403163814.1978415-1-tom.hochstein@nxp.com
State New
Headers show
Series arm/patrace: Add new package | expand

Commit Message

Tom Hochstein April 3, 2024, 4:38 p.m. UTC 
From: Tom Hochstein <tom.hochstein@oss.nxp.com>

Signed-off-by: Tom Hochstein <tom.hochstein@oss.nxp.com>
---
 ...001-CMakeLists.txt-Switch-to-python3.patch | 27 ++++++++++++++
 .../recipes-devtools/patrace/patrace_5.1.0.bb | 36 +++++++++++++++++++
 2 files changed, 63 insertions(+)
 create mode 100644 meta-arm/recipes-devtools/patrace/patrace/0001-CMakeLists.txt-Switch-to-python3.patch
 create mode 100644 meta-arm/recipes-devtools/patrace/patrace_5.1.0.bb

Comments

Ross Burton April 10, 2024, 1:36 p.m. UTC | #1 
On 3 Apr 2024, at 17:38, Tom Hochstein via lists.yoctoproject.org <tom.hochstein=nxp.com@lists.yoctoproject.org> wrote:
> +++ b/meta-arm/recipes-devtools/patrace/patrace_5.1.0.bb
> @@ -0,0 +1,36 @@
> +SUMMARY = "Software for capturing GLES calls of an application"
> +DESCRIPTION = "\
> +PATrace is software for capturing GLES calls of an application and \
> +replaying them on a different device, keeping the GPU workload the \
> +same. It's similar to the open source Apitrace project, but optimised \
> +for performance measurements."

Is this actually Arm-specific?  If it’s not then it feels like meta-oe is a better home for it.  If it is, then it needs a COMPATIBLE_MACHINE or similar to stop it building on platforms that are not supported.

> +    -DCMAKE_INSTALL_PREFIX:PATH=/opt/${BPN}”

Is there a good reason why this can’t be installed into $prefix?  If so, then a quick comment would be good.

Ross
diff mbox series

Patch

diff --git a/meta-arm/recipes-devtools/patrace/patrace/0001-CMakeLists.txt-Switch-to-python3.patch b/meta-arm/recipes-devtools/patrace/patrace/0001-CMakeLists.txt-Switch-to-python3.patch
new file mode 100644
index 00000000..8504443a
--- /dev/null
+++ b/meta-arm/recipes-devtools/patrace/patrace/0001-CMakeLists.txt-Switch-to-python3.patch
@@ -0,0 +1,27 @@ 
+From c6946a8f96a73fadf4ad1bc9b28cf5568feba473 Mon Sep 17 00:00:00 2001
+From: Tom Hochstein <tom.hochstein@oss.nxp.com>
+Date: Mon, 22 Jan 2024 17:34:17 -0600
+Subject: [PATCH] CMakeLists.txt: Switch to python3
+
+Upstream-Status: Pending
+Signed-off-by: Tom Hochstein <tom.hochstein@oss.nxp.com>
+---
+ patrace/project/cmake/CMakeLists.txt | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/patrace/project/cmake/CMakeLists.txt b/patrace/project/cmake/CMakeLists.txt
+index 8704beb..3edd4a9 100644
+--- a/patrace/project/cmake/CMakeLists.txt
++++ b/patrace/project/cmake/CMakeLists.txt
+@@ -1,7 +1,7 @@
+ cmake_minimum_required (VERSION 2.8.4)
+ 
+ project (patrace)
+-set (PYTHON_EXECUTABLE python)
++set (PYTHON_EXECUTABLE python3)
+ 
+ message(STATUS "main ar: ${CMAKE_AR}")
+ 
+-- 
+2.34.1
+
diff --git a/meta-arm/recipes-devtools/patrace/patrace_5.1.0.bb b/meta-arm/recipes-devtools/patrace/patrace_5.1.0.bb
new file mode 100644
index 00000000..c95b0691
--- /dev/null
+++ b/meta-arm/recipes-devtools/patrace/patrace_5.1.0.bb
@@ -0,0 +1,36 @@ 
+SUMMARY = "Software for capturing GLES calls of an application"
+DESCRIPTION = "\
+PATrace is software for capturing GLES calls of an application and \
+replaying them on a different device, keeping the GPU workload the \
+same. It's similar to the open source Apitrace project, but optimised \
+for performance measurements."
+SECTION = "console/tools"
+LICENSE = "MIT"
+LIC_FILES_CHKSUM = "file://LICENSE;md5=a1fcde5f6838616beb564260554a388f"
+DEPENDS = "wayland"
+
+SRC_URI = " \
+    gitsm://github.com/ARM-software/${BPN}.git;branch=master;protocol=https \
+    file://0001-CMakeLists.txt-Switch-to-python3.patch"
+SRCREV = "06ab2eb1be315408f9ac652715f2f19c5c4b3182"
+
+S = "${WORKDIR}/git"
+
+inherit cmake pkgconfig
+
+OECMAKE_GENERATOR = "Unix Makefiles"
+
+OECMAKE_SOURCEPATH = "${S}/patrace/project/cmake"
+
+EXTRA_OECMAKE = " \
+    -DWINDOWSYSTEM=wayland \
+    -DCMAKE_INSTALL_PREFIX:PATH=/opt/${BPN}"
+
+# FIXME: Don't ignore problems
+CFLAGS += "-Wno-maybe-uninitialized"
+CXXFLAGS += "-Wno-range-loop-construct -Wno-unused-variable"
+
+SOLIBS = ".so"
+FILES_SOLIBSDEV = ""
+
+FILES:${PN} = "/opt"