* [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead
@ 2018-02-09 4:25 Chang Rebecca Swee Fun
2018-02-09 4:25 ` [PATCH 2/2] maintainers.inc: change ownership for bmap-tools and wic-tools Chang Rebecca Swee Fun
` (2 more replies)
0 siblings, 3 replies; 7+ messages in thread
From: Chang Rebecca Swee Fun @ 2018-02-09 4:25 UTC (permalink / raw)
To: openembedded-core
Tarball sources can be checksummed and are faster to download.
Signed-off-by: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
---
meta/recipes-support/bmap-tools/bmap-tools_3.4.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
index 70b6e91..6975acb 100644
--- a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
+++ b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
@@ -9,10 +9,10 @@ SECTION = "console/utils"
LICENSE = "GPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-SRC_URI = "git://github.com/intel/bmap-tools.git"
-SRCREV = "9dad724104df265442226972a1e310813f9ffcba"
+SRC_URI = "https://github.com/intel/${BPN}/releases/download/v${PV}/${BPN}-${PV}.tgz"
-S = "${WORKDIR}/git"
+SRC_URI[md5sum] = "883420183611aa137cbaf68a91ef0d7e"
+SRC_URI[sha256sum] = "0064891270c180c190f41925b0f6fb9cecac3056f1168d2592dbe90e5f226c0a"
RDEPENDS_${PN} = "python3-core python3-compression python3-mmap python3-setuptools"
--
2.7.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCH 2/2] maintainers.inc: change ownership for bmap-tools and wic-tools
2018-02-09 4:25 [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead Chang Rebecca Swee Fun
@ 2018-02-09 4:25 ` Chang Rebecca Swee Fun
2018-02-09 6:08 ` [PATCHv2] bmap-tools: change SRC_URI to use tarball instead rebecca.swee.fun.chang
2018-02-09 6:52 ` [PATCH 1/2] " Alexander Kanavin
2 siblings, 0 replies; 7+ messages in thread
From: Chang Rebecca Swee Fun @ 2018-02-09 4:25 UTC (permalink / raw)
To: openembedded-core
Signed-off-by: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
---
meta/conf/distro/include/maintainers.inc | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/meta/conf/distro/include/maintainers.inc b/meta/conf/distro/include/maintainers.inc
index 34f3d81..dc7d688 100644
--- a/meta/conf/distro/include/maintainers.inc
+++ b/meta/conf/distro/include/maintainers.inc
@@ -77,7 +77,7 @@ RECIPE_MAINTAINER_pn-bjam-native = "Alexander Kanavin <alexander.kanavin@intel.c
RECIPE_MAINTAINER_pn-blktool = "Yi Zhao <yi.zhao@windriver.com>"
RECIPE_MAINTAINER_pn-blktrace = "Alexander Kanavin <alexander.kanavin@intel.com>"
RECIPE_MAINTAINER_pn-bluez5 = "Maxin B. John <maxin.john@intel.com>"
-RECIPE_MAINTAINER_pn-bmap-tools = "Ed Bartosh <ed.bartosh@linux.intel.com>"
+RECIPE_MAINTAINER_pn-bmap-tools = "Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>"
RECIPE_MAINTAINER_pn-boost = "Alexander Kanavin <alexander.kanavin@intel.com>"
RECIPE_MAINTAINER_pn-bootchart2 = "Alexander Kanavin <alexander.kanavin@intel.com>"
RECIPE_MAINTAINER_pn-bsd-headers = "Khem Raj <raj.khem@gmail.com>"
@@ -740,7 +740,7 @@ RECIPE_MAINTAINER_pn-weston = "Denys Dmytriyenko <denys@ti.com>"
RECIPE_MAINTAINER_pn-weston-init = "Denys Dmytriyenko <denys@ti.com>"
RECIPE_MAINTAINER_pn-wget = "Yi Zhao <yi.zhao@windriver.com>"
RECIPE_MAINTAINER_pn-which = "Maxin B. John <maxin.john@intel.com>"
-RECIPE_MAINTAINER_pn-wic-tools = "Ed Bartosh <ed.bartosh@linux.intel.com>"
+RECIPE_MAINTAINER_pn-wic-tools = "Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>"
RECIPE_MAINTAINER_pn-wireless-tools = "Changhyeok Bae <changhyeok.bae@lge.com>"
RECIPE_MAINTAINER_pn-wpa-supplicant = "Changhyeok Bae <changhyeok.bae@lge.com>"
RECIPE_MAINTAINER_pn-x11-common = "Armin Kuster <akuster@mvista.com>"
--
2.7.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* [PATCHv2] bmap-tools: change SRC_URI to use tarball instead
2018-02-09 4:25 [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead Chang Rebecca Swee Fun
2018-02-09 4:25 ` [PATCH 2/2] maintainers.inc: change ownership for bmap-tools and wic-tools Chang Rebecca Swee Fun
@ 2018-02-09 6:08 ` rebecca.swee.fun.chang
2018-02-09 7:56 ` [PATCHv3] " rebecca.swee.fun.chang
2018-02-09 6:52 ` [PATCH 1/2] " Alexander Kanavin
2 siblings, 1 reply; 7+ messages in thread
From: rebecca.swee.fun.chang @ 2018-02-09 6:08 UTC (permalink / raw)
To: openembedded-core
From: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
Tarball sources can be checksummed and are faster to download.
Signed-off-by: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
---
meta/recipes-support/bmap-tools/bmap-tools_3.4.bb | 6 +++---
1 file changed, 3 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
index 70b6e91..3f15e12 100644
--- a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
+++ b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
@@ -9,10 +9,10 @@ SECTION = "console/utils"
LICENSE = "GPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-SRC_URI = "git://github.com/intel/bmap-tools.git"
-SRCREV = "9dad724104df265442226972a1e310813f9ffcba"
+SRC_URI = "https://github.com/intel/${BPN}/releases/download/v${PV}/${BP}.tgz"
-S = "${WORKDIR}/git"
+SRC_URI[md5sum] = "883420183611aa137cbaf68a91ef0d7e"
+SRC_URI[sha256sum] = "0064891270c180c190f41925b0f6fb9cecac3056f1168d2592dbe90e5f226c0a"
RDEPENDS_${PN} = "python3-core python3-compression python3-mmap python3-setuptools"
--
2.7.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead
2018-02-09 4:25 [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead Chang Rebecca Swee Fun
2018-02-09 4:25 ` [PATCH 2/2] maintainers.inc: change ownership for bmap-tools and wic-tools Chang Rebecca Swee Fun
2018-02-09 6:08 ` [PATCHv2] bmap-tools: change SRC_URI to use tarball instead rebecca.swee.fun.chang
@ 2018-02-09 6:52 ` Alexander Kanavin
2018-02-09 7:14 ` Chang, Rebecca Swee Fun
2 siblings, 1 reply; 7+ messages in thread
From: Alexander Kanavin @ 2018-02-09 6:52 UTC (permalink / raw)
To: Chang Rebecca Swee Fun, openembedded-core
On 02/09/2018 06:25 AM, Chang Rebecca Swee Fun wrote:
> -SRC_URI = "git://github.com/intel/bmap-tools.git"
> -SRCREV = "9dad724104df265442226972a1e310813f9ffcba"
> +SRC_URI = "https://github.com/intel/${BPN}/releases/download/v${PV}/${BPN}-${PV}.tgz"
When you switch the upstream source location like this, you need to
verify that upstream version check still works:
bitbake -c checkpkg bmap-tools
and inspect tmp/log/checkpkg.csv to see the outcome. If bitbake can no
longer determine the latest version, set UPSTREAM_CHECK_URI to the
webpage where all the tarballs are linked from (I think it would be
https://github.com/intel/${BPN}/releases/). There's plenty of recipes in
oe-core with similar adjustments, if you need further examples.
Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead
2018-02-09 6:52 ` [PATCH 1/2] " Alexander Kanavin
@ 2018-02-09 7:14 ` Chang, Rebecca Swee Fun
2018-02-09 7:44 ` Alexander Kanavin
0 siblings, 1 reply; 7+ messages in thread
From: Chang, Rebecca Swee Fun @ 2018-02-09 7:14 UTC (permalink / raw)
To: Alexander Kanavin, openembedded-core
Hi Alex,
> -----Original Message-----
> From: Alexander Kanavin [mailto:alexander.kanavin@linux.intel.com]
> Sent: Friday, February 9, 2018 2:52 PM
> To: Chang, Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>;
> openembedded-core@lists.openembedded.org
> Subject: Re: [OE-core] [PATCH 1/2] bmap-tools: change SRC_URI to use tarball
> instead
>
> On 02/09/2018 06:25 AM, Chang Rebecca Swee Fun wrote:
> > -SRC_URI = "git://github.com/intel/bmap-tools.git"
> > -SRCREV = "9dad724104df265442226972a1e310813f9ffcba"
> > +SRC_URI =
> "https://github.com/intel/${BPN}/releases/download/v${PV}/${BPN}-${PV}.tgz"
>
> When you switch the upstream source location like this, you need to verify that
> upstream version check still works:
>
> bitbake -c checkpkg bmap-tools
>
> and inspect tmp/log/checkpkg.csv to see the outcome. If bitbake can no longer
> determine the latest version, set UPSTREAM_CHECK_URI to the webpage where
> all the tarballs are linked from (I think it would be
> https://github.com/intel/${BPN}/releases/). There's plenty of recipes in oe-core
> with similar adjustments, if you need further examples.
>
Thanks for your input! I will double check and send a new revision.
Rebecca
^ permalink raw reply [flat|nested] 7+ messages in thread
* Re: [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead
2018-02-09 7:14 ` Chang, Rebecca Swee Fun
@ 2018-02-09 7:44 ` Alexander Kanavin
0 siblings, 0 replies; 7+ messages in thread
From: Alexander Kanavin @ 2018-02-09 7:44 UTC (permalink / raw)
To: Chang, Rebecca Swee Fun, openembedded-core
On 02/09/2018 09:14 AM, Chang, Rebecca Swee Fun wrote:
>> When you switch the upstream source location like this, you need to verify that
>> upstream version check still works:
>>
>> bitbake -c checkpkg bmap-tools
>>
>> and inspect tmp/log/checkpkg.csv to see the outcome. If bitbake can no longer
>> determine the latest version, set UPSTREAM_CHECK_URI to the webpage where
>> all the tarballs are linked from (I think it would be
>> https://github.com/intel/${BPN}/releases/). There's plenty of recipes in oe-core
>> with similar adjustments, if you need further examples.
>>
> Thanks for your input! I will double check and send a new revision.
Ah, how could I forget. Devtool makes this much more convenient:
devtool latest-version bmap-tools
Alex
^ permalink raw reply [flat|nested] 7+ messages in thread
* [PATCHv3] bmap-tools: change SRC_URI to use tarball instead
2018-02-09 6:08 ` [PATCHv2] bmap-tools: change SRC_URI to use tarball instead rebecca.swee.fun.chang
@ 2018-02-09 7:56 ` rebecca.swee.fun.chang
0 siblings, 0 replies; 7+ messages in thread
From: rebecca.swee.fun.chang @ 2018-02-09 7:56 UTC (permalink / raw)
To: openembedded-core
From: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
Tarball sources can be checksummed and are faster to download.
Added UPSTREAM_CHECK_URI to improve recipe maintenance and to
verify upstream version check is still work.
Signed-off-by: Chang Rebecca Swee Fun <rebecca.swee.fun.chang@intel.com>
---
meta/recipes-support/bmap-tools/bmap-tools_3.4.bb | 8 +++++---
1 file changed, 5 insertions(+), 3 deletions(-)
diff --git a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
index 70b6e91..9d8191f 100644
--- a/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
+++ b/meta/recipes-support/bmap-tools/bmap-tools_3.4.bb
@@ -9,10 +9,12 @@ SECTION = "console/utils"
LICENSE = "GPLv2"
LIC_FILES_CHKSUM = "file://COPYING;md5=b234ee4d69f5fce4486a80fdaf4a4263"
-SRC_URI = "git://github.com/intel/bmap-tools.git"
-SRCREV = "9dad724104df265442226972a1e310813f9ffcba"
+SRC_URI = "https://github.com/intel/${BPN}/releases/download/v${PV}/${BP}.tgz"
-S = "${WORKDIR}/git"
+SRC_URI[md5sum] = "883420183611aa137cbaf68a91ef0d7e"
+SRC_URI[sha256sum] = "0064891270c180c190f41925b0f6fb9cecac3056f1168d2592dbe90e5f226c0a"
+
+UPSTREAM_CHECK_URI = "https://github.com/intel/${BPN}/releases"
RDEPENDS_${PN} = "python3-core python3-compression python3-mmap python3-setuptools"
--
2.7.4
^ permalink raw reply related [flat|nested] 7+ messages in thread
end of thread, other threads:[~2018-02-09 8:01 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2018-02-09 4:25 [PATCH 1/2] bmap-tools: change SRC_URI to use tarball instead Chang Rebecca Swee Fun
2018-02-09 4:25 ` [PATCH 2/2] maintainers.inc: change ownership for bmap-tools and wic-tools Chang Rebecca Swee Fun
2018-02-09 6:08 ` [PATCHv2] bmap-tools: change SRC_URI to use tarball instead rebecca.swee.fun.chang
2018-02-09 7:56 ` [PATCHv3] " rebecca.swee.fun.chang
2018-02-09 6:52 ` [PATCH 1/2] " Alexander Kanavin
2018-02-09 7:14 ` Chang, Rebecca Swee Fun
2018-02-09 7:44 ` Alexander Kanavin
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.