mailarchive of the ptxdist mailing list
 help / color / mirror / Atom feed
From: Christian Melki <christian.melki@t2data.com>
To: ptxdist@pengutronix.de
Subject: [ptxdist] [PATCH] host-opkg-utils: Version bump. 0.5.0 -> 0.6.3
Date: Mon, 12 Feb 2024 21:45:19 +0100	[thread overview]
Message-ID: <20240212204520.1168713-1-christian.melki@t2data.com> (raw)

https://git.yoctoproject.org/opkg-utils/log/?h=0.6.3
Added acls and xattrs. Not sure they'll be used?
Othersise, minor changes. Looks like they're trying to follow
the opkg version number.

* Rework the patch set. Try to follow the intentions
of the previous diff.

Signed-off-by: Christian Melki <christian.melki@t2data.com>
---
 ...ld-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch | 38 --------------
 ...0100-opkg-build-use-CROSS_COMPILE-ar.patch | 10 ++--
 .../0101-Update-Installed-Size-properly.patch |  6 +--
 ...xdist-the-maxium-UID-for-system-user.patch |  4 +-
 ...ld-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch | 52 +++++++++++++++++++
 ...pkg-build-fix-typo-in-compressorargs.patch |  4 +-
 .../series                                    |  0
 rules/host-opkg-utils.make                    |  4 +-
 8 files changed, 66 insertions(+), 52 deletions(-)
 delete mode 100644 patches/opkg-utils-0.5.0/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
 rename patches/{opkg-utils-0.5.0 => opkg-utils-0.6.3}/0100-opkg-build-use-CROSS_COMPILE-ar.patch (59%)
 rename patches/{opkg-utils-0.5.0 => opkg-utils-0.6.3}/0101-Update-Installed-Size-properly.patch (91%)
 rename patches/{opkg-utils-0.5.0 => opkg-utils-0.6.3}/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch (95%)
 create mode 100644 patches/opkg-utils-0.6.3/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
 rename patches/{opkg-utils-0.5.0 => opkg-utils-0.6.3}/0104-opkg-build-fix-typo-in-compressorargs.patch (86%)
 rename patches/{opkg-utils-0.5.0 => opkg-utils-0.6.3}/series (100%)

diff --git a/patches/opkg-utils-0.5.0/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch b/patches/opkg-utils-0.5.0/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
deleted file mode 100644
index b855eec53..000000000
--- a/patches/opkg-utils-0.5.0/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
+++ /dev/null
@@ -1,38 +0,0 @@
-From: Michael Olbrich <m.olbrich@pengutronix.de>
-Date: Sat, 25 Jan 2020 18:28:40 +0100
-Subject: [PATCH] Revert "opkg-build: clamp mtimes to SOURCE_DATE_EPOCH"
-
-This reverts commit c3cc95693048bdd57a82069bad47abbc72a1932e.
-
-We set the mtime already in PTXdist so it's not needed and this breaks
-building with tar <= 1.28 (e.g. CentOS 7).
-
-Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
----
- opkg-build | 9 +--------
- 1 file changed, 1 insertion(+), 8 deletions(-)
-
-diff --git a/opkg-build b/opkg-build
-index 6d6f7e92b919..2fb6216104b6 100755
---- a/opkg-build
-+++ b/opkg-build
-@@ -298,18 +298,11 @@ mkdir $tmp_dir
- 
- build_date="${SOURCE_DATE_EPOCH:-$(date +%s)}"
- 
--mtime_args=""
--# --clamp-mtime requires tar > 1.28. Only use it if SOURCE_DATE_EPOCH is set, to avoid having a generic case dependency on tar > 1.28.
--# this setting will make sure files generated at build time have consistent mtimes, for reproducible builds.
--if [ ! -z "$SOURCE_DATE_EPOCH"  ]; then
--    mtime_args="--mtime=@$build_date --clamp-mtime"
--fi
--
- export LANG=C
- export LC_ALL=C
- ( cd $pkg_dir/$CONTROL && find . -type f | sort > $tmp_dir/control_list )
- ( cd $pkg_dir && find . -path ./$CONTROL -prune -o -path . -o -print  | sort > $tmp_dir/file_list )
--( cd $pkg_dir && tar $ogargs $tsortargs --no-recursion $mtime_args -c $tarformat -T $tmp_dir/file_list | $compressor $compressorargs > $tmp_dir/data.tar.$cext )
-+( cd $pkg_dir && tar $ogargs $tsortargs --no-recursion -c $tarformat -T $tmp_dir/file_list | $compressor $compressorargs > $tmp_dir/data.tar.$cext )
- ( cd $pkg_dir/$CONTROL && tar $ogargs $tsortargs --no-recursion --mtime=@$build_date -c $tarformat -T $tmp_dir/control_list | gzip $zipargs > $tmp_dir/control.tar.gz )
- rm $tmp_dir/file_list
- rm $tmp_dir/control_list
diff --git a/patches/opkg-utils-0.5.0/0100-opkg-build-use-CROSS_COMPILE-ar.patch b/patches/opkg-utils-0.6.3/0100-opkg-build-use-CROSS_COMPILE-ar.patch
similarity index 59%
rename from patches/opkg-utils-0.5.0/0100-opkg-build-use-CROSS_COMPILE-ar.patch
rename to patches/opkg-utils-0.6.3/0100-opkg-build-use-CROSS_COMPILE-ar.patch
index 9e54dd0ec..556092909 100644
--- a/patches/opkg-utils-0.5.0/0100-opkg-build-use-CROSS_COMPILE-ar.patch
+++ b/patches/opkg-utils-0.6.3/0100-opkg-build-use-CROSS_COMPILE-ar.patch
@@ -14,15 +14,15 @@ Signed-off-by: Andreas Bießmann <andreas@biessmann.de>
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/opkg-build b/opkg-build
-index 1b564166a7a4..eca1ee2fa42c 100755
+index 6edd938b8f7e..ef2fbfa42c09 100755
 --- a/opkg-build
 +++ b/opkg-build
-@@ -324,7 +324,7 @@ fi
+@@ -352,7 +352,7 @@ fi
  
  rm -f $pkg_file
  if [ "$outer" = "ar" ] ; then
--  ( cd $tmp_dir && ar -crfD $pkg_file ./debian-binary ./control.tar.gz ./data.tar.$cext )
-+  ( cd $tmp_dir && ${CROSS_COMPILE}ar -crfD $pkg_file ./debian-binary ./control.tar.gz ./data.tar.$cext )
+-	( cd $tmp_dir && ar -crfD $pkg_file ./debian-binary ./control.tar.gz ./data.tar.$cext )
++	( cd $tmp_dir && ${CROSS_COMPILE}ar -crfD $pkg_file ./debian-binary ./control.tar.gz ./data.tar.$cext )
  else
-   ( cd $tmp_dir && tar -c $tsortargs --mtime=@$build_date $tarformat ./debian-binary ./control.tar.gz ./data.tar.$cext | gzip $zipargs > $pkg_file )
+ 	( cd $tmp_dir && tar -c $tsortargs $mtime_args $tarformat ./debian-binary ./control.tar.gz ./data.tar.$cext | gzip $zipargs > $pkg_file )
  fi
diff --git a/patches/opkg-utils-0.5.0/0101-Update-Installed-Size-properly.patch b/patches/opkg-utils-0.6.3/0101-Update-Installed-Size-properly.patch
similarity index 91%
rename from patches/opkg-utils-0.5.0/0101-Update-Installed-Size-properly.patch
rename to patches/opkg-utils-0.6.3/0101-Update-Installed-Size-properly.patch
index 3f617eaff..48a00ff9a 100644
--- a/patches/opkg-utils-0.5.0/0101-Update-Installed-Size-properly.patch
+++ b/patches/opkg-utils-0.6.3/0101-Update-Installed-Size-properly.patch
@@ -11,10 +11,10 @@ Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
  1 file changed, 2 insertions(+), 2 deletions(-)
 
 diff --git a/opkg.py b/opkg.py
-index ba947c2e887b..7b7e216419a1 100644
+index e5a65dd1ae8f..ec77c147156c 100644
 --- a/opkg.py
 +++ b/opkg.py
-@@ -248,7 +248,7 @@ class Package(object):
+@@ -276,7 +276,7 @@ class Package(object):
              lineparts = re.match(r'([\w-]*?):\s*(.*)', line)
              if lineparts:
                  name = lineparts.group(1)
@@ -23,7 +23,7 @@ index ba947c2e887b..7b7e216419a1 100644
                  value = lineparts.group(2)
                  while 1:
                      line = control.readline().rstrip()
-@@ -510,7 +510,7 @@ class Package(object):
+@@ -538,7 +538,7 @@ class Package(object):
          if 'sha256' in checksum:
              if self.sha256: out = out + "SHA256sum: %s\n" % (self.sha256)
          if self.size: out = out + "Size: %d\n" % int(self.size)
diff --git a/patches/opkg-utils-0.5.0/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch b/patches/opkg-utils-0.6.3/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch
similarity index 95%
rename from patches/opkg-utils-0.5.0/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch
rename to patches/opkg-utils-0.6.3/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch
index 8a1df58e1..ba8c852a1 100644
--- a/patches/opkg-utils-0.5.0/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch
+++ b/patches/opkg-utils-0.6.3/0102-opkg-build-in-ptxdist-the-maxium-UID-for-system-user.patch
@@ -8,11 +8,11 @@ Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
  1 file changed, 3 insertions(+), 3 deletions(-)
 
 diff --git a/opkg-build b/opkg-build
-index eca1ee2fa42c..6d6f7e92b919 100755
+index ef2fbfa42c09..6a1b631fc5d5 100755
 --- a/opkg-build
 +++ b/opkg-build
 @@ -69,14 +69,14 @@ You probably want to remove them: " >&2
- 	    fi
+ 		fi
  	fi
  
 -	large_uid_files=`find . -uid +99 -ls -printf '\\\n' || true`
diff --git a/patches/opkg-utils-0.6.3/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch b/patches/opkg-utils-0.6.3/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
new file mode 100644
index 000000000..b3c38ac3c
--- /dev/null
+++ b/patches/opkg-utils-0.6.3/0103-Revert-opkg-build-clamp-mtimes-to-SOURCE_DATE_EPOCH.patch
@@ -0,0 +1,52 @@
+From: Michael Olbrich <m.olbrich@pengutronix.de>
+Date: Sat, 25 Jan 2020 18:28:40 +0100
+Subject: [PATCH] Revert "opkg-build: clamp mtimes to SOURCE_DATE_EPOCH"
+
+This reverts commit c3cc95693048bdd57a82069bad47abbc72a1932e.
+
+We set the mtime already in PTXdist so it's not needed and this breaks
+building with tar <= 1.28 (e.g. CentOS 7).
+
+Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
+---
+ opkg-build | 23 ++---------------------
+ 1 file changed, 2 insertions(+), 21 deletions(-)
+
+diff --git a/opkg-build b/opkg-build
+index 6a1b631fc5d5..182e5d7f318d 100755
+--- a/opkg-build
++++ b/opkg-build
+@@ -314,31 +314,12 @@ fi
+ tmp_dir=$dest_dir/IPKG_BUILD.$$
+ mkdir $tmp_dir
+ 
+-mtime_args="--mtime=@${SOURCE_DATE_EPOCH:-$(date +%s)}"
+-# --clamp-mtime requires tar > 1.28. Only use it if SOURCE_DATE_EPOCH is set, to avoid having a generic case dependency on tar > 1.28.
+-# this setting will make sure files generated at build time have consistent mtimes, for reproducible builds.
+-if [ ! -z "$SOURCE_DATE_EPOCH"  ]; then
+-	mtime_args="$mtime_args --clamp-mtime"
+-fi
+-
+-# Notice, that if you create an archive in POSIX format (see section GNU tar and POSIX tar) and the environment variable POSIXLY_CORRECT is set,
+-# then the two archives created using the same options on the same set of files will not be byte-to-byte equivalent even with the above option.
+-# This is because the posix default for extended header names includes the PID of the tar process, which is different at each run. To produce
+-# byte-to-byte equivalent archives in this case, either unset POSIXLY_CORRECT, or use the following option:
+-#
+-# --pax-option=exthdr.name=%d/PaxHeaders/%f,atime:=0,ctime:=0
+-#
+-# https://www.gnu.org/software/tar/manual/html_node/PAX-keywords.html
+-if [[ "$tarformat" == "--format=posix" ]]; then
+-	mtime_args="$mtime_args --pax-option=exthdr.name=%d/PaxHeaders/%f,atime:=0,ctime:=0"
+-fi
+-
+ export LANG=C
+ export LC_ALL=C
+ ( cd $pkg_dir/$CONTROL && find . -type f | sort > $tmp_dir/control_list )
+ ( cd $pkg_dir && find . -path ./$CONTROL -prune -o -path . -o -print  | sort > $tmp_dir/file_list )
+-( cd $pkg_dir && tar $attributesargs $ogargs $tsortargs --numeric-owner --no-recursion $mtime_args -c $tarformat -T $tmp_dir/file_list | $compressor $compressorargs > $tmp_dir/data.tar.$cext )
+-( cd $pkg_dir/$CONTROL && tar $ogargs $tsortargs --no-recursion $mtime_args -c $tarformat -T $tmp_dir/control_list | gzip $zipargs > $tmp_dir/control.tar.gz )
++( cd $pkg_dir && tar $attributesargs $ogargs $tsortargs --numeric-owner --no-recursion -c $tarformat -T $tmp_dir/file_list | $compressor $compressorargs > $tmp_dir/data.tar.$cext )
++( cd $pkg_dir/$CONTROL && tar $ogargs $tsortargs --no-recursion --mtime=@$build_date -c $tarformat -T $tmp_dir/control_list | gzip $zipargs > $tmp_dir/control.tar.gz )
+ rm $tmp_dir/file_list
+ rm $tmp_dir/control_list
+ 
diff --git a/patches/opkg-utils-0.5.0/0104-opkg-build-fix-typo-in-compressorargs.patch b/patches/opkg-utils-0.6.3/0104-opkg-build-fix-typo-in-compressorargs.patch
similarity index 86%
rename from patches/opkg-utils-0.5.0/0104-opkg-build-fix-typo-in-compressorargs.patch
rename to patches/opkg-utils-0.6.3/0104-opkg-build-fix-typo-in-compressorargs.patch
index ea791bbc2..b7dd1f111 100644
--- a/patches/opkg-utils-0.5.0/0104-opkg-build-fix-typo-in-compressorargs.patch
+++ b/patches/opkg-utils-0.6.3/0104-opkg-build-fix-typo-in-compressorargs.patch
@@ -11,10 +11,10 @@ Signed-off-by: Michael Olbrich <m.olbrich@pengutronix.de>
  1 file changed, 1 insertion(+), 1 deletion(-)
 
 diff --git a/opkg-build b/opkg-build
-index 2fb6216104b6..7e1a1af99948 100755
+index 182e5d7f318d..dd8906320200 100755
 --- a/opkg-build
 +++ b/opkg-build
-@@ -236,7 +236,7 @@ if gzip --help 2>&1 | grep -- "-T" > /dev/null; then
+@@ -252,7 +252,7 @@ if gzip --help 2>&1 | grep -- "-T" > /dev/null; then
  	zipargs="-9nT"
  fi
  
diff --git a/patches/opkg-utils-0.5.0/series b/patches/opkg-utils-0.6.3/series
similarity index 100%
rename from patches/opkg-utils-0.5.0/series
rename to patches/opkg-utils-0.6.3/series
diff --git a/rules/host-opkg-utils.make b/rules/host-opkg-utils.make
index a9af478ff..a35a0da0d 100644
--- a/rules/host-opkg-utils.make
+++ b/rules/host-opkg-utils.make
@@ -14,8 +14,8 @@ HOST_PACKAGES-$(PTXCONF_HOST_OPKG_UTILS) += host-opkg-utils
 #
 # Paths and names
 #
-HOST_OPKG_UTILS_VERSION	:= 0.5.0
-HOST_OPKG_UTILS_MD5	:= c71939ee02c69462bd3ba1fe0d6de5e2
+HOST_OPKG_UTILS_VERSION	:= 0.6.3
+HOST_OPKG_UTILS_MD5	:= 2ca116eda29e0be4a89e92a9b24bda53
 HOST_OPKG_UTILS		:= opkg-utils-$(HOST_OPKG_UTILS_VERSION)
 HOST_OPKG_UTILS_SUFFIX	:= tar.gz
 HOST_OPKG_UTILS_URL	:= https://git.yoctoproject.org/opkg-utils/snapshot/$(HOST_OPKG_UTILS).$(HOST_OPKG_UTILS_SUFFIX)
-- 
2.34.1




             reply	other threads:[~2024-02-12 20:46 UTC|newest]

Thread overview: 4+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2024-02-12 20:45 Christian Melki [this message]
2024-02-12 20:45 ` [ptxdist] [PATCH] opkg: Version bump. 0.6.1 " Christian Melki
2024-02-14 16:19   ` [ptxdist] [APPLIED] " Michael Olbrich
2024-02-14 16:19 ` [ptxdist] [APPLIED] host-opkg-utils: Version bump. 0.5.0 " Michael Olbrich

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=20240212204520.1168713-1-christian.melki@t2data.com \
    --to=christian.melki@t2data.com \
    --cc=ptxdist@pengutronix.de \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox