[PATCH] dpkg: update SRC

Openembedded Core Discussions
 help / color / mirror / Atom feed

* [PATCH] dpkg: update SRC_URI
@ 2020-07-07  2:53 Changqing Li
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
  2020-07-09  9:02 ` ✗ patchtest: failure for dpkg: update SRC_URI (rev2) Patchwork
  0 siblings, 2 replies; 8+ messages in thread
From: Changqing Li @ 2020-07-07  2:53 UTC (permalink / raw)
  To: openembedded-core

From: Changqing Li <changqing.li@windriver.com>

dpkg_1.20.0.tar.xz is removed from original SRC_URI
http://ftp.debian.org/debian/pool/main/d/dpkg/dpkg_1.20.0.tar.xz,
update it to use http://snapshot.debian.org/

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-devtools/dpkg/dpkg_1.20.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
index c98a9e58e2..089caeffb5 100644
--- a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
+++ b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
@@ -1,7 +1,7 @@
 require dpkg.inc
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
-SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
+SRC_URI = "http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool/main/d/dpkg/${BPN}_${PV}.tar.xz \
            file://noman.patch \
            file://remove-tar-no-timestamp.patch \
            file://arch_pm.patch \
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH] dpkg: update SRC_URI
  2020-07-07  2:53 [PATCH] dpkg: update SRC_URI Changqing Li
@ 2020-07-07  7:36 ` Alexander Kanavin
  2020-07-07  7:41   ` Changqing Li
                     ` (3 more replies)
  2020-07-09  9:02 ` ✗ patchtest: failure for dpkg: update SRC_URI (rev2) Patchwork
  1 sibling, 4 replies; 8+ messages in thread
From: Alexander Kanavin @ 2020-07-07  7:36 UTC (permalink / raw)
  To: Changqing Li; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 453 bytes --]

On Tue, 7 Jul 2020 at 04:55, Changqing Li <changqing.li@windriver.com>
wrote:

> -SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
> +SRC_URI = "
> http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool/main/d/dpkg/${BPN}_${PV}.tar.xz
> \
>

This breaks both automated version checks, and automated version upgrades.
Can you change SRC_URI to take dpkg from git?
https://salsa.debian.org/dpkg-team/dpkg

Alex

[-- Attachment #2: Type: text/html, Size: 985 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH] dpkg: update SRC_URI
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
@ 2020-07-07  7:41   ` Changqing Li
  2020-07-09  8:54   ` [PATCH] dpkg: change SRC_URI to take dpkg from git Changqing Li
                     ` (2 subsequent siblings)
  3 siblings, 0 replies; 8+ messages in thread
From: Changqing Li @ 2020-07-07  7:41 UTC (permalink / raw)
  To: Alexander Kanavin; +Cc: OE-core

[-- Attachment #1: Type: text/plain, Size: 706 bytes --]


On 7/7/20 3:36 PM, Alexander Kanavin wrote:
> On Tue, 7 Jul 2020 at 04:55, Changqing Li <changqing.li@windriver.com 
> <mailto:changqing.li@windriver.com>> wrote:
>
>     -SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
>     +SRC_URI =
>     "http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool/main/d/dpkg/${BPN}_${PV}.tar.xz
>     <http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool/main/d/dpkg/$%7BBPN%7D_$%7BPV%7D.tar.xz>
>     \
>
>
> This breaks both automated version checks, and automated version 
> upgrades. Can you change SRC_URI to take dpkg from git?
> https://salsa.debian.org/dpkg-team/dpkg
>
> Alex

OK,  thanks.


[-- Attachment #2: Type: text/html, Size: 1636 bytes --]

^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH] dpkg: change SRC_URI to take dpkg from git
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
  2020-07-07  7:41   ` Changqing Li
@ 2020-07-09  8:54   ` Changqing Li
  2020-07-10  2:08   ` [PATCH V3] " Changqing Li
  2020-07-10  9:21   ` [PATCH V4] " Changqing Li
  3 siblings, 0 replies; 8+ messages in thread
From: Changqing Li @ 2020-07-09  8:54 UTC (permalink / raw)
  To: openembedded-core

From: Changqing Li <changqing.li@windriver.com>

tar.xz on original URL have been removed, switch to use git

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-devtools/dpkg/dpkg.inc       |  9 ++++++++-
 meta/recipes-devtools/dpkg/dpkg_1.20.0.bb | 10 +++++++---
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/dpkg/dpkg.inc b/meta/recipes-devtools/dpkg/dpkg.inc
index 1c3c585d79..a309613259 100644
--- a/meta/recipes-devtools/dpkg/dpkg.inc
+++ b/meta/recipes-devtools/dpkg/dpkg.inc
@@ -7,7 +7,7 @@ DEPENDS_class-native = "bzip2-replacement-native zlib-native virtual/update-alte
 RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} perl"
 RDEPENDS_${PN}_class-native = ""
 
-UPSTREAM_CHECK_URI = "${DEBIAN_MIRROR}/main/d/dpkg/"
+UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>(\d+(\.\d+)+))"
 
 inherit autotools gettext perlnative pkgconfig perl-version update-alternatives
 
@@ -30,6 +30,13 @@ EXTRA_OECONF_append_class-target = " --disable-update-alternatives DEB_HOST_ARCH
 PACKAGECONFIG = "liblzma"
 PACKAGECONFIG[liblzma] = "--with-liblzma,--without-liblzma, xz"
 
+do_configure_prepend () {
+        # autotools_do_configure updates po/Makefile.in.in, we also need
+        # update dselect/po and scripts/po
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/dselect/po/
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/scripts/po/
+}
+
 do_install_append () {
 	if [ "${PN}" = "dpkg-native" ]; then
 		# update-alternatives doesn't have an offline mode
diff --git a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
index 089caeffb5..935dcac918 100644
--- a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
+++ b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
@@ -1,7 +1,7 @@
 require dpkg.inc
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
-SRC_URI = "http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool/main/d/dpkg/${BPN}_${PV}.tar.xz \
+SRC_URI = "git://salsa.debian.org/dpkg-team/dpkg.git;protocol=https \
            file://noman.patch \
            file://remove-tar-no-timestamp.patch \
            file://arch_pm.patch \
@@ -18,5 +18,9 @@ SRC_URI = "http://snapshot.debian.org/archive/debian-debug/20200308T085219Z/pool
 
 SRC_URI_append_class-native = " file://0001-build.c-ignore-return-of-1-from-tar-cf.patch"
 
-SRC_URI[md5sum] = "f88f077236a3ff3decae3b25c989893d"
-SRC_URI[sha256sum] = "b633cc2b0e030efb61e11029d8a3fb1123f719864c9992da2e52b471c96d0900"
+SRCREV = "314ac02663c5bd1a82b34745150bf13a39a549a3"
+
+S = "${WORKDIR}/git"
+
+#autotools.bbclass default AUTOTOOLS_AUXDIR is ${S}, we need to under ${S}/build-aux
+AUTOTOOLS_AUXDIR = "${S}/build-aux"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* [PATCH V3] dpkg: change SRC_URI to take dpkg from git
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
  2020-07-07  7:41   ` Changqing Li
  2020-07-09  8:54   ` [PATCH] dpkg: change SRC_URI to take dpkg from git Changqing Li
@ 2020-07-10  2:08   ` Changqing Li
  2020-07-10  6:02     ` [OE-core] " Richard Purdie
  2020-07-10  9:21   ` [PATCH V4] " Changqing Li
  3 siblings, 1 reply; 8+ messages in thread
From: Changqing Li @ 2020-07-10  2:08 UTC (permalink / raw)
  To: openembedded-core

From: Changqing Li <changqing.li@windriver.com>

tar.xz on original URL have been removed, switch to use git

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-devtools/dpkg/dpkg.inc       |  9 ++++++++-
 meta/recipes-devtools/dpkg/dpkg_1.20.0.bb | 10 +++++++---
 2 files changed, 15 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/dpkg/dpkg.inc b/meta/recipes-devtools/dpkg/dpkg.inc
index 1c3c585d79..a309613259 100644
--- a/meta/recipes-devtools/dpkg/dpkg.inc
+++ b/meta/recipes-devtools/dpkg/dpkg.inc
@@ -7,7 +7,7 @@ DEPENDS_class-native = "bzip2-replacement-native zlib-native virtual/update-alte
 RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} perl"
 RDEPENDS_${PN}_class-native = ""
 
-UPSTREAM_CHECK_URI = "${DEBIAN_MIRROR}/main/d/dpkg/"
+UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>(\d+(\.\d+)+))"
 
 inherit autotools gettext perlnative pkgconfig perl-version update-alternatives
 
@@ -30,6 +30,13 @@ EXTRA_OECONF_append_class-target = " --disable-update-alternatives DEB_HOST_ARCH
 PACKAGECONFIG = "liblzma"
 PACKAGECONFIG[liblzma] = "--with-liblzma,--without-liblzma, xz"
 
+do_configure_prepend () {
+        # autotools_do_configure updates po/Makefile.in.in, we also need
+        # update dselect/po and scripts/po
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/dselect/po/
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/scripts/po/
+}
+
 do_install_append () {
 	if [ "${PN}" = "dpkg-native" ]; then
 		# update-alternatives doesn't have an offline mode
diff --git a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
index c98a9e58e2..935dcac918 100644
--- a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
+++ b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
@@ -1,7 +1,7 @@
 require dpkg.inc
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
-SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
+SRC_URI = "git://salsa.debian.org/dpkg-team/dpkg.git;protocol=https \
            file://noman.patch \
            file://remove-tar-no-timestamp.patch \
            file://arch_pm.patch \
@@ -18,5 +18,9 @@ SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
 
 SRC_URI_append_class-native = " file://0001-build.c-ignore-return-of-1-from-tar-cf.patch"
 
-SRC_URI[md5sum] = "f88f077236a3ff3decae3b25c989893d"
-SRC_URI[sha256sum] = "b633cc2b0e030efb61e11029d8a3fb1123f719864c9992da2e52b471c96d0900"
+SRCREV = "314ac02663c5bd1a82b34745150bf13a39a549a3"
+
+S = "${WORKDIR}/git"
+
+#autotools.bbclass default AUTOTOOLS_AUXDIR is ${S}, we need to under ${S}/build-aux
+AUTOTOOLS_AUXDIR = "${S}/build-aux"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* Re: [OE-core] [PATCH V3] dpkg: change SRC_URI to take dpkg from git
  2020-07-10  2:08   ` [PATCH V3] " Changqing Li
@ 2020-07-10  6:02     ` Richard Purdie
  0 siblings, 0 replies; 8+ messages in thread
From: Richard Purdie @ 2020-07-10  6:02 UTC (permalink / raw)
  To: Changqing Li, openembedded-core

On Fri, 2020-07-10 at 10:08 +0800, Changqing Li wrote:
> From: Changqing Li <changqing.li@windriver.com>
> 
> tar.xz on original URL have been removed, switch to use git
> 
> Signed-off-by: Changqing Li <changqing.li@windriver.com>
> ---
>  meta/recipes-devtools/dpkg/dpkg.inc       |  9 ++++++++-
>  meta/recipes-devtools/dpkg/dpkg_1.20.0.bb | 10 +++++++---
>  2 files changed, 15 insertions(+), 4 deletions(-)

The autobuilder didn't like this change :(

https://autobuilder.yoctoproject.org/typhoon/#/builders/83/builds/1143

Cheers,

Richard


^ permalink raw reply	[flat|nested] 8+ messages in thread

* [PATCH V4] dpkg: change SRC_URI to take dpkg from git
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
                     ` (2 preceding siblings ...)
  2020-07-10  2:08   ` [PATCH V3] " Changqing Li
@ 2020-07-10  9:21   ` Changqing Li
  3 siblings, 0 replies; 8+ messages in thread
From: Changqing Li @ 2020-07-10  9:21 UTC (permalink / raw)
  To: openembedded-core

From: Changqing Li <changqing.li@windriver.com>

tar.xz on original URL have been removed, switch to use git

Signed-off-by: Changqing Li <changqing.li@windriver.com>
---
 meta/recipes-devtools/dpkg/dpkg.inc       | 14 +++++++++++++-
 meta/recipes-devtools/dpkg/dpkg_1.20.0.bb |  7 ++++---
 2 files changed, 17 insertions(+), 4 deletions(-)

diff --git a/meta/recipes-devtools/dpkg/dpkg.inc b/meta/recipes-devtools/dpkg/dpkg.inc
index 1c3c585d79..04fe85c4a8 100644
--- a/meta/recipes-devtools/dpkg/dpkg.inc
+++ b/meta/recipes-devtools/dpkg/dpkg.inc
@@ -7,7 +7,7 @@ DEPENDS_class-native = "bzip2-replacement-native zlib-native virtual/update-alte
 RDEPENDS_${PN} = "${VIRTUAL-RUNTIME_update-alternatives} perl"
 RDEPENDS_${PN}_class-native = ""
 
-UPSTREAM_CHECK_URI = "${DEBIAN_MIRROR}/main/d/dpkg/"
+UPSTREAM_CHECK_GITTAGREGEX = "(?P<pver>(\d+(\.\d+)+))"
 
 inherit autotools gettext perlnative pkgconfig perl-version update-alternatives
 
@@ -30,6 +30,18 @@ EXTRA_OECONF_append_class-target = " --disable-update-alternatives DEB_HOST_ARCH
 PACKAGECONFIG = "liblzma"
 PACKAGECONFIG[liblzma] = "--with-liblzma,--without-liblzma, xz"
 
+
+#autotools.bbclass default AUTOTOOLS_AUXDIR is ${S}, we need to under ${S}/build-aux
+AUTOTOOLS_AUXDIR = "${S}/build-aux"
+
+do_configure_prepend () {
+        mkdir -p ${AUTOTOOLS_AUXDIR}
+        # autotools_do_configure updates po/Makefile.in.in, we also need
+        # update dselect/po and scripts/po
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/dselect/po/
+        cp -f ${STAGING_DATADIR_NATIVE}/gettext/po/Makefile.in.in ${S}/scripts/po/
+}
+
 do_install_append () {
 	if [ "${PN}" = "dpkg-native" ]; then
 		# update-alternatives doesn't have an offline mode
diff --git a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
index c98a9e58e2..334294e6f8 100644
--- a/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
+++ b/meta/recipes-devtools/dpkg/dpkg_1.20.0.bb
@@ -1,7 +1,7 @@
 require dpkg.inc
 LIC_FILES_CHKSUM = "file://COPYING;md5=751419260aa954499f7abaabaa882bbe"
 
-SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
+SRC_URI = "git://salsa.debian.org/dpkg-team/dpkg.git;protocol=https \
            file://noman.patch \
            file://remove-tar-no-timestamp.patch \
            file://arch_pm.patch \
@@ -18,5 +18,6 @@ SRC_URI = "${DEBIAN_MIRROR}/main/d/${BPN}/${BPN}_${PV}.tar.xz \
 
 SRC_URI_append_class-native = " file://0001-build.c-ignore-return-of-1-from-tar-cf.patch"
 
-SRC_URI[md5sum] = "f88f077236a3ff3decae3b25c989893d"
-SRC_URI[sha256sum] = "b633cc2b0e030efb61e11029d8a3fb1123f719864c9992da2e52b471c96d0900"
+SRCREV = "314ac02663c5bd1a82b34745150bf13a39a549a3"
+
+S = "${WORKDIR}/git"
-- 
2.17.1


^ permalink raw reply related	[flat|nested] 8+ messages in thread

* ✗ patchtest: failure for dpkg: update SRC_URI (rev2)
  2020-07-07  2:53 [PATCH] dpkg: update SRC_URI Changqing Li
  2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
@ 2020-07-09  9:02 ` Patchwork
  1 sibling, 0 replies; 8+ messages in thread
From: Patchwork @ 2020-07-09  9:02 UTC (permalink / raw)
  To: changqing.li; +Cc: openembedded-core

== Series Details ==

Series: dpkg: update SRC_URI (rev2)
Revision: 2
URL   : https://patchwork.openembedded.org/series/25025/
State : failure

== Summary ==

Thank you for submitting this patch series to OpenEmbedded Core. This is
an automated response. Several tests have been executed on the proposed
series by patchtest resulting in the following failures:

* Issue             Series does not apply on top of target branch [test_series_merge_on_head] 
  Suggested fix    Rebase your series on top of targeted branch
  Targeted branch  master (currently at b3c96103a5)

If you believe any of these test results are incorrect, please reply to the
mailing list (openembedded-core@lists.openembedded.org) raising your concerns.
Otherwise we would appreciate you correcting the issues and submitting a new
version of the patchset if applicable. Please ensure you add/increment the
version number when sending the new version (i.e. [PATCH] -> [PATCH v2] ->
[PATCH v3] -> ...).

---
Guidelines:     https://www.openembedded.org/wiki/Commit_Patch_Message_Guidelines
Test framework: http://git.yoctoproject.org/cgit/cgit.cgi/patchtest
Test suite:     http://git.yoctoproject.org/cgit/cgit.cgi/patchtest-oe

^ permalink raw reply	[flat|nested] 8+ messages in thread

end of thread, other threads:[~2020-07-10  9:24 UTC | newest]

Thread overview: 8+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2020-07-07  2:53 [PATCH] dpkg: update SRC_URI Changqing Li
2020-07-07  7:36 ` [OE-core] " Alexander Kanavin
2020-07-07  7:41   ` Changqing Li
2020-07-09  8:54   ` [PATCH] dpkg: change SRC_URI to take dpkg from git Changqing Li
2020-07-10  2:08   ` [PATCH V3] " Changqing Li
2020-07-10  6:02     ` [OE-core] " Richard Purdie
2020-07-10  9:21   ` [PATCH V4] " Changqing Li
2020-07-09  9:02 ` ✗ patchtest: failure for dpkg: update SRC_URI (rev2) Patchwork

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox