Upstream version check fixes for various recipes

PATCH In review Alexander Kanavin 7 participants 69 messages Jul 30, 2015 – Aug 7, 2015

Share & tools

Alexander Kanavin

author

2015-07-30 13:31 UTC #0 lore raw

cover Upstream version check fixes for various recipes

This patchset contains various recipe fixes that allow upstream version checks. It also updates recipes that fetch sources from Google Code (about to shut down) to use alternatives. Please see individual commits for what is fixed and why.

The following changes since commit 3aeeda88e236f3da577adc20597dab38a66157c5:

wget.py: fix incorrect regexes (2015-07-30 14:47:12 +0300)

are available in the git repository at:

git://git.yoctoproject.org/poky-contrib akanavin/recipe-reporting-fixes
http://git.yoctoproject.org/cgit.cgi/poky-contrib/log/?h=akanavin/recipe-reporting-fixes

Alexander Kanavin (26):
libproxy: update SRC_URI to not use Google Code
fotowall: update SRC_URI to use github instead of qt-apps.org
distcc: update SRC_URI to not use Google Code
adt-installer: update SRC_URI to not use Google Code
ossp-uuid: update SRC_URI to not use Google Code
eglinfo: change version from 1.0 to 1.0.0 to match the upstream git
tag
gptfdisk: use tarballs instead of git for fetching source code
libmad: use ftp instead of sourceforge for consistency with libid3tag
lsb: set PV to 1.4 to match upstream tarball version
linux-yocto-dev: only numeric characters are allowed in package
versions taken from git
nss: reorder SRC_URI lines to fix upstream version check
pseudo: fix upstream SRC_URI to enable upstream version check
oprofile: reorder SRC_URI lines to fix upstream version check
puzzles: switch svn for git, as upstream has done the same
syslinux: drop v6.xx/ from directory path in SRC_URI
shared-mime-info: reorder SRC_URI lines to fix upstream version check
tremor: switch SRC_URI from svn to tarballs on debian mirrors
x264: only numeric characters are allowed in package versions taken
from git
xf86-video-omapfb: switch SRC_URI to debian git repo
stress: use developer's website for SRC_URI
systemd: remove PV from the recipe
unzip,zip: fix PV to allow upstream version check
guilt-native: set PV in the recipe to allow upstream version check distrodata.bbclass: do not skip -native recipes in checkpkg task distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before
comparing versions in checkpkg
distrodata.bbclass: drop the manual upstream version check logic

 meta/classes/distrodata.bbclass                    | 27 ++-----------------
 meta/recipes-core/systemd/systemd_219.bb           |  2 --
 meta/recipes-devtools/distcc/distcc_3.1.bb         |  3 ++-
 .../fdisk/{gptfdisk_git.bb => gptfdisk_0.8.10.bb}  |  8 +++---
 meta/recipes-devtools/guilt/guilt-native_git.bb    |  2 +-
 .../installer/adt-installer_1.0.bb                 |  2 +-
 meta/recipes-devtools/ossp-uuid/ossp-uuid_1.6.2.bb |  2 +-
 meta/recipes-devtools/pseudo/pseudo_1.6.7.bb       |  2 +-
 meta/recipes-devtools/syslinux/syslinux_6.03.bb    |  2 +-
 meta/recipes-extended/lsb/lsb_4.1.bb               |  1 +
 meta/recipes-extended/stress/stress_1.0.4.bb       |  2 +-
 meta/recipes-extended/unzip/unzip_6.0.bb           |  3 ++-
 meta/recipes-extended/zip/zip.inc                  |  3 ++-
 .../{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb}     |  0
 .../{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb}   |  0
 .../xorg-driver/xf86-video-omapfb_git.bb           |  2 +-
 meta/recipes-kernel/linux/linux-yocto-dev.bb       |  2 +-
 meta/recipes-kernel/oprofile/oprofile.inc          |  2 +-
 meta/recipes-kernel/oprofile/oprofile_1.0.0.bb     |  6 ++---
 meta/recipes-multimedia/libmad/libmad_0.15.1b.bb   |  2 +-
 .../obsolete_automake_macros.patch                 |  0
 .../tremor-arm-thumb2.patch                        |  0
 .../tremor/{tremor_20120314.bb => tremor_18153.bb} |  6 +++--
 meta/recipes-multimedia/x264/x264_git.bb           |  2 +-
 meta/recipes-qt/qt-apps/fotowall_0.9.bb            |  3 ++-
 .../puzzles/files/puzzles_x32_abi_time.patch       | 30 ----------------------
 .../puzzles/{puzzles_r10286.bb => puzzles_git.bb}  |  7 ++---
 meta/recipes-support/libproxy/libproxy_0.4.11.bb   |  7 +++--
 meta/recipes-support/nss/nss.inc                   |  2 +-
 meta/recipes-support/nss/nss_3.17.3.bb             |  6 ++---
 .../shared-mime-info/shared-mime-info_1.4.bb       |  2 +-
 31 files changed, 44 insertions(+), 94 deletions(-)
 rename meta/recipes-devtools/fdisk/{gptfdisk_git.bb => gptfdisk_0.8.10.bb} (75%)
 rename meta/recipes-graphics/eglinfo/{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb} (100%)
 rename meta/recipes-graphics/eglinfo/{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb} (100%)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/obsolete_automake_macros.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/tremor-arm-thumb2.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor_20120314.bb => tremor_18153.bb} (66%)
 delete mode 100644 meta/recipes-sato/puzzles/files/puzzles_x32_abi_time.patch
 rename meta/recipes-sato/puzzles/{puzzles_r10286.bb => puzzles_git.bb} (89%)

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #1 lore raw

PATCH 1/26 libproxy: update SRC_URI to not use Google Code

Google Code is shutting down soon, so libproxy tarball is now fetched from Debian mirrors

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-support/libproxy/libproxy_0.4.11.bb | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)

meta/recipes-support/libproxy/libproxy_0.4.11.bb +3 -4

index 9d388d6..06d0b9b 100644
@@ -8,12 +8,11 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c \
 
 DEPENDS = "gconf glib-2.0"
 
-SRC_URI = "http://libproxy.googlecode.com/files/libproxy-${PV}.zip \
+SRC_URI = "${DEBIAN_MIRROR}/main/libp/${BPN}/${BPN}_${PV}.orig.tar.gz \
            file://0001-test-Include-sys-select.h-for-select.patch \
           "
-
-SRC_URI[md5sum] = "06be230b8b612e64997f8f15af6373aa"
-SRC_URI[sha256sum] = "3175abcb49c0b37fd82c697927783bf7d53de3884a8559bc594e59caa09bdb04"
+SRC_URI[md5sum] = "3cd1ae2a4abecf44b3f24d6639d2cd84"
+SRC_URI[sha256sum] = "dc3f33de54163718f82b3e7c496a7de97f8862578414b8ecaad3cbfe4821864b"
 
 inherit cmake pkgconfig

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #6 lore raw

PATCH 2/26 fotowall: update SRC_URI to use github instead of qt-apps.org

In reply to Alexander Kanavin

GitHub is where upstream development is happening, so we are more likely to find out about new releases from there

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-qt/qt-apps/fotowall_0.9.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

meta/recipes-qt/qt-apps/fotowall_0.9.bb +2 -1

index 48dcc02..587b074 100644
@@ -10,7 +10,8 @@ SECTION = "x11/apps"
 
 PR = "r4"
 
-SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2 \
+SRCREV = "06d5a4142c599604d9a9fd64727b6945eb8cd3da"
+SRC_URI = "git://github.com/enricoros/fotowall.git \
            file://ExportWizard-depends-on-ui_wizard.patch \
            "

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #9 lore raw

PATCH 3/26 distcc: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Google Code is shutting down soon and distcc has moved to github.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/distcc/distcc_3.1.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

meta/recipes-devtools/distcc/distcc_3.1.bb +2 -1

index e4b9313..b598c77 100644
@@ -17,13 +17,14 @@ PACKAGECONFIG[popt] = "--without-included-popt,--with-included-popt,popt"
 
 RRECOMMENDS_${PN} = "avahi-daemon"
 
-SRC_URI = "http://distcc.googlecode.com/files/${BPN}-${PV}.tar.bz2 \
+SRC_URI = "git://github.com/distcc/distcc.git \
            file://separatebuilddir.patch \
            file://0001-zeroconf-Include-fcntl.h.patch \
            file://default \
            file://distccmon-gnome.desktop \
            file://distcc \
            file://distcc.service"
+SRCREV = "9a172d3d20042754a30c0cd9a5d63fd1a8120c31"
 
 SRC_URI[md5sum] = "a1a9d3853df7133669fffec2a9aab9f3"
 SRC_URI[sha256sum] = "f55dbafd76bed3ce57e1bbcdab1329227808890d90f4c724fcd2d53f934ddd89"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #11 lore raw

PATCH 4/26 adt-installer: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Google Code is shutting down and opkg tarballs are now available from yoctoproject.org

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/installer/adt-installer_1.0.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-devtools/installer/adt-installer_1.0.bb +1 -1

index 07070b5..c871e42 100644
@@ -38,7 +38,7 @@ ADT_DIR = "${WORKDIR}/adt-installer/"
 S = "${WORKDIR}/opkg-${PV}"
 
 PV = "0.2.0"
-SRC_URI = "https://opkg.googlecode.com/files/opkg-${PV}.tar.gz \
+SRC_URI = "http://downloads.yoctoproject.org/releases/opkg/opkg-${PV}.tar.gz \
            file://wget_cache.patch \
            file://adt_installer \
            file://scripts/adt_installer_internal \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #12 lore raw

PATCH 5/26 ossp-uuid: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Google Code is shutting down so tarballs will be fetched from a Debian mirror instead

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/ossp-uuid/ossp-uuid_1.6.2.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-devtools/ossp-uuid/ossp-uuid_1.6.2.bb +1 -1

index 33bed75..08926ba 100644
@@ -17,7 +17,7 @@ LIC_FILES_CHKSUM = "file://README;beginline=30;endline=55;md5=b394fadb039bbfca6a
 
 PR = "r2"
 
-SRC_URI = "http://gnome-build-stage-1.googlecode.com/files/uuid-1.6.2.tar.gz \
+SRC_URI = "${DEBIAN_MIRROR}/main/o/${BPN}/${BPN}_${PV}.orig.tar.gz \
 	   file://0001-Change-library-name.patch \
 	   file://0002-uuid-preserve-m-option-status-in-v-option-handling.patch \
 	   file://0003-Fix-whatis-entries.patch \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #13 lore raw

PATCH 6/26 eglinfo: change version from 1.0 to 1.0.0 to match the upstream git tag

In reply to Alexander Kanavin

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-graphics/eglinfo/{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb}  | 0
 .../recipes-graphics/eglinfo/{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb} | 0
 2 files changed, 0 insertions(+), 0 deletions(-)
 rename meta/recipes-graphics/eglinfo/{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb} (100%)
 rename meta/recipes-graphics/eglinfo/{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb} (100%)

meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.0.bb renamed from meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.bb +0 -0

similarity index 100%
rename from meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.bb
rename to meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.0.bb

meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.0.bb renamed from meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.bb +0 -0

similarity index 100%
rename from meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.bb
rename to meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.0.bb

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #17 lore raw

PATCH 7/26 gptfdisk: use tarballs instead of git for fetching source code

In reply to Alexander Kanavin

git repository does not have version tags, so upstream version check cannot be performed

Signed-off-by Alexander Kanavin <[email protected]>

---
 .../fdisk/{gptfdisk_git.bb => gptfdisk_0.8.10.bb}                 | 8 +++-----
 1 file changed, 3 insertions(+), 5 deletions(-)
 rename meta/recipes-devtools/fdisk/{gptfdisk_git.bb => gptfdisk_0.8.10.bb} (75%)

meta/recipes-devtools/fdisk/gptfdisk_0.8.10.bb renamed from meta/recipes-devtools/fdisk/gptfdisk_git.bb +3 -5

similarity index 75%
rename from meta/recipes-devtools/fdisk/gptfdisk_git.bb
rename to meta/recipes-devtools/fdisk/gptfdisk_0.8.10.bb
index 477c1d9..3b94b56 100644
@@ -5,11 +5,9 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=59530bdf33659b29e73d4adb9f9f6552"
 
 DEPENDS = "util-linux popt ncurses"
 
-PV = "0.8.10+git${SRCPV}"
-SRCREV = "a920398fa393f9d6301b32b191bc01e086ab8bc8"
-SRC_URI = "git://git.code.sf.net/p/gptfdisk/code"
-
-S = "${WORKDIR}/git"
+SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${PV}/${BP}.tar.gz"
+SRC_URI[md5sum] = "9cf4246c181c324bdbd553fe9b348373"
+SRC_URI[sha256sum] = "73e64151203ae0c347c488358e71ca582bb7fb7f0d66df86b71c42050390eb9b"
 
 do_install() {
     install -d ${D}${sbindir}

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #18 lore raw

PATCH 8/26 libmad: use ftp instead of sourceforge for consistency with libid3tag

In reply to Alexander Kanavin

Both ftp and SF are official code sources, but ftp is less finicky for upstream version checks.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-multimedia/libmad/libmad_0.15.1b.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-multimedia/libmad/libmad_0.15.1b.bb +1 -1

index 30937eb..9e08b16 100644
@@ -10,7 +10,7 @@ SECTION = "libs"
 DEPENDS = "libid3tag"
 PR = "r3"
 
-SRC_URI = "${SOURCEFORGE_MIRROR}/mad/libmad-${PV}.tar.gz \
+SRC_URI = "ftp://ftp.mars.org/pub/mpeg/libmad-${PV}.tar.gz \
            file://no-force-mem.patch \
            file://add-pkgconfig.patch \
            file://fix_for_mips_with_gcc-4.5.0.patch \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #19 lore raw

PATCH 9/26 lsb: set PV to 1.4 to match upstream tarball version

In reply to Alexander Kanavin

This allows upstream version check to work

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-extended/lsb/lsb_4.1.bb | 1 +
 1 file changed, 1 insertion(+)

meta/recipes-extended/lsb/lsb_4.1.bb +1 -0

index 6215b62..171a6c8 100644
@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
            file://lsb_pidofproc \
            file://lsb_start_daemon \
            "
+PV = "1.4"
 
 SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
 SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #25 lore raw

PATCH 10/26 linux-yocto-dev: only numeric characters are allowed in package versions taken from git

In reply to Alexander Kanavin

See regexes in get_recipe_pv_without_srcpv() in meta/lib/oe/recipeutils.py

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-kernel/linux/linux-yocto-dev.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-kernel/linux/linux-yocto-dev.bb +1 -1

index 0be3089..ab094cc 100644
@@ -36,7 +36,7 @@ SRC_URI = "git://git.yoctoproject.org/linux-yocto-dev.git;branch=${KBRANCH};name
 SRCREV_machine ?= '${@oe.utils.conditional("PREFERRED_PROVIDER_virtual/kernel", "linux-yocto-dev", "${AUTOREV}", "29594404d7fe73cd80eaa4ee8c43dcc53970c60e", d)}'
 SRCREV_meta ?= '${@oe.utils.conditional("PREFERRED_PROVIDER_virtual/kernel", "linux-yocto-dev", "${AUTOREV}", "29594404d7fe73cd80eaa4ee8c43dcc53970c60e", d)}'
 
-LINUX_VERSION ?= "4.1-rc+"
+LINUX_VERSION ?= "4.1"
 LINUX_VERSION_EXTENSION ?= "-yoctodev-${LINUX_KERNEL_TYPE}"
 PV = "${LINUX_VERSION}+git${SRCPV}"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #26 lore raw

PATCH 11/26 nss: reorder SRC_URI lines to fix upstream version check

In reply to Alexander Kanavin

The source tarball should be listed first

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-support/nss/nss.inc       | 2 +-
 meta/recipes-support/nss/nss_3.17.3.bb | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

meta/recipes-support/nss/nss.inc +1 -1

index 665e4e6..b21dc39 100644
@@ -12,7 +12,7 @@ LICENSE = "MPL-2.0 | (MPL-2.0 & GPL-2.0+) | (MPL-2.0 & LGPL-2.1+)"
 
 LIC_FILES_CHKSUM = "file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
                     file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=6bf96825e3d7ce4de25621ae886cc859"
-SRC_URI = "\
+SRC_URI += "\
     file://nss-fix-support-cross-compiling.patch \
     file://nss-no-rpath-for-cross-compiling.patch \
     file://nss-fix-incorrect-shebang-of-perl.patch \

meta/recipes-support/nss/nss_3.17.3.bb +3 -3

index e7d0780..b22eb95 100644
@@ -1,7 +1,7 @@
-require nss.inc
-
-SRC_URI += "\
+SRC_URI = "\
     http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/NSS_3_17_3_RTM/src/${BP}.tar.gz \
 "
 SRC_URI[md5sum] = "fba7489e1b26f2a0bfe5527430fd61e1"
 SRC_URI[sha256sum] = "f4d5e9035a2f84f25f35c283de3b0ff60d72e918748de25eaf017ed201fa21d5"
+
+require nss.inc

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #30 lore raw

PATCH 12/26 pseudo: fix upstream SRC_URI to enable upstream version check

In reply to Alexander Kanavin

Previous URI did not allow directory listings.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/pseudo/pseudo_1.6.7.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-devtools/pseudo/pseudo_1.6.7.bb +1 -1

index dc55950..b8c20ad 100644
@@ -1,7 +1,7 @@
 require pseudo.inc
 
 SRC_URI = " \
-    http://www.yoctoproject.org/downloads/${BPN}/${BPN}-${PV}.tar.bz2 \
+    http://downloads.yoctoproject.org/releases/pseudo/${BPN}-${PV}.tar.bz2 \
     file://fallback-passwd \
     file://fallback-group \
 "

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #31 lore raw

PATCH 13/26 oprofile: reorder SRC_URI lines to fix upstream version check

In reply to Alexander Kanavin

The source tarball should be listed first.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-kernel/oprofile/oprofile.inc      | 2 +-
 meta/recipes-kernel/oprofile/oprofile_1.0.0.bb | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

meta/recipes-kernel/oprofile/oprofile.inc +1 -1

index 49e6b16..ff360d1 100644
@@ -18,7 +18,7 @@ FILES_${PN} = "${bindir} ${libdir}/${BPN}/lib*${SOLIBS} ${datadir}/${BPN}"
 FILES_${PN}-dev += "${libdir}/${BPN}/lib*${SOLIBSDEV} ${libdir}/${BPN}/lib*.la"
 FILES_${PN}-staticdev += "${libdir}/${BPN}/lib*.a"
 
-SRC_URI = "file://filemode-fix.patch \
+SRC_URI += "file://filemode-fix.patch \
            file://acinclude.m4 \
            file://automake-foreign.patch \
            file://oprofile-cross-compile-tests.patch \

meta/recipes-kernel/oprofile/oprofile_1.0.0.bb +3 -3

index f4a34b3..f75a1dd 100644
@@ -1,13 +1,13 @@
-require oprofile.inc
-
 DEPENDS += "virtual/kernel"
 DEPENDS_append_powerpc64 = " libpfm4"
 
-SRC_URI += "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
+SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
 
 SRC_URI[md5sum] = "ba0b340e5c421a93959776c836ed35b3"
 SRC_URI[sha256sum] = "847110b4ecdcf8c8353cd38f94c1b704aad4bfcd9453e38b88d112cfb7e3c45a"
 
+require oprofile.inc
+
 S = "${WORKDIR}/oprofile-${PV}"
 
 PR = "r1"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #34 lore raw

PATCH 14/26 puzzles: switch svn for git, as upstream has done the same

In reply to Alexander Kanavin

Also, drop puzzles_x32_abi_time.patch as it has been merged upstream

Signed-off-by Alexander Kanavin <[email protected]>

---
 .../puzzles/files/puzzles_x32_abi_time.patch       | 30 ----------------------
 .../puzzles/{puzzles_r10286.bb => puzzles_git.bb}  |  7 ++---
 2 files changed, 4 insertions(+), 33 deletions(-)
 delete mode 100644 meta/recipes-sato/puzzles/files/puzzles_x32_abi_time.patch
 rename meta/recipes-sato/puzzles/{puzzles_r10286.bb => puzzles_git.bb} (89%)

meta/recipes-sato/puzzles/files/puzzles_x32_abi_time.patch deleted +0 ~~-30~~

deleted file mode 100644
index 63ab0b8..0000000
@@ -1,30 +0,0 @@
-Fix printf's for time_t value add castings because ISOC90
-don't support long long int that is used in x32 ABI for 
-time_t.
-
-Upstream-Status: Backport
-
-Signed-off-by: Aníbal Limón <[email protected]>
-
---- a/magnets.c	2015-03-15 11:57:39.106674811 +0000
-+++ b/magnets.c	2015-03-15 11:57:58.638674795 +0000
-@@ -2562,7 +2562,7 @@
-         goto done;
-     }
-     s = new_game(NULL, p, desc);
--    printf("%s:%s (seed %ld)\n", id, desc, seed);
-+    printf("%s:%s (seed %ld)\n", id, desc, (long) seed);
-     if (aux) {
-         /* We just generated this ourself. */
-         if (verbose || print) {
---- a/signpost.c	2015-03-15 11:58:52.866674751 +0000
-+++ b/signpost.c	2015-03-15 11:59:08.190674738 +0000
-@@ -2393,7 +2393,7 @@
-         }
-     }
- 
--    sprintf(newseed, "%lu", time(NULL));
-+    sprintf(newseed, "%lu", (unsigned long) time(NULL));
-     seedstr = dupstr(newseed);
- 
-     if (id || !stdin_desc) {

meta/recipes-sato/puzzles/puzzles_git.bb renamed from meta/recipes-sato/puzzles/puzzles_r10286.bb +4 -3

similarity index 89%
rename from meta/recipes-sato/puzzles/puzzles_r10286.bb
rename to meta/recipes-sato/puzzles/puzzles_git.bb
index 1ad8d1b..6346c06 100644
@@ -10,12 +10,13 @@ REQUIRED_DISTRO_FEATURES = "x11"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://LICENCE;md5=33bcd4bce8f3c197f2aefbdbd2d299bc"
 
-SRC_URI = "svn://svn.tartarus.org/sgt;module=puzzles;rev=${MOD_PV} \
+SRC_URI = "git://git.tartarus.org/simon/puzzles.git \
            file://fix-compiling-failure-with-option-g-O.patch \
-           file://puzzles_x32_abi_time.patch \
 "
+SRCREV = "aa1a9375be96cd8e0f89a670359c3c80ad7cf937"
+PV = "0.0+git${SRCPV}"
 
-S = "${WORKDIR}/${BPN}"
+S = "${WORKDIR}/git"
 
 inherit autotools-brokensep distro_features_check

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #39 lore raw

PATCH 15/26 syslinux: drop v6.xx/ from directory path in SRC_URI

In reply to Alexander Kanavin

This allows upstream version check.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/syslinux/syslinux_6.03.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-devtools/syslinux/syslinux_6.03.bb +1 -1

index c42f05d..ef9ae2f 100644
@@ -8,7 +8,7 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=0636e73ff0215e8d672dc4c32c317bb3 \
 # ldlinux.* stuff for now, so skip mtools-native
 DEPENDS = "nasm-native util-linux e2fsprogs"
 
-SRC_URI = "${KERNELORG_MIRROR}/linux/utils/boot/syslinux/6.xx/syslinux-${PV}.tar.xz \
+SRC_URI = "${KERNELORG_MIRROR}/linux/utils/boot/syslinux/syslinux-${PV}.tar.xz \
            file://syslinux-fix-parallel-building-issue.patch \
            file://syslinux-libupload-depend-lib.patch \
            file://syslinux-remove-clean-script.patch \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #40 lore raw

PATCH 16/26 shared-mime-info: reorder SRC_URI lines to fix upstream version check

In reply to Alexander Kanavin

Upstream tarball should be listed first.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-support/shared-mime-info/shared-mime-info_1.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-support/shared-mime-info/shared-mime-info_1.4.bb +1 -1

index 3797972..8c4fde8 100644
@@ -1,6 +1,6 @@
 require shared-mime-info.inc
 
-SRC_URI =+ "file://parallelmake.patch \
+SRC_URI += "file://parallelmake.patch \
 	    file://install-data-hook.patch"
 
 SRC_URI[md5sum] = "16c02f7b658fff2a9c207406d388ea31"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #41 lore raw

PATCH 17/26 tremor: switch SRC_URI from svn to tarballs on debian mirrors

In reply to Alexander Kanavin

There is no support for checking svn repositories for new commits, so let's use debian tarballs and assume that Debian has a dedicated maintainer who would check the upstream now and then.

Signed-off-by Alexander Kanavin <[email protected]>

---
 .../{tremor-20120314 => tremor}/obsolete_automake_macros.patch      | 0
 .../tremor/{tremor-20120314 => tremor}/tremor-arm-thumb2.patch      | 0
 .../tremor/{tremor_20120314.bb => tremor_18153.bb}                  | 6 ++++--
 3 files changed, 4 insertions(+), 2 deletions(-)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/obsolete_automake_macros.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/tremor-arm-thumb2.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor_20120314.bb => tremor_18153.bb} (66%)

meta/recipes-multimedia/tremor/tremor/obsolete_automake_macros.patch renamed from meta/recipes-multimedia/tremor/tremor-20120314/obsolete_automake_macros.patch +0 -0

similarity index 100%
rename from meta/recipes-multimedia/tremor/tremor-20120314/obsolete_automake_macros.patch
rename to meta/recipes-multimedia/tremor/tremor/obsolete_automake_macros.patch

meta/recipes-multimedia/tremor/tremor/tremor-arm-thumb2.patch renamed from meta/recipes-multimedia/tremor/tremor-20120314/tremor-arm-thumb2.patch +0 -0

similarity index 100%
rename from meta/recipes-multimedia/tremor/tremor-20120314/tremor-arm-thumb2.patch
rename to meta/recipes-multimedia/tremor/tremor/tremor-arm-thumb2.patch

meta/recipes-multimedia/tremor/tremor_18153.bb renamed from meta/recipes-multimedia/tremor/tremor_20120314.bb +4 -2

similarity index 66%
rename from meta/recipes-multimedia/tremor/tremor_20120314.bb
rename to meta/recipes-multimedia/tremor/tremor_18153.bb
index afc78ac..c27fe8c 100644
@@ -8,12 +8,14 @@ DEPENDS = "libogg"
 SRCDATE = "${PV}"
 PR = "r1"
 
-SRC_URI = "svn://svn.xiph.org/trunk;module=Tremor;rev=18221;protocol=http \
+SRC_URI = "${DEBIAN_MIRROR}/main/libv/libvorbisidec/libvorbisidec_1.0.2+svn${PV}.orig.tar.gz \
            file://obsolete_automake_macros.patch;striplevel=0 \
            file://tremor-arm-thumb2.patch \
 "
+SRC_URI[md5sum] = "4190859414c5d6760e316b5cf00fe7c5"
+SRC_URI[sha256sum] = "4dc8c224289da3479fc10ce4e49ffbb85c790eb2fe55ef480934a265ee0a6782"
 
-S = "${WORKDIR}/Tremor"
+S = "${WORKDIR}/libvorbisidec-1.0.2+svn${PV}"
 
 inherit autotools pkgconfig

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #43 lore raw

PATCH 18/26 x264: only numeric characters are allowed in package versions taken from git

In reply to Alexander Kanavin

See regexes in get_recipe_pv_without_srcpv() in meta/lib/oe/recipeutils.py

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-multimedia/x264/x264_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-multimedia/x264/x264_git.bb +1 -1

index 0a69909..cd29d72 100644
@@ -14,7 +14,7 @@ SRC_URI = "git://git.videolan.org/x264.git \
 
 SRCREV = "c8a773ebfca148ef04f5a60d42cbd7336af0baf6"
 
-PV = "r2491+git${SRCPV}"
+PV = "2491+git${SRCPV}"
 
 S = "${WORKDIR}/git"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #44 lore raw

PATCH 19/26 xf86-video-omapfb: switch SRC_URI to debian git repo

In reply to Alexander Kanavin

The previous upstream is no longer available.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-graphics/xorg-driver/xf86-video-omapfb_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-graphics/xorg-driver/xf86-video-omapfb_git.bb +1 -1

index 4ad2538..e649270 100644
@@ -13,7 +13,7 @@ SRCREV = "28c006c94e57ea71df11ec4fff79d7ffcfc4860f"
 PR = "${INC_PR}.7"
 PV = "0.1.1+gitr${SRCPV}"
 
-SRC_URI = "git://git.pingu.fi/xf86-video-omapfb;protocol=http \
+SRC_URI = "git://anonscm.debian.org/collab-maint/xf86-video-omapfb.git \
   file://0001-Revert-Set-a-large-CRTC-upper-limit-to-not-prune-lar.patch \
   file://0002-Revert-Set-virtual-size-when-configuring-framebuffer.patch \
   file://0003-force-plain-mode.patch \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #46 lore raw

PATCH 20/26 stress: use developer's website for SRC_URI

In reply to Alexander Kanavin

So that upstream version check happens at the upstream, not at Fedora's build servers.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-extended/stress/stress_1.0.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-extended/stress/stress_1.0.4.bb +1 -1

index 44f2fc6..df96fc0 100644
@@ -4,7 +4,7 @@ HOMEPAGE = "http://people.seas.harvard.edu/~apw/stress/"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 
-SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/stress/stress-${PV}.tar.gz/a607afa695a511765b40993a64c6e2f4/stress-${PV}.tar.gz \
+SRC_URI = "http://people.seas.harvard.edu/~apw/stress/stress-${PV}.tar.gz \
            file://texinfo.patch \
            "

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #49 lore raw

PATCH 21/26 systemd: remove PV from the recipe

In reply to Alexander Kanavin

It is unnecessary because the recipe filename already contains it.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-core/systemd/systemd_219.bb | 2 --
 1 file changed, 2 deletions(-)

meta/recipes-core/systemd/systemd_219.bb +0 -2

index 83e4bce..148a077 100644
@@ -27,8 +27,6 @@ inherit gtk-doc useradd pkgconfig autotools perlnative update-rc.d update-altern
 
 SRCREV = "85a6fabdd3e43cfab0fc6359e9f2a9e368d4a3ed"
 
-PV = "219-stable+git${SRCPV}"
-
 SRC_URI = "git://anongit.freedesktop.org/systemd/systemd-stable;branch=v219-stable;protocol=git \
            file://0002-shared-missing.h-fall-back-to-insecure-getenv.patch \
            file://0003-binfmt-Don-t-install-dependency-links-at-install-tim.patch \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #59 lore raw

PATCH 22/26 unzip, zip: fix PV to allow upstream version check

In reply to Alexander Kanavin

Upstream does not separate major and minor version numbers in the tarball filename, so the best we can do is to squash them together as well :-/

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-extended/unzip/unzip_6.0.bb | 3 ++-
 meta/recipes-extended/zip/zip.inc        | 3 ++-
 2 files changed, 4 insertions(+), 2 deletions(-)

meta/recipes-extended/unzip/unzip_6.0.bb +2 -1

index 4a0a713..8ac3745 100644
@@ -6,7 +6,8 @@ LIC_FILES_CHKSUM = "file://LICENSE;md5=94caec5a51ef55ef711ee4e8b1c69e29"
 PE = "1"
 PR = "r5"
 
-SRC_URI = "ftp://ftp.info-zip.org/pub/infozip/src/unzip60.tgz \
+PV = "60"
+SRC_URI = "ftp://ftp.info-zip.org/pub/infozip/src/unzip${PV}.tgz \
 	file://avoid-strip.patch \
 	file://define-ldflags.patch \
 	file://06-unzip60-alt-iconv-utf8_CVE-2015-1315.patch \

meta/recipes-extended/zip/zip.inc +2 -1

index 6221c5e..5e92daa 100644
@@ -5,7 +5,8 @@ SECTION = "console/utils"
 LICENSE = "BSD-3-Clause"
 LIC_FILES_CHKSUM = "file://LICENSE;md5=04d43c5d70b496c032308106e26ae17d"
 
-SRC_URI = "ftp://ftp.info-zip.org/pub/infozip/src/zip${@d.getVar('PV',1).replace('.', '')}.tgz"
+PV = "30"
+SRC_URI = "ftp://ftp.info-zip.org/pub/infozip/src/zip${PV}.tgz"
 
 EXTRA_OEMAKE = "'CC=${CC}' 'BIND=${CC}' 'AS=${CC} -c' 'CPP=${CPP}' \
 		'CFLAGS=-I. -DUNIX ${CFLAGS}' 'INSTALL=install' \

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #60 lore raw

PATCH 23/26 guilt-native: set PV in the recipe to allow upstream version check

In reply to Alexander Kanavin

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/guilt/guilt-native_git.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-devtools/guilt/guilt-native_git.bb +1 -1

index dfb3378..b63c68c 100644
@@ -8,7 +8,7 @@ inherit native
 SRC_URI = "git://repo.or.cz/guilt.git \
            file://guilt-bash.patch \
            "
-
+PV = "0.35+git${SRCPV}"
 SRCREV = "c2a5bae511c6d5354aa4e1cb59069c31df2b8eeb"
 
 S = "${WORKDIR}/git"

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #61 lore raw

PATCH 24/26 distrodata.bbclass: do not skip -native recipes in checkpkg task

In reply to Alexander Kanavin

Some recipes are available only in the -native flavour, and we need to check their upstream version too from the command line.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/classes/distrodata.bbclass | 3 ---
 1 file changed, 3 deletions(-)

meta/classes/distrodata.bbclass +0 -3

index a228242..af3af33 100644
@@ -218,9 +218,6 @@ python checkpkg_eventhandler() {
                     header = row
                     continue
 
-                if '-native' in pn or 'nativesdk-' in pn:
-                    continue
-
                 if not pn in package_dict.keys():
                     package_dict[pn] = row
             f.close()

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #62 lore raw

PATCH 25/26 distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

In reply to Alexander Kanavin

That function is intended for use with git sources and contains some restrictive regexes that don't work correctly when there are alphabetic characters in the version string. We can simply compare the versions directly.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/classes/distrodata.bbclass | 5 +----
 1 file changed, 1 insertion(+), 4 deletions(-)

meta/classes/distrodata.bbclass +1 -4

index af3af33..d33187b 100644
@@ -341,10 +341,7 @@ python do_checkpkg() {
         if not pupver:
             pupver = "N/A"
         else:
-            pv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pversion, uri_type)
-            upv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pupver, uri_type)
-
-            cmp = vercmp_string(pv, upv)
+            cmp = vercmp_string(pversion, pupver)
             if cmp == -1:
                 pstatus = "UPDATE"
             elif cmp == 0:

Signature · 1 line

2.1.4

Alexander Kanavin

author

2015-07-30 13:31 UTC #68 lore raw

PATCH 26/26 distrodata.bbclass: drop the manual upstream version check logic

In reply to Alexander Kanavin

It's already taken care of in meta/lib/oe/recipeutils.py

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/classes/distrodata.bbclass | 19 +------------------
 1 file changed, 1 insertion(+), 18 deletions(-)

meta/classes/distrodata.bbclass +1 ~~-18~~

index d33187b..7cb7736 100644
@@ -347,23 +347,6 @@ python do_checkpkg() {
             elif cmp == 0:
                 pstatus = "MATCH"
 
-        """Read from manual distro tracking fields as alternative"""
-        pmver = d.getVar("RECIPE_UPSTREAM_VERSION", True)
-        if not pmver:
-            pmver = "N/A"
-            pmstatus = "ErrNoRecipeData"
-        else:
-            mpv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pmver, uri_type)
-            upv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pupver, uri_type)
-
-            cmp = vercmp_string(mpv, upv)
-            if cmp == -1:
-                pmstatus = "UPDATE"
-            elif cmp == 0:
-                pmstatus = "MATCH"
-            else:
-                pmstatus = ""
-
         psrcuri = psrcuri.split()[0]
         pdepends = "".join(pdepends.split("\t"))
         pdesc = "".join(pdesc.split("\t"))

@@ -372,7 +355,7 @@ python do_checkpkg() {
         with open(logfile, "a") as f:
             writer = csv.writer(f, delimiter='\t')
             writer.writerow([pname, pversion, pupver, plicense, psection, phome, 
-                prelease, pdepends, pbugtracker, ppe, pdesc, pstatus, pmver, 
+                prelease, pdepends, pbugtracker, ppe, pdesc, pstatus, pupver,
                 psrcuri, maintainer, no_upgr_reason])
             f.close()
         bb.utils.unlockfile(lf)

Signature · 1 line

2.1.4

Martin Jansa

2015-07-30 14:28 UTC #2 lore raw

Re: [PATCH 01/26] libproxy: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Quoted 15 lines · Alexander Kanavin

Google Code is shutting down soon, so libproxy tarball is now fetched from Debian mirrors
Signed-off-by Alexander Kanavin <[email protected]>
---
 meta/recipes-support/libproxy/libproxy_0.4.11.bb | 7 +++----
 1 file changed, 3 insertions(+), 4 deletions(-)
meta/recipes-support/libproxy/libproxy_0.4.11.bb +0 -0
index 9d388d6..06d0b9b 100644
@@ -8,12 +8,11 @@ LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c \
 
 DEPENDS = "gconf glib-2.0"

meta/recipes-support/libproxy/libproxy_0.4.11.bb LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c \ · line 15

Quoted 10 lines

meta/recipes-support/libproxy/libproxy_0.4.11.bb +3 -4

 
-SRC_URI = "http://libproxy.googlecode.com/files/libproxy-${PV}.zip \
+SRC_URI = "${DEBIAN_MIRROR}/main/libp/${BPN}/${BPN}_${PV}.orig.tar.gz \
            file://0001-test-Include-sys-select.h-for-select.patch \
           "
-
-SRC_URI[md5sum] = "06be230b8b612e64997f8f15af6373aa"
-SRC_URI[sha256sum] = "3175abcb49c0b37fd82c697927783bf7d53de3884a8559bc594e59caa09bdb04"
+SRC_URI[md5sum] = "3cd1ae2a4abecf44b3f24d6639d2cd84"
+SRC_URI[sha256sum] = "dc3f33de54163718f82b3e7c496a7de97f8862578414b8ecaad3cbfe4821864b"

Why has debian mirror different checksums?

Include the diff in commit message and warn premirror maintainers that they need to drop old file (otherwise it will fail) - which is very annoying.

Quoted 11 lines

meta/recipes-support/libproxy/libproxy_0.4.11.bb +0 -0

 inherit cmake pkgconfig

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:29 UTC #7 lore raw

Re: [PATCH 02/26] fotowall: update SRC_URI to use github instead of qt-apps.org

In reply to Alexander Kanavin

Quoted 13 lines · Alexander Kanavin

GitHub is where upstream development is happening, so we are more likely to find out about new releases from there
Signed-off-by Alexander Kanavin <[email protected]>
---
 meta/recipes-qt/qt-apps/fotowall_0.9.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)
meta/recipes-qt/qt-apps/fotowall_0.9.bb +0 -0
index 48dcc02..587b074 100644

meta/recipes-qt/qt-apps/fotowall_0.9.bb SECTION = "x11/apps" · line 16

Quoted 8 lines

meta/recipes-qt/qt-apps/fotowall_0.9.bb +2 -1

@@ -10,7 +10,8 @@ SECTION = "x11/apps"
 
 PR = "r4"
 
-SRC_URI = "http://qt-apps.org/CONTENT/content-files/71316-Fotowall-0.9.tar.bz2 \
+SRCREV = "06d5a4142c599604d9a9fd64727b6945eb8cd3da"
+SRC_URI = "git://github.com/enricoros/fotowall.git \
            file://ExportWizard-depends-on-ui_wizard.patch \
            "

So it doesn't need SRC_URI checksums anymore.. drop:
SRC_URI[md5sum] = "142ef697332e0777c6d22c5bc96cc438"
SRC_URI[sha256sum] = "e4d0c005d2cb1d7c09438bfc3098eadebc08946e4fbc0655b7fc8b046de3810d"

Quoted 9 lines

Signature · 8 lines

-- 
2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:29 UTC #10 lore raw

Re: [PATCH 03/26] distcc: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Quoted 17 lines · Alexander Kanavin

Google Code is shutting down soon and distcc has moved to github.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-devtools/distcc/distcc_3.1.bb | 3 ++-
 1 file changed, 2 insertions(+), 1 deletion(-)

meta/recipes-devtools/distcc/distcc_3.1.bb +1 -1

index e4b9313..b598c77 100644
@@ -17,13 +17,14 @@ PACKAGECONFIG[popt] = "--without-included-popt,--with-included-popt,popt"
 
 RRECOMMENDS_${PN} = "avahi-daemon"
 
-SRC_URI = "http://distcc.googlecode.com/files/${BPN}-${PV}.tar.bz2 \
+SRC_URI = "git://github.com/distcc/distcc.git \

meta/recipes-devtools/distcc/distcc_3.1.bb PACKAGECONFIG[popt] = "--without-included-popt,--with-included-popt,popt" · line 30

Quoted 10 lines

meta/recipes-devtools/distcc/distcc_3.1.bb +1 -0

            file://separatebuilddir.patch \
            file://0001-zeroconf-Include-fcntl.h.patch \
            file://default \
            file://distccmon-gnome.desktop \
            file://distcc \
            file://distcc.service"
+SRCREV = "9a172d3d20042754a30c0cd9a5d63fd1a8120c31"
 
 SRC_URI[md5sum] = "a1a9d3853df7133669fffec2a9aab9f3"
 SRC_URI[sha256sum] = "f55dbafd76bed3ce57e1bbcdab1329227808890d90f4c724fcd2d53f934ddd89"

drop

Quoted 8 lines

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Burton, Ross

2015-07-30 14:30 UTC #3 lore raw

Re: [PATCH 01/26] libproxy: update SRC_URI to not use Google Code

In reply to Martin Jansa

Quoted 2 lines · Martin Jansa

Why has debian mirror different checksums?

Ones a zip, the other is tar.gz.

Ross

Martin Jansa

2015-07-30 14:31 UTC #14 lore raw

Re: [PATCH 06/26] eglinfo: change version from 1.0 to 1.0.0 to match the upstream git tag

In reply to Alexander Kanavin

meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.0.bb

Quoted 16 lines · Alexander Kanavin

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-graphics/eglinfo/{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb}  | 0
 .../recipes-graphics/eglinfo/{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb} | 0
 2 files changed, 0 insertions(+), 0 deletions(-)
 rename meta/recipes-graphics/eglinfo/{eglinfo-fb_1.0.bb => eglinfo-fb_1.0.0.bb} (100%)
 rename meta/recipes-graphics/eglinfo/{eglinfo-x11_1.0.bb => eglinfo-x11_1.0.0.bb} (100%)

meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.0.bb renamed from meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.bb +0 -0

similarity index 100%
rename from meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.bb
rename to meta/recipes-graphics/eglinfo/eglinfo-fb_1.0.0.bb

meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.0.bb renamed from meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.bb +0 -0

similarity index 100%
rename from meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.bb
rename to meta/recipes-graphics/eglinfo/eglinfo-x11_1.0.0.bb

Why don't you set PV in eglinfo.inc and rename them to _git.bb?

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:31 UTC #20 lore raw

Re: [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version

In reply to Alexander Kanavin

Quoted 12 lines · Alexander Kanavin

This allows upstream version check to work
Signed-off-by Alexander Kanavin <[email protected]>
---
 meta/recipes-extended/lsb/lsb_4.1.bb | 1 +
 1 file changed, 1 insertion(+)
meta/recipes-extended/lsb/lsb_4.1.bb +0 -0
index 6215b62..171a6c8 100644

meta/recipes-extended/lsb/lsb_4.1.bb SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar · line 24

Quoted 4 lines

meta/recipes-extended/lsb/lsb_4.1.bb +1 -0

@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
            file://lsb_pidofproc \
            file://lsb_start_daemon \
            "
+PV = "1.4"

And isn't it confusing to say 4.1 in recipe name and 1.4 inside of the recipe?

Quoted 11 lines

meta/recipes-extended/lsb/lsb_4.1.bb +0 -0

 SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
 SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172"

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:33 UTC #27 lore raw

Re: [PATCH 11/26] nss: reorder SRC_URI lines to fix upstream version check

In reply to Alexander Kanavin

Quoted 26 lines · Alexander Kanavin

The source tarball should be listed first

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-support/nss/nss.inc       | 2 +-
 meta/recipes-support/nss/nss_3.17.3.bb | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

meta/recipes-support/nss/nss.inc +1 -1

index 665e4e6..b21dc39 100644
@@ -12,7 +12,7 @@ LICENSE = "MPL-2.0 | (MPL-2.0 & GPL-2.0+) | (MPL-2.0 & LGPL-2.1+)"
 
 LIC_FILES_CHKSUM = "file://nss/lib/freebl/mpi/doc/LICENSE;md5=491f158d09d948466afce85d6f1fe18f \
                     file://nss/lib/freebl/mpi/doc/LICENSE-MPL;md5=6bf96825e3d7ce4de25621ae886cc859"
-SRC_URI = "\
+SRC_URI += "\
     file://nss-fix-support-cross-compiling.patch \
     file://nss-no-rpath-for-cross-compiling.patch \
     file://nss-fix-incorrect-shebang-of-perl.patch \

meta/recipes-support/nss/nss_3.17.3.bb +0 -0

index e7d0780..b22eb95 100644

meta/recipes-support/nss/nss_3.17.3.bb line 3

Quoted 6 lines

meta/recipes-support/nss/nss_3.17.3.bb +1 -3

@@ -1,7 +1,7 @@
-require nss.inc
-
-SRC_URI += "\
+SRC_URI = "\
     http://ftp.mozilla.org/pub/mozilla.org/security/nss/releases/NSS_3_17_3_RTM/src/${BP}.tar.gz \
 "

Why don't you move this to .inc file and replace "3_17_3_RTM" with some variable set in .bb, it would be easier to read than this reorder and require .inc later..

Quoted 12 lines

meta/recipes-support/nss/nss_3.17.3.bb +2 -0

 SRC_URI[md5sum] = "fba7489e1b26f2a0bfe5527430fd61e1"
 SRC_URI[sha256sum] = "f4d5e9035a2f84f25f35c283de3b0ff60d72e918748de25eaf017ed201fa21d5"
+
+require nss.inc

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:34 UTC #32 lore raw

Re: [PATCH 13/26] oprofile: reorder SRC_URI lines to fix upstream version check

In reply to Alexander Kanavin

Quoted 26 lines · Alexander Kanavin

The source tarball should be listed first.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-kernel/oprofile/oprofile.inc      | 2 +-
 meta/recipes-kernel/oprofile/oprofile_1.0.0.bb | 6 +++---
 2 files changed, 4 insertions(+), 4 deletions(-)

meta/recipes-kernel/oprofile/oprofile.inc +1 -1

index 49e6b16..ff360d1 100644
@@ -18,7 +18,7 @@ FILES_${PN} = "${bindir} ${libdir}/${BPN}/lib*${SOLIBS} ${datadir}/${BPN}"
 FILES_${PN}-dev += "${libdir}/${BPN}/lib*${SOLIBSDEV} ${libdir}/${BPN}/lib*.la"
 FILES_${PN}-staticdev += "${libdir}/${BPN}/lib*.a"
 
-SRC_URI = "file://filemode-fix.patch \
+SRC_URI += "file://filemode-fix.patch \
            file://acinclude.m4 \
            file://automake-foreign.patch \
            file://oprofile-cross-compile-tests.patch \

meta/recipes-kernel/oprofile/oprofile_1.0.0.bb +0 -0

index f4a34b3..f75a1dd 100644

meta/recipes-kernel/oprofile/oprofile_1.0.0.bb line 4

Quoted 7 lines

meta/recipes-kernel/oprofile/oprofile_1.0.0.bb +1 -3

@@ -1,13 +1,13 @@
-require oprofile.inc
-
 DEPENDS += "virtual/kernel"
 DEPENDS_append_powerpc64 = " libpfm4"
 
-SRC_URI += "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"
+SRC_URI = "${SOURCEFORGE_MIRROR}/${BPN}/${BPN}-${PV}.tar.gz"

Move this to .inc instead of re-ordering.

Quoted 17 lines

meta/recipes-kernel/oprofile/oprofile_1.0.0.bb +2 -0

 SRC_URI[md5sum] = "ba0b340e5c421a93959776c836ed35b3"
 SRC_URI[sha256sum] = "847110b4ecdcf8c8353cd38f94c1b704aad4bfcd9453e38b88d112cfb7e3c45a"
 
+require oprofile.inc
+
 S = "${WORKDIR}/oprofile-${PV}"
 
 PR = "r1"

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:34 UTC #35 lore raw

Re: [PATCH 14/26] puzzles: switch svn for git, as upstream has done the same

In reply to Alexander Kanavin

Quoted 54 lines · Alexander Kanavin

Also, drop puzzles_x32_abi_time.patch as it has been merged upstream

Signed-off-by Alexander Kanavin <[email protected]>

---
 .../puzzles/files/puzzles_x32_abi_time.patch       | 30 ----------------------
 .../puzzles/{puzzles_r10286.bb => puzzles_git.bb}  |  7 ++---
 2 files changed, 4 insertions(+), 33 deletions(-)
 delete mode 100644 meta/recipes-sato/puzzles/files/puzzles_x32_abi_time.patch
 rename meta/recipes-sato/puzzles/{puzzles_r10286.bb => puzzles_git.bb} (89%)

meta/recipes-sato/puzzles/files/puzzles_x32_abi_time.patch deleted +0 ~~-30~~

deleted file mode 100644
index 63ab0b8..0000000
@@ -1,30 +0,0 @@
-Fix printf's for time_t value add castings because ISOC90
-don't support long long int that is used in x32 ABI for 
-time_t.
-
-Upstream-Status: Backport
-
-Signed-off-by: Aníbal Limón <[email protected]>
-
---- a/magnets.c	2015-03-15 11:57:39.106674811 +0000
-+++ b/magnets.c	2015-03-15 11:57:58.638674795 +0000
-@@ -2562,7 +2562,7 @@
-         goto done;
-     }
-     s = new_game(NULL, p, desc);
--    printf("%s:%s (seed %ld)\n", id, desc, seed);
-+    printf("%s:%s (seed %ld)\n", id, desc, (long) seed);
-     if (aux) {
-         /* We just generated this ourself. */
-         if (verbose || print) {
---- a/signpost.c	2015-03-15 11:58:52.866674751 +0000
-+++ b/signpost.c	2015-03-15 11:59:08.190674738 +0000
-@@ -2393,7 +2393,7 @@
-         }
-     }
- 
--    sprintf(newseed, "%lu", time(NULL));
-+    sprintf(newseed, "%lu", (unsigned long) time(NULL));
-     seedstr = dupstr(newseed);
- 
-     if (id || !stdin_desc) {

meta/recipes-sato/puzzles/puzzles_git.bb renamed from meta/recipes-sato/puzzles/puzzles_r10286.bb +0 -0

similarity index 89%
rename from meta/recipes-sato/puzzles/puzzles_r10286.bb
rename to meta/recipes-sato/puzzles/puzzles_git.bb
index 1ad8d1b..6346c06 100644

meta/recipes-sato/puzzles/puzzles_git.bb REQUIRED_DISTRO_FEATURES = "x11" · line 17

Quoted 10 lines

meta/recipes-sato/puzzles/puzzles_git.bb renamed from meta/recipes-sato/puzzles/puzzles_r10286.bb +3 -2

@@ -10,12 +10,13 @@ REQUIRED_DISTRO_FEATURES = "x11"
 LICENSE = "MIT"
 LIC_FILES_CHKSUM = "file://LICENCE;md5=33bcd4bce8f3c197f2aefbdbd2d299bc"
 
-SRC_URI = "svn://svn.tartarus.org/sgt;module=puzzles;rev=${MOD_PV} \
+SRC_URI = "git://git.tartarus.org/simon/puzzles.git \
            file://fix-compiling-failure-with-option-g-O.patch \
-           file://puzzles_x32_abi_time.patch \
 "
+SRCREV = "aa1a9375be96cd8e0f89a670359c3c80ad7cf937"
+PV = "0.0+git${SRCPV}"

Doesn't PV go backwards with this?

Quoted 14 lines

meta/recipes-sato/puzzles/puzzles_git.bb renamed from meta/recipes-sato/puzzles/puzzles_r10286.bb +1 -1

-S = "${WORKDIR}/${BPN}"
+S = "${WORKDIR}/git"
 
 inherit autotools-brokensep distro_features_check

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:35 UTC #42 lore raw

Re: [PATCH 17/26] tremor: switch SRC_URI from svn to tarballs on debian mirrors

In reply to Alexander Kanavin

meta/recipes-multimedia/tremor/tremor_18153.bb

Quoted 29 lines · Alexander Kanavin

There is no support for checking svn repositories for new commits, so let's use debian tarballs and assume that Debian has a dedicated maintainer who would check the upstream now and then.
Signed-off-by Alexander Kanavin <[email protected]>
---
 .../{tremor-20120314 => tremor}/obsolete_automake_macros.patch      | 0
 .../tremor/{tremor-20120314 => tremor}/tremor-arm-thumb2.patch      | 0
 .../tremor/{tremor_20120314.bb => tremor_18153.bb}                  | 6 ++++--
 3 files changed, 4 insertions(+), 2 deletions(-)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/obsolete_automake_macros.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor-20120314 => tremor}/tremor-arm-thumb2.patch (100%)
 rename meta/recipes-multimedia/tremor/{tremor_20120314.bb => tremor_18153.bb} (66%)
meta/recipes-multimedia/tremor/tremor/obsolete_automake_macros.patch renamed from meta/recipes-multimedia/tremor/tremor-20120314/obsolete_automake_macros.patch +0 -0
similarity index 100%
rename from meta/recipes-multimedia/tremor/tremor-20120314/obsolete_automake_macros.patch
rename to meta/recipes-multimedia/tremor/tremor/obsolete_automake_macros.patch
meta/recipes-multimedia/tremor/tremor/tremor-arm-thumb2.patch renamed from meta/recipes-multimedia/tremor/tremor-20120314/tremor-arm-thumb2.patch +0 -0
similarity index 100%
rename from meta/recipes-multimedia/tremor/tremor-20120314/tremor-arm-thumb2.patch
rename to meta/recipes-multimedia/tremor/tremor/tremor-arm-thumb2.patch
meta/recipes-multimedia/tremor/tremor_18153.bb renamed from meta/recipes-multimedia/tremor/tremor_20120314.bb +0 -0
similarity index 66%
rename from meta/recipes-multimedia/tremor/tremor_20120314.bb
rename to meta/recipes-multimedia/tremor/tremor_18153.bb
index afc78ac..c27fe8c 100644

PV going backwards isn't acceptable for package feeds

Quoted 25 lines

meta/recipes-multimedia/tremor/tremor_18153.bb renamed from meta/recipes-multimedia/tremor/tremor_20120314.bb +4 -2

@@ -8,12 +8,14 @@ DEPENDS = "libogg"
 SRCDATE = "${PV}"
 PR = "r1"
 
-SRC_URI = "svn://svn.xiph.org/trunk;module=Tremor;rev=18221;protocol=http \
+SRC_URI = "${DEBIAN_MIRROR}/main/libv/libvorbisidec/libvorbisidec_1.0.2+svn${PV}.orig.tar.gz \
            file://obsolete_automake_macros.patch;striplevel=0 \
            file://tremor-arm-thumb2.patch \
 "
+SRC_URI[md5sum] = "4190859414c5d6760e316b5cf00fe7c5"
+SRC_URI[sha256sum] = "4dc8c224289da3479fc10ce4e49ffbb85c790eb2fe55ef480934a265ee0a6782"
 
-S = "${WORKDIR}/Tremor"
+S = "${WORKDIR}/libvorbisidec-1.0.2+svn${PV}"
 
 inherit autotools pkgconfig

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Martin Jansa

2015-07-30 14:36 UTC #50 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Alexander Kanavin

Quoted 1 line · Alexander Kanavin

It is unnecessary because the recipe filename already contains it.

No, it contains only 219

Quoted 27 lines

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-core/systemd/systemd_219.bb | 2 --
 1 file changed, 2 deletions(-)

meta/recipes-core/systemd/systemd_219.bb +0 -2

index 83e4bce..148a077 100644
@@ -27,8 +27,6 @@ inherit gtk-doc useradd pkgconfig autotools perlnative update-rc.d update-altern
 
 SRCREV = "85a6fabdd3e43cfab0fc6359e9f2a9e368d4a3ed"
 
-PV = "219-stable+git${SRCPV}"
-
 SRC_URI = "git://anongit.freedesktop.org/systemd/systemd-stable;branch=v219-stable;protocol=git \
            file://0002-shared-missing.h-fall-back-to-insecure-getenv.patch \
            file://0003-binfmt-Don-t-install-dependency-links-at-install-tim.patch \

Signature · 7 lines

2.1.4

-- 
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Signature · 1 line

Martin 'JaMa' Jansa     jabber: [email protected]

Burton, Ross

2015-07-30 15:02 UTC #15 lore raw

Re: [PATCH 06/26] eglinfo: change version from 1.0 to 1.0.0 to match the upstream git tag

In reply to Martin Jansa

Quoted 2 lines · Martin Jansa

Why don't you set PV in eglinfo.inc and rename them to _git.bb?

I think we're going to need the TSC to resolve the opinion in style here. Again, when it's a release that just happens to use git, I do prefer keeping the PV in the filename.

Ross

Burton, Ross

2015-07-30 15:04 UTC #28 lore raw

Re: [PATCH 11/26] nss: reorder SRC_URI lines to fix upstream version check

In reply to Martin Jansa

Quoted 4 lines · Martin Jansa

Why don't you move this to .inc file and replace "3_17_3_RTM" with some variable set in .bb, it would be easier to read than this reorder and require .inc later..

Joe sent a patch earlier to remove the inc file entirely, so please rebase on top of that (it's in ross/mut).

Ross

Alexander Kanavin

author

2015-07-30 15:11 UTC #4 lore raw

Re: [PATCH 01/26] libproxy: update SRC_URI to not use Google Code

In reply to Martin Jansa

Quoted 12 lines · Martin Jansa

meta/recipes-support/libproxy/libproxy_0.4.11.bb LIC_FILES_CHKSUM = "file://COPYING;md5=4fbd65380cdd255951079008b364516c \ · line 15

Quoted 9 lines

meta/recipes-support/libproxy/libproxy_0.4.11.bb +3 -4

-SRC_URI = "http://libproxy.googlecode.com/files/libproxy-${PV}.zip \
+SRC_URI = "${DEBIAN_MIRROR}/main/libp/${BPN}/${BPN}_${PV}.orig.tar.gz \
             file://0001-test-Include-sys-select.h-for-select.patch \
            "
-
-SRC_URI[md5sum] = "06be230b8b612e64997f8f15af6373aa"
-SRC_URI[sha256sum] = "3175abcb49c0b37fd82c697927783bf7d53de3884a8559bc594e59caa09bdb04"
+SRC_URI[md5sum] = "3cd1ae2a4abecf44b3f24d6639d2cd84"
+SRC_URI[sha256sum] = "dc3f33de54163718f82b3e7c496a7de97f8862578414b8ecaad3cbfe4821864b"

Why has debian mirror different checksums?

Quoted 3 lines

Include the diff in commit message and warn premirror maintainers that they need to drop old file (otherwise it will fail) - which is very annoying.

I guess this is not needed because they are in fact two different filenames/formats?

Alex

Alexander Kanavin

author

2015-07-30 15:13 UTC #8 lore raw

Re: [PATCH 02/26] fotowall: update SRC_URI to use github instead of qt-apps.org

In reply to Martin Jansa

On 07/30/2015 05:29 PM, Martin Jansa wrote:

Quoted 4 lines

So it doesn't need SRC_URI checksums anymore.. drop:
SRC_URI[md5sum] = "142ef697332e0777c6d22c5bc96cc438"
SRC_URI[sha256sum] = "e4d0c005d2cb1d7c09438bfc3098eadebc08946e4fbc0655b7fc8b046de3810d"

Done.

Alex

Alexander Kanavin

author

2015-07-30 15:17 UTC #16 lore raw

Re: [PATCH 06/26] eglinfo: change version from 1.0 to 1.0.0 to match the upstream git tag

In reply to Burton, Ross

Quoted 7 lines · Ross

Why don't you set PV in eglinfo.inc and rename them to _git.bb
<http://git.bb>?
I think we're going to need the TSC to resolve the opinion in style here. Again, when it's a release that just happens to use git, I do prefer keeping the PV in the filename.

I agree; it is good to easily tell what is a versioned release, and what is a git snapshot. Also, there's less "PV = ..." clutter inside the recipe that way.

Alex

Burton, Ross

2015-07-30 15:20 UTC #5 lore raw

Re: [PATCH 01/26] libproxy: update SRC_URI to not use Google Code

In reply to Alexander Kanavin

Quoted 3 lines · Alexander Kanavin

I guess this is not needed because they are in fact two different filenames/formats?

Correct.

Ross

Alexander Kanavin

author

2015-07-30 15:21 UTC #21 lore raw

Re: [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version

In reply to Martin Jansa

Quoted 9 lines · Martin Jansa

meta/recipes-extended/lsb/lsb_4.1.bb SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar · line 21
Quoted 6 lines
meta/recipes-extended/lsb/lsb_4.1.bb +1 -0
index 6215b62..171a6c8 100644
@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
+PV = "1.4"
And isn't it confusing to say 4.1 in recipe name and 1.4 inside of the recipe?

It's confusing to begin with to say the recipe is 4.1, but the tarball it fetches is 1.4. I don't see how this makes the confusion much worse, but at least it fixes the upstream version check.

Alex

Alexander Kanavin

author

2015-07-30 15:32 UTC #51 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Martin Jansa

Quoted 5 lines · Martin Jansa

Quoted 1 line · Alexander Kanavin
It is unnecessary because the recipe filename already contains it.
No, it contains only 219
Quoted 1 line
-PV = "219-stable+git${SRCPV}"

The idea here is that we only need the basic version, if the revision being fetched matches that version, and there is no need for all that git revision clutter in the version string.

Alex

Alexander Kanavin

author

2015-07-30 15:40 UTC #33 lore raw

Re: [PATCH 13/26] oprofile: reorder SRC_URI lines to fix upstream version check

In reply to Martin Jansa

Quoted 3 lines · Martin Jansa

Move this to .inc instead of re-ordering.

Done.

Alex

Alexander Kanavin

author

2015-07-30 16:03 UTC #29 lore raw

Re: [PATCH 11/26] nss: reorder SRC_URI lines to fix upstream version check

In reply to Burton, Ross

Quoted 7 lines · Ross

Why don't you move this to .inc file and replace "3_17_3_RTM" with some variable set in .bb, it would be easier to read than this reorder and require .inc later..
Joe sent a patch earlier to remove the inc file entirely, so please rebase on top of that (it's in ross/mut).

Done; after rebasing the patch disappeared altogether :)

Alex

Alexander Kanavin

author

2015-07-30 16:05 UTC #36 lore raw

Re: [PATCH 14/26] puzzles: switch svn for git, as upstream has done the same

In reply to Martin Jansa

Quoted 1 line · Martin Jansa

Doesn't PV go backwards with this?

Yes; Ross should make a suggestion of how to avoid that, and fix the upstream check at the same time - for this recipe and others where I tweak the PV in a similar manner.

Alex

Burton, Ross

2015-07-30 16:25 UTC #52 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Alexander Kanavin

Quoted 9 lines · Alexander Kanavin

No, it contains only 219
Quoted 4 lines
Quoted 3 lines
-PV = "219-stable+git${SRCPV}"
The idea here is that we only need the basic version, if the revision being fetched matches that version, and there is no need for all that git revision clutter in the version string.

For the release tag I'd agree with you, but as 219 < 219-stable we can't just change this now (without adding an epoch, and adding epochs are evil).

Does the RRS have a way of mapping a PV to something that can be used in the upstream comparison?

Ross

Alexander Kanavin

author

2015-07-30 16:34 UTC #53 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

Quoted 5 lines · Ross

For the release tag I'd agree with you, but as 219 < 219-stable we can't just change this now (without adding an epoch, and adding epochs are evil).
Does the RRS have a way of mapping a PV to something that can be used in the upstream comparison?

I just noticed that specifically systemd-stable repository does not have release tags for new versions at all, and just adds branches for newer versions:

http://cgit.freedesktop.org/systemd/systemd-stable/

RRS does not support branch names as indication of "new releases" (and it shouldn't IMO), so maybe this patch can be simply dropped, and systemd will not have automatic upstream-check.

Alex

Max Krummenacher

2015-07-30 17:45 UTC #47 lore raw

Re: [PATCH 20/26] stress: use developer's website for SRC_URI

In reply to Alexander Kanavin

Hi Alexander

Please note that the package has now a changed file COPYING for GPLv3 (but the sources still reference GPLv2)

So unless you still have stress-1.0.4.tar.gz in your download folder the package checksums and LIC_FILES_CHKSUM will throw an error/warning and then fallback successfully to the OE mirror.

Regards
Max
Am Donnerstag, den 30.07.2015, 16:31 +0300 schrieb Alexander Kanavin:

Quoted 24 lines

So that upstream version check happens at the upstream, not at Fedora's build servers.

Signed-off-by Alexander Kanavin <[email protected]>

---
 meta/recipes-extended/stress/stress_1.0.4.bb | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

meta/recipes-extended/stress/stress_1.0.4.bb +1 -1

index 44f2fc6..df96fc0 100644
@@ -4,7 +4,7 @@ HOMEPAGE = "http://people.seas.harvard.edu/~apw/stress/"
 LICENSE = "GPLv2"
 LIC_FILES_CHKSUM = "file://COPYING;md5=d32239bcb673463ab874e80d47fae504"
 
-SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/stress/stress-${PV}.tar.gz/a607afa695a511765b40993a64c6e2f4/stress-${PV}.tar.gz \
+SRC_URI = "http://people.seas.harvard.edu/~apw/stress/stress-${PV}.tar.gz \
            file://texinfo.patch \
            "

Signature · 1 line

2.1.4

Burton, Ross

2015-07-30 21:03 UTC #45 lore raw

Re: [PATCH 19/26] xf86-video-omapfb: switch SRC_URI to debian git repo

In reply to Alexander Kanavin

meta/recipes-graphics/xorg-driver/xf86-video-omapfb_git.bb SRCREV = "28c006c94e57ea71df11ec4fff79d7ffcfc4860f" · line 17

Quoted 7 lines · Alexander Kanavin

meta/recipes-graphics/xorg-driver/xf86-video-omapfb_git.bb +1 -1

@@ -13,7 +13,7 @@ SRCREV = "28c006c94e57ea71df11ec4fff79d7ffcfc4860f"
 PR = "${INC_PR}.7"
 PV = "0.1.1+gitr${SRCPV}"
 
-SRC_URI = "git://git.pingu.fi/xf86-video-omapfb;protocol=http \
+SRC_URI = "git://anonscm.debian.org/collab-maint/xf86-video-omapfb.git \

Produces this on the autobuilder:

ERROR: Fetcher failure: Unable to find revision 28c006c94e57ea71df11ec4fff79d7ffcfc4860f in branch master even from upstream

(attempting to pack right now, so can't check if thats transient or a genuine failure)

Ross

Alexander Kanavin

author

2015-07-31 12:16 UTC #48 lore raw

Re: [PATCH 20/26] stress: use developer's website for SRC_URI

In reply to Max Krummenacher

Quoted 10 lines · Max Krummenacher

Please note that the package has now a changed file COPYING for GPLv3 (but the sources still reference GPLv2)
So unless you still have stress-1.0.4.tar.gz in your download folder the package checksums and LIC_FILES_CHKSUM will throw an error/warning and then fallback successfully to the OE mirror.
meta/recipes-extended/stress/stress_1.0.4.bb HOMEPAGE = "http://people.seas.harvard.edu/~apw/stress/" · line 9
Quoted 4 lines
meta/recipes-extended/stress/stress_1.0.4.bb +1 -1
-SRC_URI = "http://pkgs.fedoraproject.org/repo/pkgs/stress/stress-${PV}.tar.gz/a607afa695a511765b40993a64c6e2f4/stress-${PV}.tar.gz \
+SRC_URI = "http://people.seas.harvard.edu/~apw/stress/stress-${PV}.tar.gz \
             file://texinfo.patch \
             "

That's right. Fedora's tarball has a GPLv3 COPYING in it, while upstream tarball has GPLv2 (the source trees are otherwise identical).

I've updated all the checksums in the recipe to match upstream tarball.

Alex

Alexander Kanavin

author

2015-07-31 12:23 UTC #58 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Burton, Ross

Quoted 5 lines · Ross

For the release tag I'd agree with you, but as 219 < 219-stable we can't just change this now (without adding an epoch, and adding epochs are evil).
Does the RRS have a way of mapping a PV to something that can be used in the upstream comparison?

I don't really know, but I agree that something like RRS_PV would be good to have (and if it's absent then RRS will fall back to PV of course).

Anibal, what's your take?

Alex

Pascal Bach

2015-07-31 12:40 UTC #54 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Alexander Kanavin

Am 30.07.2015 um 18:35 schrieb Alexander Kanavin:

Quoted 5 lines · Ross

Quoted 4 lines
For the release tag I'd agree with you, but as 219 < 219-stable we can't just change this now (without adding an epoch, and adding epochs are evil).

Quoted 7 lines

Quoted 2 lines
Does the RRS have a way of mapping a PV to something that can be used in the upstream comparison?
I just noticed that specifically systemd-stable repository does not have release tags for new versions at all, and just adds branches for newer versions:
http://cgit.freedesktop.org/systemd/systemd-stable/

New tags for systemd are created but not on the repository above but on their git repository:
https://github.com/systemd/systemd

I think this is the now main repository.

Pascal

Quoted 5 lines

RRS does not support branch names as indication of "new releases" (and it shouldn't IMO), so maybe this patch can be simply dropped, and systemd will not have automatic upstream-check.
Alex

Alexander Kanavin

author

2015-07-31 12:52 UTC #55 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

Quoted 8 lines · Pascal Bach

Quoted 4 lines
I just noticed that specifically systemd-stable repository does not have release tags for new versions at all, and just adds branches for newer versions:
http://cgit.freedesktop.org/systemd/systemd-stable/
New tags for systemd are created but not on the repository above but on their git repository:
https://github.com/systemd/systemd
I think this is the now main repository.

The idea behind systemd-stable repository is described here:
http://www.freedesktop.org/wiki/Software/systemd/Backports/

However, it hasn't received any new commits since early May, and still stuck at v219, so should we update to a new version from the main repository? What's oe-core policy here?

Alex

Alexander Kanavin

author

2015-07-31 13:02 UTC #56 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

Quoted 6 lines · Alexander Kanavin

The idea behind systemd-stable repository is described here:
http://www.freedesktop.org/wiki/Software/systemd/Backports/
However, it hasn't received any new commits since early May, and still stuck at v219, so should we update to a new version from the main repository? What's oe-core policy here?

Ah, the systemd-stable repo has also moved to https://github.com/systemd/systemd-stable

It has new commits, but still no tags and no versions newer than v219.

Alex

Pascal Bach

2015-07-31 13:03 UTC #57 lore raw

Re: [PATCH 21/26] systemd: remove PV from the recipe

In reply to Alexander Kanavin

Quoted 4 lines

The idea behind systemd-stable repository is described here:
http://www.freedesktop.org/wiki/Software/systemd/Backports/

Thanks I didn't know that.

Quoted 2 lines

However, it hasn't received any new commits since early May, and still stuck at v219, so should we update to a new version from the main repository? What's oe-core policy here?

Not sure what would be the right appreoach, I would also be interested to hear the policy of oe-core for this.

Pascal

Aníbal Limón

2015-07-31 14:54 UTC #63 lore raw

Re: [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

In reply to Alexander Kanavin

Hi Alexander,

I added get_recipe_pv_without_srcpv for compare git versions because vercmp_string don't take into account git versioning (git, gitAUTOINC) causing wrong comparisons.

We need to review if removal don't affect the current upstream detection.

Regards,
alimon

Quoted 25 lines · Alexander Kanavin

Signed-off-by Alexander Kanavin <[email protected]>

---
  meta/classes/distrodata.bbclass | 5 +----
  1 file changed, 1 insertion(+), 4 deletions(-)

meta/classes/distrodata.bbclass +1 -4

index af3af33..d33187b 100644
@@ -341,10 +341,7 @@ python do_checkpkg() {
          if not pupver:
              pupver = "N/A"
          else:
-            pv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pversion, uri_type)
-            upv, _, _ = oe.recipeutils.get_recipe_pv_without_srcpv(pupver, uri_type)
-
-            cmp = vercmp_string(pv, upv)
+            cmp = vercmp_string(pversion, pupver)
              if cmp == -1:
                  pstatus = "UPDATE"
              elif cmp == 0:

Alexander Kanavin

author

2015-07-31 15:08 UTC #64 lore raw

Re: [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

Quoted 6 lines · Aníbal Limón

I added get_recipe_pv_without_srcpv for compare git versions because vercmp_string
don't take into account git versioning (git, gitAUTOINC) causing wrong comparisons.
We need to review if removal don't affect the current upstream detection.

These are the regex strings that get_recipe_pv_without_srcpv() is using:

for git sources:
git_regex = re.compile("(?P<pfx>(v|))(?P<ver>((\d+[\.\-_]*)+))(?P<sfx>(\+|)(git|)(r|)(AUTOINC|)(\+|))(?P<rev>.*)")

for everything else:
regex = re.compile("(?P<pfx>(v|r|))(?P<ver>((\d+[\.\-_]*)+))")

Both regular expressions are too restrictive in allowing only numeric characters in the <ver> component. If you try to match, say 3.1rc1 and 3.1rc2, they will be both stripped of the '.rcN part, <ver> will be set to 3, and vercmp_string will of course say they match!

Alex

Aníbal Limón

2015-07-31 15:31 UTC #65 lore raw

Re: [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

Quoted 10 lines · Alexander Kanavin

Quoted 6 lines · Aníbal Limón
I added get_recipe_pv_without_srcpv for compare git versions because vercmp_string
don't take into account git versioning (git, gitAUTOINC) causing wrong comparisons.
We need to review if removal don't affect the current upstream detection.
These are the regex strings that get_recipe_pv_without_srcpv() is using:

Quoted 7 lines

for git sources:
git_regex = re.compile("(?P<pfx>(v|))(?P<ver>((\d+[\.\-_]*)+))(?P<sfx>(\+|)(git|)(r|)(AUTOINC|)(\+|))(?P<rev>.*)")
for everything else:
regex = re.compile("(?P<pfx>(v|r|))(?P<ver>((\d+[\.\-_]*)+))")
Both regular expressions are too restrictive in allowing only numeric characters in the <ver> component. If you try to match, say 3.1rc1 and 3.1rc2, they will be both stripped of the '.rcN part, <ver> will be set to 3, and vercmp_string will of course say they match!

In this case we need to add rcN into the regex because if you cmp something like,

Quoted 2 lines

Quoted 2 lines
Quoted 2 lines
import bb
bb.utils.vercmp_string('3.0+gitX', '2.9+gitX')

It returns 1 that means 2.9+gitX is greater than 3.0+gitX.

alimon

Quoted 2 lines

Alex

Aníbal Limón

2015-07-31 15:34 UTC #66 lore raw

Re: [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

Quoted 21 lines · Aníbal Limón

Quoted 10 lines · Alexander Kanavin
Quoted 6 lines · Aníbal Limón
I added get_recipe_pv_without_srcpv for compare git versions because vercmp_string
don't take into account git versioning (git, gitAUTOINC) causing wrong comparisons.
We need to review if removal don't affect the current upstream detection.
These are the regex strings that get_recipe_pv_without_srcpv() is using:
Quoted 7 lines
for git sources:
git_regex = re.compile("(?P<pfx>(v|))(?P<ver>((\d+[\.\-_]*)+))(?P<sfx>(\+|)(git|)(r|)(AUTOINC|)(\+|))(?P<rev>.*)")
for everything else:
regex = re.compile("(?P<pfx>(v|r|))(?P<ver>((\d+[\.\-_]*)+))")
Both regular expressions are too restrictive in allowing only numeric characters in the <ver> component. If you try to match, say 3.1rc1 and 3.1rc2, they will be both stripped of the '.rcN part, <ver> will be set to 3, and vercmp_string will of course say they match!

Quoted 8 lines

In this case we need to add rcN into the regex because if you cmp something like,
Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
import bb
bb.utils.vercmp_string('3.0+gitX', '2.9+gitX')
It returns 1 that means 2.9+gitX is greater than 3.0+gitX.
alimon

I made a mistake here the case that i say is when have a prefix like,

Quoted 2 lines

Quoted 2 lines
Quoted 2 lines
import bb
bb.utils.vercmp_string('r2.7+gitX', '2.9+gitX')

alimon

Quoted 4 lines

Quoted 3 lines
Alex

[email protected]

author

2015-07-31 19:46 UTC #67 lore raw

Re: [PATCH 25/26] distrodata.bbclass: do not use get_recipe_pv_without_srcpv() before comparing versions in checkpkg

Quoted 6 lines

Quoted 2 lines
Quoted 2 lines
re.compile("(?P<pfx>(v|))(?P<ver>((\d+[\.\-_]*)+))(?P<sfx>(\+|)(git|)(r|)(AUTOINC|)(\+|))(?P<rev>.*)") regex = re.compile("(?P<pfx>(v|r|))(?P<ver>((\d+[\.\-_]*)+))")
I made a mistake here the case that i say is when have a prefix like,
Quoted 2 lines
Quoted 2 lines
Quoted 2 lines
import bb
bb.utils.vercmp_string('r2.7+gitX', '2.9+gitX')

But the <pfx> component should take care of any such leading characters, right? So when they are taken out, there is no reason to limit the rest to just numbers. I'll send a patch which changes the regexes to allow "pre", "rc", "final", "stable", or anything else like "1.0.2a". We can't hardcode everything that people can come up with, so it's best to write a generic regex.

Alex

Khem Raj

2015-08-07 07:19 UTC #37 lore raw

Re: [PATCH 14/26] puzzles: switch svn for git, as upstream has done the same

In reply to Alexander Kanavin

Quoted 8 lines · Alexander Kanavin

Quoted 1 line · Martin Jansa
Doesn't PV go backwards with this?
Yes; Ross should make a suggestion of how to avoid that, and fix the upstream check at the same time - for this recipe and others where I tweak the PV in a similar manner.

define PE

Quoted 8 lines

Alex
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Khem Raj

2015-08-07 07:21 UTC #22 lore raw

Re: [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version

In reply to Alexander Kanavin

Quoted 15 lines · Alexander Kanavin

This allows upstream version check to work
Signed-off-by Alexander Kanavin <[email protected]>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 1 +
1 file changed, 1 insertion(+)
meta/recipes-extended/lsb/lsb_4.1.bb +0 -0
index 6215b62..171a6c8 100644

meta/recipes-extended/lsb/lsb_4.1.bb SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar · line 25

Quoted 5 lines

meta/recipes-extended/lsb/lsb_4.1.bb +1 -0

@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
           file://lsb_pidofproc \
           file://lsb_start_daemon \
           "
+PV = “1.4”

it will send version backwards, why is this needed ?

Quoted 10 lines

SRC_URI[md5sum] = "30537ef5a01e0ca94b7b8eb6a36bb1e4"
SRC_URI[sha256sum] = "99321288f8d62e7a1d485b7c6bdccf06766fb8ca603c6195806e4457fdf17172" --
2.1.4
--
_______________________________________________
Openembedded-core mailing list
[email protected]
http://lists.openembedded.org/mailman/listinfo/openembedded-core

Alexander Kanavin

author

2015-08-07 12:07 UTC #38 lore raw

Re: [PATCH 14/26] puzzles: switch svn for git, as upstream has done the same

In reply to Khem Raj

Quoted 6 lines · Khem Raj

Quoted 3 lines
Quoted 1 line
Doesn't PV go backwards with this?
Yes; Ross should make a suggestion of how to avoid that, and fix the upstream check at the same time - for this recipe and others where I tweak the PV in a similar manner.
define PE

Ross said he would like to avoid that; I hope he'll make a comment after coming back from holiday.

Alex

Alexander Kanavin

author

2015-08-07 12:08 UTC #23 lore raw

Re: [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version

In reply to Khem Raj

Quoted 7 lines · Khem Raj

Quoted 7 lines
This allows upstream version check to work
Signed-off-by Alexander Kanavin <[email protected]>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 1 +
1 file changed, 1 insertion(+)

Quoted 12 lines

meta/recipes-extended/lsb/lsb_4.1.bb SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar · line 25
Quoted 10 lines
meta/recipes-extended/lsb/lsb_4.1.bb +1 -0
index 6215b62..171a6c8 100644
@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
            file://lsb_pidofproc \
            file://lsb_start_daemon \
            "
+PV = “1.4”
 
it will send version backwards, why is this needed ?

See above - "This allows upstream version check to work".

Alex

Khem Raj

2015-08-07 19:22 UTC #24 lore raw

Re: [PATCH 09/26] lsb: set PV to 1.4 to match upstream tarball version

Quoted 25 lines · Alexander Kanavin

Quoted 7 lines · Khem Raj
Quoted 7 lines
This allows upstream version check to work
Signed-off-by Alexander Kanavin <[email protected]>
---
meta/recipes-extended/lsb/lsb_4.1.bb | 1 +
1 file changed, 1 insertion(+)
Quoted 12 lines
meta/recipes-extended/lsb/lsb_4.1.bb SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar · line 25
Quoted 10 lines
meta/recipes-extended/lsb/lsb_4.1.bb +1 -0
index 6215b62..171a6c8 100644
@@ -21,6 +21,7 @@ SRC_URI = "${SOURCEFORGE_MIRROR}/project/lsb/lsb_release/1.4/lsb-release-1.4.tar
           file://lsb_pidofproc \
           file://lsb_start_daemon \
           "
+PV = “1.4”
 
it will send version backwards, why is this needed ?

Quoted 1 line

See above - "This allows upstream version check to work”.

it seems a futile cause in front of regressing on upgrade policy

Quoted 4 lines

Alex