[PATCH v2 0/1] bootchart2: Several fixes

[PATCH v2 0/1] bootchart2: Several fixes

[liezhi.yang]

From: Robert Yang <liezhi.yang@windriver.com>

* V2: Fixed Ross' comments:
  - Add -n to gzip to make the resulting files reproducible, so we don't need to
    decompress them.
  
  - Makefile: Let bootchartd.conf use EARLY_PREFIX, so that bootchartd.conf can
    install to correct location.
  
  - Set MANDIR and DOCDIR to make the files install to correct location for
    bootchart2-native.

// Robert

The following changes since commit abe08a3238d162a1bdbc68172307eb3eb127bbb1:

  python_mesonpy: remove cython-specific INSANE_SKIPs (2024-11-12 23:45:58 +0000)

are available in the Git repository at:

  https://github.com/robertlinux/yocto rbt/bootchart2
  https://github.com/robertlinux/yocto/tree/rbt/bootchart2

Robert Yang (1):
  bootchart2: Several fixes

 ...efile-Let-bootchartd.conf-use-EARLY_.patch | 30 +++++++++++++++
 .../0002-Makefile-Add-n-to-gzip.patch         | 37 +++++++++++++++++++
 .../bootchart2/bootchart2_0.14.9.bb           |  7 ++--
 3 files changed, 71 insertions(+), 3 deletions(-)
 create mode 100644 meta/recipes-devtools/bootchart2/bootchart2/0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch
 create mode 100644 meta/recipes-devtools/bootchart2/bootchart2/0002-Makefile-Add-n-to-gzip.patch

-- 
2.25.1

[PATCH v2 1/1] bootchart2: Several fixes

[liezhi.yang]

From: Robert Yang <liezhi.yang@windriver.com>

Fixed Ross' comments:
* Add -n to gzip to make the resulting files reproducible, so we don't need to
  decompress them.

* Makefile: Let bootchartd.conf use EARLY_PREFIX, so that bootchartd.conf can
  install to correct location.

* Set MANDIR and DOCDIR to make the files install to correct location for
  bootchart2-native.

Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
---
 ...efile-Let-bootchartd.conf-use-EARLY_.patch | 30 +++++++++++++++
 .../0002-Makefile-Add-n-to-gzip.patch         | 37 +++++++++++++++++++
 .../bootchart2/bootchart2_0.14.9.bb           |  7 ++--
 3 files changed, 71 insertions(+), 3 deletions(-)
 create mode 100644 meta/recipes-devtools/bootchart2/bootchart2/0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch
 create mode 100644 meta/recipes-devtools/bootchart2/bootchart2/0002-Makefile-Add-n-to-gzip.patch

diff --git a/meta/recipes-devtools/bootchart2/bootchart2/0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch b/meta/recipes-devtools/bootchart2/bootchart2/0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch
new file mode 100644
index 0000000000..8209a776ba
--- /dev/null
+++ b/meta/recipes-devtools/bootchart2/bootchart2/0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch
@@ -0,0 +1,30 @@
+From 0ba0986f1ebc5ac1ad0a3d51e8039dbdccb31c25 Mon Sep 17 00:00:00 2001
+From: Robert Yang <liezhi.yang@windriver.com>
+Date: Thu, 14 Nov 2024 07:33:56 +0000
+Subject: [PATCH 1/2] Makefile: Let bootchartd.conf use EARLY_PREFIX
+
+So that bootchartd.conf can install to correct location when cross build.
+
+Upstream-Status: Submitted [https://github.com/xrmx/bootchart/pull/105]
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
+---
+ Makefile | 2 +-
+ 1 file changed, 1 insertion(+), 1 deletion(-)
+
+diff --git a/Makefile b/Makefile
+index b99b141..99a3ec7 100644
+--- a/Makefile
++++ b/Makefile
+@@ -100,7 +100,7 @@ install-chroot:
+ 
+ install-collector: all install-chroot
+ 	install -m 755 -D bootchartd $(DESTDIR)$(EARLY_PREFIX)/sbin/$(PROGRAM_PREFIX)bootchartd$(PROGRAM_SUFFIX)
+-	install -m 644 -D bootchartd.conf $(DESTDIR)/etc/$(PROGRAM_PREFIX)bootchartd$(PROGRAM_SUFFIX).conf
++	install -m 644 -D bootchartd.conf $(DESTDIR)$(EARLY_PREFIX)/etc/$(PROGRAM_PREFIX)bootchartd$(PROGRAM_SUFFIX).conf
+ 	install -m 755 -D bootchart-collector $(DESTDIR)$(PKGLIBDIR)/$(PROGRAM_PREFIX)bootchart$(PROGRAM_SUFFIX)-collector
+ 
+ install-docs:
+-- 
+2.44.1
+
diff --git a/meta/recipes-devtools/bootchart2/bootchart2/0002-Makefile-Add-n-to-gzip.patch b/meta/recipes-devtools/bootchart2/bootchart2/0002-Makefile-Add-n-to-gzip.patch
new file mode 100644
index 0000000000..baa338759b
--- /dev/null
+++ b/meta/recipes-devtools/bootchart2/bootchart2/0002-Makefile-Add-n-to-gzip.patch
@@ -0,0 +1,37 @@
+From aec8c971e09a28a5f8cd655ccd2f0cf1b057d756 Mon Sep 17 00:00:00 2001
+From: Robert Yang <liezhi.yang@windriver.com>
+Date: Thu, 14 Nov 2024 07:35:11 +0000
+Subject: [PATCH 2/2] Makefile: Add -n to gzip
+
+To make the resulting files reproducible.
+
+-n --no-name
+    When compressing, do not save the original file name and timestamp by default.
+
+Upstream-Status: Submitted [https://github.com/xrmx/bootchart/pull/106]
+
+Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
+---
+ Makefile | 6 +++---
+ 1 file changed, 3 insertions(+), 3 deletions(-)
+
+diff --git a/Makefile b/Makefile
+index 99a3ec7..87de486 100644
+--- a/Makefile
++++ b/Makefile
+@@ -107,9 +107,9 @@ install-docs:
+ 	install -m 644 -D README $(DESTDIR)$(DOCDIR)/README
+ 	install -m 644 -D README.pybootchart $(DESTDIR)$(DOCDIR)/README.pybootchart
+ 	mkdir -p $(DESTDIR)$(MANDIR)
+-	gzip -c bootchart2.1 > $(DESTDIR)$(MANDIR)/bootchart2.1.gz
+-	gzip -c bootchartd.1 > $(DESTDIR)$(MANDIR)/$(PROGRAM_PREFIX)bootchartd$(PROGRAM_SUFFIX).1.gz
+-	gzip -c pybootchartgui.1 > $(DESTDIR)$(MANDIR)/pybootchartgui.1.gz
++	gzip -n -c bootchart2.1 > $(DESTDIR)$(MANDIR)/bootchart2.1.gz
++	gzip -n -c bootchartd.1 > $(DESTDIR)$(MANDIR)/$(PROGRAM_PREFIX)bootchartd$(PROGRAM_SUFFIX).1.gz
++	gzip -n -c pybootchartgui.1 > $(DESTDIR)$(MANDIR)/pybootchartgui.1.gz
+ 
+ install-service:
+ 	mkdir -p $(DESTDIR)$(SYSTEMD_UNIT_DIR)
+-- 
+2.44.1
+
diff --git a/meta/recipes-devtools/bootchart2/bootchart2_0.14.9.bb b/meta/recipes-devtools/bootchart2/bootchart2_0.14.9.bb
index 8a211366bf..ce030cec98 100644
--- a/meta/recipes-devtools/bootchart2/bootchart2_0.14.9.bb
+++ b/meta/recipes-devtools/bootchart2/bootchart2_0.14.9.bb
@@ -94,6 +94,8 @@ SRC_URI = "git://github.com/xrmx/bootchart.git;branch=master;protocol=https \
            file://0001-collector-Allocate-space-on-heap-for-chunks.patch \
            file://0001-bootchartd.in-make-sure-only-one-bootchartd-process.patch \
            file://0001-Do-not-include-linux-fs.h.patch \
+           file://0001-Makefile-Let-bootchartd.conf-use-EARLY_.patch \
+           file://0002-Makefile-Add-n-to-gzip.patch \
           "
 
 S = "${WORKDIR}/git"
@@ -129,6 +131,8 @@ do_install () {
     export DESTDIR="${D}"
     export LIBDIR="/${baselib}"
     export EARLY_PREFIX="${root_prefix}"
+    export MANDIR="${mandir}/man1"
+    export DOCDIR="${docdir}"
 
     oe_runmake install NO_PYTHON_COMPILE=1
     install -d ${D}${sysconfdir}/init.d
@@ -138,9 +142,6 @@ do_install () {
 
    # Use python 3 instead of python 2
    sed -i -e '1s,#!.*python.*,#!${USRBINPATH}/env python3,' ${D}${bindir}/pybootchartgui
-
-    # The timestamps embedded in compressed man pages is not reproducible
-    gzip -d ${D}${mandir}/man1/*.gz
 }
 
 PACKAGES =+ "pybootchartgui"
-- 
2.25.1

Re: [OE-core] [PATCH v2 1/1] bootchart2: Several fixes

[Mathieu Dubois-Briand]

On Thu, Nov 14, 2024 at 12:25:17AM -0800, Robert Yang via lists.openembedded.org wrote:
> From: Robert Yang <liezhi.yang@windriver.com>
> 
> Fixed Ross' comments:
> * Add -n to gzip to make the resulting files reproducible, so we don't need to
>   decompress them.
> 
> * Makefile: Let bootchartd.conf use EARLY_PREFIX, so that bootchartd.conf can
>   install to correct location.
> 
> * Set MANDIR and DOCDIR to make the files install to correct location for
>   bootchart2-native.
> 
> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>

Hi Robert,

Thanks for the v2, but I believe this is failing some builds on the
autobuilder:

ERROR: bootchart2-0.14.9-r0 do_package: QA Issue: bootchart2:
Files/directories were installed but not shipped in any package:
  /usr/etc
  /usr/etc/bootchartd.conf

https://valkyrie.yoctoproject.org/#/builders/25/builds/432/steps/14/logs/stdio

Can you look at this issue please ?

-- 
Mathieu Dubois-Briand, Bootlin
Embedded Linux and Kernel engineering
https://bootlin.com

Re: [OE-core] [PATCH v2 1/1] bootchart2: Several fixes

[Robert Yang]

On 11/15/24 00:28, Mathieu Dubois-Briand wrote:
> On Thu, Nov 14, 2024 at 12:25:17AM -0800, Robert Yang via lists.openembedded.org wrote:
>> From: Robert Yang <liezhi.yang@windriver.com>
>>
>> Fixed Ross' comments:
>> * Add -n to gzip to make the resulting files reproducible, so we don't need to
>>    decompress them.
>>
>> * Makefile: Let bootchartd.conf use EARLY_PREFIX, so that bootchartd.conf can
>>    install to correct location.
>>
>> * Set MANDIR and DOCDIR to make the files install to correct location for
>>    bootchart2-native.
>>
>> Signed-off-by: Robert Yang <liezhi.yang@windriver.com>
> 
> Hi Robert,
> 
> Thanks for the v2, but I believe this is failing some builds on the
> autobuilder:
> 
> ERROR: bootchart2-0.14.9-r0 do_package: QA Issue: bootchart2:
> Files/directories were installed but not shipped in any package:
>    /usr/etc
>    /usr/etc/bootchartd.conf

Sorry, it will be failed when:
DISTRO_FEATURES:append = " usrmerge"

I will fix it.

// Robert

> 
> https://valkyrie.yoctoproject.org/#/builders/25/builds/432/steps/14/logs/stdio
> 
> Can you look at this issue please ?
>