Skip to contentSkip to conversation

bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

PATCH In review Adrian Bunk 4 participants 5 messages Jul 29, 2019 – Jul 30, 2019

lore
Share & tools
lore

Adrian Bunk

author
#0 lore raw

PATCH 1/1 bind: Remove RECIPE_NO_UPDATE_REASON and follow the ESV releases

Signed-off-by Adrian Bunk <[email protected]>
---
 meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +2 -2
index 1355841e6b..93c406f778 100644
@@ -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
Signature · 1 line
2.17.1
#1 lore raw
Quoted 10 lines · Adrian Bunk
Signed-off-by Adrian Bunk <[email protected]>
---
 meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +0 -0
index 1355841e6b..93c406f778 100644

meta/recipes-connectivity/bind/bind_9.11.5-P4.bb SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960" · line 30

Quoted 7 lines
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +2 -2
@@ -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

Quoted 3 lines
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +0 -0
 inherit autotools update-rc.d systemd useradd pkgconfig multilib_script
author
#2 lore raw
Quoted 15 lines · On Tue, Jul 30, 2019 at 02:20:40AM -0700, akuster808 wrote:
Quoted 12 lines · Adrian Bunk
Signed-off-by Adrian Bunk <[email protected]>
---
 meta/recipes-connectivity/bind/bind_9.11.5-P4.bb | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +0 -0
index 1355841e6b..93c406f778 100644
@@ -26,8 +26,8 @@ SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960"
 SRC_URI[sha256sum] = "7e8c08192bcbaeb6e9f2391a70e67583b027b90e8c4bc1605da6eb126edde434"
Quoted 7 lines

meta/recipes-connectivity/bind/bind_9.11.5-P4.bb SRC_URI[md5sum] = "8ddab4b61fa4516fe404679c74e37960" · line 30

Quoted 5 lines
meta/recipes-connectivity/bind/bind_9.11.5-P4.bb +2 -2
 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.

Quoted 1 line

Its a function over maintenance question.

Yes.

Quoted 2 lines

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.

Quoted 1 line

- Armin

cu
Adrian

Signature · 4 lines
       "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
#3 lore raw
In reply to akuster808
Quoted 6 lines · On Tue, 30 Jul 2019 at 11:21, akuster808 <[email protected]> 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

#4 lore raw
Quoted 5 lines · Alexander Kanavin

On Tue, 30 Jul 2019 at 11:21, akuster808 <[email protected] <mailto:[email protected]>> wrote:

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

Quoted 7 lines

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

Quoted 3 lines

Alex