Re: [OE-core] [PATCH 1/1] build-compare: 2015.02.10 -> 2019.08-14

2019-09-02 Thread Robert Yang

Sorry, the subject should be:

2015.02.10 -> 2019.08.14 (not 08-14).

I've updated it in the PULL.

// Robert

On 9/2/19 4:11 PM, Robert Yang wrote:

* Removed the following patches which are already merged by upstream:
   0001-Add-support-for-deb-and-ipk-packaging.patch
   Rename-rpm-check.sh-to-pkg-diff.sh.patch
   functions.sh-improve-deb-and-ipk-checking.patch
   functions.sh-remove-space-at-head.patch
   functions.sh-run-rpm-once-to-make-it-faster.patch
   pkg-diff.sh-check-for-fifo-named-pipe.patch
   pkg-diff.sh-check_single_file-return-at-once-when-sa.patch
   pkg-diff.sh-remove-space-in-the-end-for-ftype.patch

* Rebased Ignore-DWARF-sections.patch

This version is very outstanding when compare binary packages, e.g.:
PRSERV_HOST = "localhost:0"
INHERIT += "packagefeed-stability"
PACKAGE_CLASSES = "package_ipk
$ bitbake opkg
$ find tmp/deploy/ipk >/tmp/ipk_1

Add a "bbnote 'hello'" to autotools.bbclass' autotools_do_configure.

* BEFORE the upgrading, the result is:
$ diff /tmp/ipk_1  /tmp/ipk_2 -Nur | diffstat
  ipk_2 | 1570 
+-
  1 file changed, 785 insertions(+), 785 deletions(-)

* AFTER the upgrading, the result is:
$ bitbake opkg
$ find tmp/deploy/ipk >/tmp/ipk_2
$ diff /tmp/ipk_1 /tmp/ipk_2 -Nur
No output

And if we really modifed a recipe such as opkg, then it would show that it is
changed.

For a full world build AFTER the upgrading:
$ diff /tmp/ipk_6 /tmp/ipk_7 -Nur | diffstat
  ipk_7 | 2090 
+-
  1 file changed, 1045 insertions(+), 1045 deletions(-)

There are 10968 packages in totall, 1045 ones have been changed, so we can still
improve it in the future.

Signed-off-by: Robert Yang 
---
  .../build-compare/build-compare_git.bb |   12 +-
  ...001-Add-support-for-deb-and-ipk-packaging.patch |   64 -
  .../files/Ignore-DWARF-sections.patch  |   17 +-
  .../files/Rename-rpm-check.sh-to-pkg-diff.sh.patch | 1599 
  ...functions.sh-improve-deb-and-ipk-checking.patch |  117 --
  .../files/functions.sh-remove-space-at-head.patch  |   41 -
  ...nctions.sh-run-rpm-once-to-make-it-faster.patch |  361 -
  .../pkg-diff.sh-check-for-fifo-named-pipe.patch|   35 -
  ...-check_single_file-return-at-once-when-sa.patch |   37 -
  ...diff.sh-remove-space-in-the-end-for-ftype.patch |   32 -
  10 files changed, 9 insertions(+), 2306 deletions(-)
  delete mode 100644 
meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/Rename-rpm-check.sh-to-pkg-diff.sh.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-improve-deb-and-ipk-checking.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-remove-space-at-head.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-run-rpm-once-to-make-it-faster.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-check-for-fifo-named-pipe.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-check_single_file-return-at-once-when-sa.patch
  delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-remove-space-in-the-end-for-ftype.patch

diff --git a/meta/recipes-devtools/build-compare/build-compare_git.bb 
b/meta/recipes-devtools/build-compare/build-compare_git.bb
index efcf6b6..b0560cc 100644
--- a/meta/recipes-devtools/build-compare/build-compare_git.bb
+++ b/meta/recipes-devtools/build-compare/build-compare_git.bb
@@ -6,22 +6,14 @@ LICENSE = "GPLv2"
  LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
  
  SRC_URI = "git://github.com/openSUSE/build-compare.git \

-   file://Rename-rpm-check.sh-to-pkg-diff.sh.patch;striplevel=1 \
 file://Ignore-DWARF-sections.patch;striplevel=1 \
-   file://0001-Add-support-for-deb-and-ipk-packaging.patch \
-   file://functions.sh-remove-space-at-head.patch \
-   file://functions.sh-run-rpm-once-to-make-it-faster.patch \
-   file://pkg-diff.sh-check-for-fifo-named-pipe.patch \
-   file://pkg-diff.sh-check_single_file-return-at-once-when-sa.patch \
-   file://pkg-diff.sh-remove-space-in-the-end-for-ftype.patch \
-   file://functions.sh-improve-deb-and-ipk-checking.patch \
 "
  
  # Date matches entry in build-compare.changes and date of SRCREV.

  #
-SRCREV = "c5352c054c6ef15735da31b76d6d88620f4aff0a"
+SRCREV = "4dfa207660776cae120afa4353aec7f1f2a998d2"
  PE = "1"
-PV = "2015.02.10+git${SRCPV}"
+PV = "2019.08.14+git${SRCPV}"
  UPSTREAM_CHECK_COMMITS = "1"
  
  S = "${WORKDIR}/git"

diff --git 
a/meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
 
b/meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
deleted file mode 

[OE-core] [PATCH 1/1] build-compare: 2015.02.10 -> 2019.08-14

2019-09-02 Thread Robert Yang
* Removed the following patches which are already merged by upstream:
  0001-Add-support-for-deb-and-ipk-packaging.patch
  Rename-rpm-check.sh-to-pkg-diff.sh.patch
  functions.sh-improve-deb-and-ipk-checking.patch
  functions.sh-remove-space-at-head.patch
  functions.sh-run-rpm-once-to-make-it-faster.patch
  pkg-diff.sh-check-for-fifo-named-pipe.patch
  pkg-diff.sh-check_single_file-return-at-once-when-sa.patch
  pkg-diff.sh-remove-space-in-the-end-for-ftype.patch

* Rebased Ignore-DWARF-sections.patch

This version is very outstanding when compare binary packages, e.g.:
PRSERV_HOST = "localhost:0"
INHERIT += "packagefeed-stability"
PACKAGE_CLASSES = "package_ipk
$ bitbake opkg
$ find tmp/deploy/ipk >/tmp/ipk_1

Add a "bbnote 'hello'" to autotools.bbclass' autotools_do_configure.

* BEFORE the upgrading, the result is:
$ diff /tmp/ipk_1  /tmp/ipk_2 -Nur | diffstat
 ipk_2 | 1570 +-
 1 file changed, 785 insertions(+), 785 deletions(-)

* AFTER the upgrading, the result is:
$ bitbake opkg
$ find tmp/deploy/ipk >/tmp/ipk_2
$ diff /tmp/ipk_1 /tmp/ipk_2 -Nur
No output

And if we really modifed a recipe such as opkg, then it would show that it is
changed.

For a full world build AFTER the upgrading:
$ diff /tmp/ipk_6 /tmp/ipk_7 -Nur | diffstat
 ipk_7 | 2090 +-
 1 file changed, 1045 insertions(+), 1045 deletions(-)

There are 10968 packages in totall, 1045 ones have been changed, so we can still
improve it in the future.

Signed-off-by: Robert Yang 
---
 .../build-compare/build-compare_git.bb |   12 +-
 ...001-Add-support-for-deb-and-ipk-packaging.patch |   64 -
 .../files/Ignore-DWARF-sections.patch  |   17 +-
 .../files/Rename-rpm-check.sh-to-pkg-diff.sh.patch | 1599 
 ...functions.sh-improve-deb-and-ipk-checking.patch |  117 --
 .../files/functions.sh-remove-space-at-head.patch  |   41 -
 ...nctions.sh-run-rpm-once-to-make-it-faster.patch |  361 -
 .../pkg-diff.sh-check-for-fifo-named-pipe.patch|   35 -
 ...-check_single_file-return-at-once-when-sa.patch |   37 -
 ...diff.sh-remove-space-in-the-end-for-ftype.patch |   32 -
 10 files changed, 9 insertions(+), 2306 deletions(-)
 delete mode 100644 
meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/Rename-rpm-check.sh-to-pkg-diff.sh.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-improve-deb-and-ipk-checking.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-remove-space-at-head.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/functions.sh-run-rpm-once-to-make-it-faster.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-check-for-fifo-named-pipe.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-check_single_file-return-at-once-when-sa.patch
 delete mode 100644 
meta/recipes-devtools/build-compare/files/pkg-diff.sh-remove-space-in-the-end-for-ftype.patch

diff --git a/meta/recipes-devtools/build-compare/build-compare_git.bb 
b/meta/recipes-devtools/build-compare/build-compare_git.bb
index efcf6b6..b0560cc 100644
--- a/meta/recipes-devtools/build-compare/build-compare_git.bb
+++ b/meta/recipes-devtools/build-compare/build-compare_git.bb
@@ -6,22 +6,14 @@ LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
 SRC_URI = "git://github.com/openSUSE/build-compare.git \
-   file://Rename-rpm-check.sh-to-pkg-diff.sh.patch;striplevel=1 \
file://Ignore-DWARF-sections.patch;striplevel=1 \
-   file://0001-Add-support-for-deb-and-ipk-packaging.patch \
-   file://functions.sh-remove-space-at-head.patch \
-   file://functions.sh-run-rpm-once-to-make-it-faster.patch \
-   file://pkg-diff.sh-check-for-fifo-named-pipe.patch \
-   file://pkg-diff.sh-check_single_file-return-at-once-when-sa.patch \
-   file://pkg-diff.sh-remove-space-in-the-end-for-ftype.patch \
-   file://functions.sh-improve-deb-and-ipk-checking.patch \
"
 
 # Date matches entry in build-compare.changes and date of SRCREV.
 #
-SRCREV = "c5352c054c6ef15735da31b76d6d88620f4aff0a"
+SRCREV = "4dfa207660776cae120afa4353aec7f1f2a998d2"
 PE = "1"
-PV = "2015.02.10+git${SRCPV}"
+PV = "2019.08.14+git${SRCPV}"
 UPSTREAM_CHECK_COMMITS = "1"
 
 S = "${WORKDIR}/git"
diff --git 
a/meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
 
b/meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
deleted file mode 100644
index 82fd816..000
--- 
a/meta/recipes-devtools/build-compare/files/0001-Add-support-for-deb-and-ipk-packaging.patch
+++ /dev/null
@@ -1,64 +0,0 @@
-From