[OE-core] [PATCH v3 4/8] logrotate: replace fedorahosted.org SRC_URI with github.com source

[Choong, Yin Thong]

Is wired. I bitbake successful on my machine. bitbake logrotate -ccleanall and bitbake logrotate.
In this case, I should submit my entire patch or only resubmit this patch [PATCH v3 4/8] with cover letter?

Thanks and Regards
Choong YinThong

-----Original Message-----
From: Richard Purdie [mailto:richard.purdie at linuxfoundation.org] 
Sent: Friday, April 7, 2017 3:37 PM
To: Choong, Yin Thong <yin.thong.choong at intel.com>; openembedded-core at lists.openembedded.org
Subject: Re: [OE-core] [PATCH v3 4/8] logrotate: replace fedorahosted.org SRC_URI with github.com source

On Thu, 2017-04-06 at 01:35 -0700, yin.thong.choong at intel.com wrote:
> From: Choong YinThong <yin.thong.choong at intel.com>
> 
> fedorahosted.org was retired on March 1st, 2017. This is to update the 
> SRC_URI to point to github.com.
> 
> [YOCTO #11226]
> 
> Signed-off-by: Choong YinThong <yin.thong.choong at intel.com>

This failed autobuilder testing in multilib builds:

https://autobuilder.yocto.io/builders/nightly-world-lsb/builds/214/steps/BuildImages/logs/stdio
https://autobuilder.yocto.io/builders/nightly-world/builds/214/steps/BuildImages/logs/stdio

Probably you need ${BPN}, not ${PN}.

>  meta/recipes-extended/logrotate/logrotate_3.9.1.bb | 19
> +++++++++++++++----
>  1 file changed, 15 insertions(+), 4 deletions(-)
> 
> diff --git a/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
> b/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
> index 9c2dfe0..5cbd9ff 100644
> --- a/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
> +++ b/meta/recipes-extended/logrotate/logrotate_3.9.1.bb
> @@ -1,6 +1,6 @@
>  SUMMARY = "Rotates, compresses, removes and mails system log files"
>  SECTION = "console/utils"
> -HOMEPAGE = "https://fedorahosted.org/logrotate/"
> +HOMEPAGE = "https://github.com/logrotate/logrotate/issues"
>  LICENSE = "GPLv2"
>  
>  # TODO: logrotate 3.8.8 adds autotools/automake support, update 
> recipe to use it.
> @@ -10,14 +10,25 @@ DEPENDS="coreutils popt"
>  
>  LIC_FILES_CHKSUM =
> "file://COPYING;md5=18810669f13b87348459e611d31ab760"
>  
> -SRC_URI = "https://fedorahosted.org/releases/l/o/logrotate/logrotate
> -${PV}.tar.gz \
> +# When updating logrotate to latest upstream, SRC_URI should point
> to
> +# a proper release tarball from https://github.com/logrotate/logrota
> te/releases
> +# and we have to take the snapshot for now because there is no such # 
> +tarball available for 3.9.1.
> +
> +S = "${WORKDIR}/${PN}-r3-9-1"
> +
> +UPSTREAM_CHECK_URI = "https://github.com/${PN}/${PN}/releases"
> +
> +SRC_URI = "https://github.com/${PN}/${PN}/archive/r3-9-1.tar.gz \
>             file://act-as-mv-when-rotate.patch \
>             file://update-the-manual.patch \
>             file://disable-check-different-filesystems.patch \
>              "
>  
> -SRC_URI[md5sum] = "4492b145b6d542e4a2f41e77fa199ab0"
> -SRC_URI[sha256sum] =
> "022769e3288c80981559a8421703c88e8438b447235e36dd3c8e97cd94c52545"
> +# Checksum changed due to tarball source folder changes in upstream
> +			
> +SRC_URI[md5sum] = "8572b7c2cf9ade09a8a8e10098500fb3"
> +SRC_URI[sha256sum] =
> "5bf8e478c428e7744fefa465118f8296e7e771c981fb6dffb7527856a0ea3617"
>  
>  PACKAGECONFIG ?= "${@bb.utils.filter('DISTRO_FEATURES', 'acl 
> selinux', d)}"
>  
> --
> 2.7.4
>