[PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[Adrian Bunk]

Signed-off-by: Adrian Bunk <bunk@stusta.de>
---
 meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
index 1355841e6b..93c406f778 100644
--- a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
+++ b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
@@ -26,8 +26,8 @@ SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960"
 SRC_URI[sha256sum] = "7e8c08192bcbaeb6e9f2391a70e67583b027b90e8c4bc1605da6eb126edde434"
 
 UPSTREAM_CHECK_URI = "https://ftp.isc.org/isc/bind9/"
-UPSTREAM_CHECK_REGEX = "(?P<pver>9(\.\d+)+(-P\d+)*)/"
-RECIPE_NO_UPDATE_REASON = "9.11 is LTS 2021"
+# stay at 9.11 until 9.16, from 9.16 follow the ESV versions divisible by 4
+UPSTREAM_CHECK_REGEX = "(?P<pver>9.(11|16|20|24|28)(\.\d+)+(-P\d+)*)/"
 
 inherit autotools update-rc.d systemd useradd pkgconfig multilib_script
 
-- 
2.17.1

Re: [PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[akuster808]

On 7/29/19 3:00 AM, Adrian Bunk wrote:
> Signed-off-by: Adrian Bunk <bunk@stusta.de>
> ---
>  meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
>  1 file changed, 2 insertions(+), 2 deletions(-)
>
> diff --git a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> index 1355841e6b..93c406f778 100644
> --- a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> +++ b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> @@ -26,8 +26,8 @@ SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960"
>  SRC_URI[sha256sum] = "7e8c08192bcbaeb6e9f2391a70e67583b027b90e8c4bc1605da6eb126edde434"
>  
>  UPSTREAM_CHECK_URI = "https://ftp.isc.org/isc/bind9/"
> -UPSTREAM_CHECK_REGEX = "(?P<pver>9(\.\d+)+(-P\d+)*)/"
> -RECIPE_NO_UPDATE_REASON = "9.11 is LTS 2021"
> +# stay at 9.11 until 9.16, from 9.16 follow the ESV versions divisible by 4
> +UPSTREAM_CHECK_REGEX = "(?P<pver>9.(11|16|20|24|28)(\.\d+)+(-P\d+)*)/"

So are you setting the package update policy to LTS only ?  Its a
function over maintenance question.

I don't understand why this package is being singled out. I think it
should be an OE  architecture discussion

- Armin
>  
>  inherit autotools update-rc.d systemd useradd pkgconfig multilib_script
>  

Re: [PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[Adrian Bunk]

On Tue, Jul 30, 2019 at 02:20:40AM -0700, akuster808 wrote:
> 
> 
> On 7/29/19 3:00 AM, Adrian Bunk wrote:
> > Signed-off-by: Adrian Bunk <bunk@stusta.de>
> > ---
> >  meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
> >  1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> > index 1355841e6b..93c406f778 100644
> > --- a/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> > +++ b/meta/recipes-connectivity/bind/bind_9.11.5-P4.bb
> > @@ -26,8 +26,8 @@ SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960"
> >  SRC_URI[sha256sum] = "7e8c08192bcbaeb6e9f2391a70e67583b027b90e8c4bc1605da6eb126edde434"
> >  
> >  UPSTREAM_CHECK_URI = "https://ftp.isc.org/isc/bind9/"
> > -UPSTREAM_CHECK_REGEX = "(?P<pver>9(\.\d+)+(-P\d+)*)/"
> > -RECIPE_NO_UPDATE_REASON = "9.11 is LTS 2021"
> > +# stay at 9.11 until 9.16, from 9.16 follow the ESV versions divisible by 4
> > +UPSTREAM_CHECK_REGEX = "(?P<pver>9.(11|16|20|24|28)(\.\d+)+(-P\d+)*)/"
> 
> So are you setting the package update policy to LTS only ?

RECIPE_NO_UPDATE_REASON was a bug in any case since it prevented 
upgrades on the 9.11 branch.

> Its a function over maintenance question.

Yes.

> I don't understand why this package is being singled out. I think it
> should be an OE  architecture discussion

My impression is that the default in OE is to follow upstream LTS,
and not doing so would be the exception.

> - Armin

cu
Adrian

-- 

       "Is there not promise of rain?" Ling Tan asked suddenly out
        of the darkness. There had been need of rain for many days.
       "Only a promise," Lao Er said.
                                       Pearl S. Buck - Dragon Seed

Re: [PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[Alexander Kanavin]

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

On Tue, 30 Jul 2019 at 11:21, akuster808 <akuster808@gmail.com> wrote:

> So are you setting the package update policy to LTS only ?  Its a
> function over maintenance question.
>
> I don't understand why this package is being singled out. I think it
> should be an OE  architecture discussion
>

I am not sure why are you objecting to this. This line:
RECIPE_NO_UPDATE_REASON = "9.11 is LTS 2021"
was added by you, and more or less states that update policy for bind is
LTS-only.

Alex

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

Re: [PATCH] bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

[Khem Raj]

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


On 7/30/19 6:01 AM, Alexander Kanavin wrote:
> On Tue, 30 Jul 2019 at 11:21, akuster808 <akuster808@gmail.com 
> <mailto:akuster808@gmail.com>> wrote:
>
>     So are you setting the package update policy to LTS only ?  Its a
>     function over maintenance question.
>
>     I don't understand why this package is being singled out. I think it
>     should be an OE  architecture discussion
>
>
> I am not sure why are you objecting to this. This line:
> RECIPE_NO_UPDATE_REASON = "9.11 is LTS 2021"
> was added by you, and more or less states that update policy for bind 
> is LTS-only.

I see a technical reason where dependent packages like dhcp expects 9.11 
if used from system, there is an option to built the bundled version of 
bind9 but that results in duplication. Unless there are other compelling 
reason for upgrading, I would like us to stick with 9.11

>
> Alex
>

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