Re: [ptxdist] [PATCH] networkmanager: version bump 1.20.8-> 1.22.6
On Fri, Feb 14 2020 at 09:36 +0100, Michael Olbrich wrote: > On Wed, Feb 12, 2020 at 09:30:29PM +0100, Uwe Kleine-König wrote: >> On Wed, Feb 12, 2020 at 11:11:12AM +0100, Ulrich Ölmann wrote: >> > Two new config options have been added in the meantime (see NetworkManager >> > commits [1] & [2]) which are both set to their reasonable default values. >> > >> > The license did not change, but its text was adjusted - citing from >> > NetworkManagers's commit [3]: >> > >> > The update to the GPL text is purely cosmetic. However, shipping the >> > exact >> > same file as GNU publishes may help distros that deduplicate the license >> > texts >> > or hardlink duplicates. >> > >> > [1] >> > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/69f048bf0ca387d2dc4683cfdfe9d170bfceb52b >> > [2] >> > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/d27fcd07541ae6f524115d5b0f36e14673135ca3 >> > [3] >> > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/e9f2ea6c22f36cb7986d2228763629ed44b9e76b >> > >> > Signed-off-by: Ulrich Ölmann >> > --- >> > rules/networkmanager.make | 10 ++ >> > 1 file changed, 6 insertions(+), 4 deletions(-) >> > >> > diff --git a/rules/networkmanager.make b/rules/networkmanager.make >> > index de0e755b3a5d..86542b6f74e1 100644 >> > --- a/rules/networkmanager.make >> > +++ b/rules/networkmanager.make >> > @@ -15,15 +15,15 @@ PACKAGES-$(PTXCONF_NETWORKMANAGER) += networkmanager >> > # >> > # Paths and names >> > # >> > -NETWORKMANAGER_VERSION:= 1.20.8 >> > -NETWORKMANAGER_MD5:= c0ceb5ab14bfdfeee07536d94cc5c548 >> > +NETWORKMANAGER_VERSION:= 1.22.6 >> > +NETWORKMANAGER_MD5:= 0f4b493cc0c67f94a2ad3573363eb3b2 >> > NETWORKMANAGER:= NetworkManager-$(NETWORKMANAGER_VERSION) >> > NETWORKMANAGER_SUFFIX := tar.xz >> > -NETWORKMANAGER_URL:= >> > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.20/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) >> > +NETWORKMANAGER_URL:= >> > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.22/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) >> >> I wonder if it would make sense to use some make magic to derive the >> directory name (1.22) from NETWORKMANAGER_VERSION. >> >> Hmm, this is more complicated than I thought, the following works: >> >> EMPTY = >> SPACE = $(EMPTY) $(EMPTY) >> NETWORKMANAGER_MAJOR := $(subst $(SPACE),.,$(wordlist 1, 2, $(subst ., >> ,$(NETWORKMANAGER_VERSION >> >> or >> >> NETWORKMANAGER_V1 = $(word 1, $(subst ., ,$(NETWORKMANAGER_VERSION))) >> NETWORKMANAGER_V2 = $(word 2, $(subst ., ,$(NETWORKMANAGER_VERSION))) >> NETWORKMANAGER_MAJOR = $(NETWORKMANAGER_V1).$(NETWORKMANAGER_V2) >> >> Unless someone can come up with something easier, probably not worth the >> effort? > > There is a much simpler way to do this: > > NETWORKMANAGER_URL := > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/$(basename > $(NETWORKMANAGER_VERSION))/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) Just sent an implicit v2 bumping NetworkManager's version up to 1.22.8 which has been released yesterday. Ulrich > We already use this in several other packages. This does not work when a > '0' for the patchlevel version is omitted but that's not the case here. > > Michael -- Pengutronix e.K. | Ulrich Ölmann | Industrial Linux Solutions | http://www.pengutronix.de/ | Peiner Str. 6-8, 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | ___ ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH] networkmanager: version bump 1.20.8-> 1.22.6
On Wed, Feb 12, 2020 at 09:30:29PM +0100, Uwe Kleine-König wrote: > On Wed, Feb 12, 2020 at 11:11:12AM +0100, Ulrich Ölmann wrote: > > Two new config options have been added in the meantime (see NetworkManager > > commits [1] & [2]) which are both set to their reasonable default values. > > > > The license did not change, but its text was adjusted - citing from > > NetworkManagers's commit [3]: > > > > The update to the GPL text is purely cosmetic. However, shipping the exact > > same file as GNU publishes may help distros that deduplicate the license > > texts > > or hardlink duplicates. > > > > [1] > > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/69f048bf0ca387d2dc4683cfdfe9d170bfceb52b > > [2] > > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/d27fcd07541ae6f524115d5b0f36e14673135ca3 > > [3] > > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/e9f2ea6c22f36cb7986d2228763629ed44b9e76b > > > > Signed-off-by: Ulrich Ölmann > > --- > > rules/networkmanager.make | 10 ++ > > 1 file changed, 6 insertions(+), 4 deletions(-) > > > > diff --git a/rules/networkmanager.make b/rules/networkmanager.make > > index de0e755b3a5d..86542b6f74e1 100644 > > --- a/rules/networkmanager.make > > +++ b/rules/networkmanager.make > > @@ -15,15 +15,15 @@ PACKAGES-$(PTXCONF_NETWORKMANAGER) += networkmanager > > # > > # Paths and names > > # > > -NETWORKMANAGER_VERSION := 1.20.8 > > -NETWORKMANAGER_MD5 := c0ceb5ab14bfdfeee07536d94cc5c548 > > +NETWORKMANAGER_VERSION := 1.22.6 > > +NETWORKMANAGER_MD5 := 0f4b493cc0c67f94a2ad3573363eb3b2 > > NETWORKMANAGER := NetworkManager-$(NETWORKMANAGER_VERSION) > > NETWORKMANAGER_SUFFIX := tar.xz > > -NETWORKMANAGER_URL := > > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.20/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) > > +NETWORKMANAGER_URL := > > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.22/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) > > I wonder if it would make sense to use some make magic to derive the > directory name (1.22) from NETWORKMANAGER_VERSION. > > Hmm, this is more complicated than I thought, the following works: > > EMPTY = > SPACE = $(EMPTY) $(EMPTY) > NETWORKMANAGER_MAJOR := $(subst $(SPACE),.,$(wordlist 1, 2, $(subst ., > ,$(NETWORKMANAGER_VERSION > > or > > NETWORKMANAGER_V1 = $(word 1, $(subst ., ,$(NETWORKMANAGER_VERSION))) > NETWORKMANAGER_V2 = $(word 2, $(subst ., ,$(NETWORKMANAGER_VERSION))) > NETWORKMANAGER_MAJOR = $(NETWORKMANAGER_V1).$(NETWORKMANAGER_V2) > > Unless someone can come up with something easier, probably not worth the > effort? There is a much simpler way to do this: NETWORKMANAGER_URL := https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/$(basename $(NETWORKMANAGER_VERSION))/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) We already use this in several other packages. This does not work when a '0' for the patchlevel version is omitted but that's not the case here. Michael -- Pengutronix e.K. | | Steuerwalder Str. 21 | http://www.pengutronix.de/ | 31137 Hildesheim, Germany | Phone: +49-5121-206917-0| Amtsgericht Hildesheim, HRA 2686 | Fax: +49-5121-206917- | ___ ptxdist mailing list ptxdist@pengutronix.de
Re: [ptxdist] [PATCH] networkmanager: version bump 1.20.8-> 1.22.6
On Wed, Feb 12, 2020 at 11:11:12AM +0100, Ulrich Ölmann wrote: > Two new config options have been added in the meantime (see NetworkManager > commits [1] & [2]) which are both set to their reasonable default values. > > The license did not change, but its text was adjusted - citing from > NetworkManagers's commit [3]: > > The update to the GPL text is purely cosmetic. However, shipping the exact > same file as GNU publishes may help distros that deduplicate the license > texts > or hardlink duplicates. > > [1] > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/69f048bf0ca387d2dc4683cfdfe9d170bfceb52b > [2] > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/d27fcd07541ae6f524115d5b0f36e14673135ca3 > [3] > https://gitlab.freedesktop.org/NetworkManager/NetworkManager/commit/e9f2ea6c22f36cb7986d2228763629ed44b9e76b > > Signed-off-by: Ulrich Ölmann > --- > rules/networkmanager.make | 10 ++ > 1 file changed, 6 insertions(+), 4 deletions(-) > > diff --git a/rules/networkmanager.make b/rules/networkmanager.make > index de0e755b3a5d..86542b6f74e1 100644 > --- a/rules/networkmanager.make > +++ b/rules/networkmanager.make > @@ -15,15 +15,15 @@ PACKAGES-$(PTXCONF_NETWORKMANAGER) += networkmanager > # > # Paths and names > # > -NETWORKMANAGER_VERSION := 1.20.8 > -NETWORKMANAGER_MD5 := c0ceb5ab14bfdfeee07536d94cc5c548 > +NETWORKMANAGER_VERSION := 1.22.6 > +NETWORKMANAGER_MD5 := 0f4b493cc0c67f94a2ad3573363eb3b2 > NETWORKMANAGER := NetworkManager-$(NETWORKMANAGER_VERSION) > NETWORKMANAGER_SUFFIX:= tar.xz > -NETWORKMANAGER_URL := > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.20/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) > +NETWORKMANAGER_URL := > https://ftp.gnome.org/pub/GNOME/sources/NetworkManager/1.22/$(NETWORKMANAGER).$(NETWORKMANAGER_SUFFIX) I wonder if it would make sense to use some make magic to derive the directory name (1.22) from NETWORKMANAGER_VERSION. Hmm, this is more complicated than I thought, the following works: EMPTY = SPACE = $(EMPTY) $(EMPTY) NETWORKMANAGER_MAJOR := $(subst $(SPACE),.,$(wordlist 1, 2, $(subst ., ,$(NETWORKMANAGER_VERSION or NETWORKMANAGER_V1 = $(word 1, $(subst ., ,$(NETWORKMANAGER_VERSION))) NETWORKMANAGER_V2 = $(word 2, $(subst ., ,$(NETWORKMANAGER_VERSION))) NETWORKMANAGER_MAJOR = $(NETWORKMANAGER_V1).$(NETWORKMANAGER_V2) Unless someone can come up with something easier, probably not worth the effort? Best regards Uwe -- Pengutronix e.K. | Uwe Kleine-König| Industrial Linux Solutions | https://www.pengutronix.de/ | ___ ptxdist mailing list ptxdist@pengutronix.de