fff-sysupgrade: Remove dependency from fff-hoods

Submitted by Adrian Schmutzler on Nov. 21, 2017, 4:50 p.m.

Details

Message ID 1511283035-3013-1-git-send-email-freifunk@adrianschmutzler.de
State Changes Requested
Delegated to: Adrian Schmutzler
Headers show

Commit Message

Adrian Schmutzler Nov. 21, 2017, 4:50 p.m.
Sysupgrade implements a lib file from fff-hoods to look for the
hood file and read the upgrade path from it.
If fff-hoods is not desired, this poses a problem. Thus, we
move the lib file to fff-config, as its primary purpose is
actually providing the paths to the hood files (=config).

Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
---
 .../fff-config/files/lib/functions/fff/keyxchange  | 32 ++++++++++++++++++++++
 src/packages/fff/fff-hoods/Makefile                |  2 +-
 .../fff-hoods/files/lib/functions/fff/keyxchange   | 32 ----------------------
 src/packages/fff/fff-sysupgrade/Makefile           |  2 +-
 4 files changed, 34 insertions(+), 34 deletions(-)
 create mode 100644 src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
 delete mode 100644 src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
new file mode 100644
index 0000000..30963ae
--- /dev/null
+++ b/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
@@ -0,0 +1,32 @@ 
+#!/bin/sh
+# Copyright 2017 Adrian Schmutzler
+# License GPLv3
+
+. /usr/share/libubox/jshn.sh
+
+hoodfile="/tmp/keyxchangev2data"
+hoodfilecopy="/www/hood/keyxchangev2data"
+
+getJsonPath() {
+	jsonfile=""
+	if [ -s "$hoodfilecopy" ] ; then
+		jsonfile="$hoodfilecopy"
+	elif [ -s "$hoodfile" ] ; then
+		jsonfile="$hoodfile"
+	fi
+	echo "$jsonfile"
+}
+
+getUpgradePath() {
+	jsonfile="$(getJsonPath)"
+
+	if [ -n "$jsonfile" ] ; then
+		json_load "$(cat "$jsonfile")"
+		json_select hood
+		json_get_var upath upgrade_path
+		echo "$upath"
+	else
+		return 1
+	fi
+	return 0
+}
diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile
index 11ab6d1..1a616ff 100644
--- a/src/packages/fff/fff-hoods/Makefile
+++ b/src/packages/fff/fff-hoods/Makefile
@@ -1,7 +1,7 @@ 
 include $(TOPDIR)/rules.mk
 
 PKG_NAME:=fff-hoods
-PKG_VERSION:=0.0.1
+PKG_VERSION:=2
 PKG_RELEASE:=1
 
 PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
diff --git a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
deleted file mode 100644
index 30963ae..0000000
--- a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
+++ /dev/null
@@ -1,32 +0,0 @@ 
-#!/bin/sh
-# Copyright 2017 Adrian Schmutzler
-# License GPLv3
-
-. /usr/share/libubox/jshn.sh
-
-hoodfile="/tmp/keyxchangev2data"
-hoodfilecopy="/www/hood/keyxchangev2data"
-
-getJsonPath() {
-	jsonfile=""
-	if [ -s "$hoodfilecopy" ] ; then
-		jsonfile="$hoodfilecopy"
-	elif [ -s "$hoodfile" ] ; then
-		jsonfile="$hoodfile"
-	fi
-	echo "$jsonfile"
-}
-
-getUpgradePath() {
-	jsonfile="$(getJsonPath)"
-
-	if [ -n "$jsonfile" ] ; then
-		json_load "$(cat "$jsonfile")"
-		json_select hood
-		json_get_var upath upgrade_path
-		echo "$upath"
-	else
-		return 1
-	fi
-	return 0
-}
diff --git a/src/packages/fff/fff-sysupgrade/Makefile b/src/packages/fff/fff-sysupgrade/Makefile
index 4a03652..5bccc65 100644
--- a/src/packages/fff/fff-sysupgrade/Makefile
+++ b/src/packages/fff/fff-sysupgrade/Makefile
@@ -13,7 +13,7 @@  define Package/$(PKG_NAME)
     CATEGORY:=Freifunk
     TITLE:=Freifunk-Franken sysupgrade
     URL:=http://www.freifunk-franken.de
-    DEPENDS:=+fff-boardname +fff-config +fff-hoods +@BUSYBOX_CONFIG_WGET
+    DEPENDS:=+fff-boardname +fff-config +@BUSYBOX_CONFIG_WGET
 endef
 
 define Package/$(PKG_NAME)/description

Comments

Fabian Blaese Nov. 21, 2017, 9:18 p.m.
Brauchen wir für die Gateway Firmware.

Reviewed-by: Fabian Bläse <fabian@blaese.de>

> On 21. Nov 2017, at 17:50, Adrian Schmutzler <freifunk@adrianschmutzler.de> wrote:
> 
> Sysupgrade implements a lib file from fff-hoods to look for the
> hood file and read the upgrade path from it.
> If fff-hoods is not desired, this poses a problem. Thus, we
> move the lib file to fff-config, as its primary purpose is
> actually providing the paths to the hood files (=config).
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
> .../fff-config/files/lib/functions/fff/keyxchange  | 32 ++++++++++++++++++++++
> src/packages/fff/fff-hoods/Makefile                |  2 +-
> .../fff-hoods/files/lib/functions/fff/keyxchange   | 32 ----------------------
> src/packages/fff/fff-sysupgrade/Makefile           |  2 +-
> 4 files changed, 34 insertions(+), 34 deletions(-)
> create mode 100644 src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
> delete mode 100644 src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> 
> diff --git a/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
> new file mode 100644
> index 0000000..30963ae
> --- /dev/null
> +++ b/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
> @@ -0,0 +1,32 @@
> +#!/bin/sh
> +# Copyright 2017 Adrian Schmutzler
> +# License GPLv3
> +
> +. /usr/share/libubox/jshn.sh
> +
> +hoodfile="/tmp/keyxchangev2data"
> +hoodfilecopy="/www/hood/keyxchangev2data"
> +
> +getJsonPath() {
> +	jsonfile=""
> +	if [ -s "$hoodfilecopy" ] ; then
> +		jsonfile="$hoodfilecopy"
> +	elif [ -s "$hoodfile" ] ; then
> +		jsonfile="$hoodfile"
> +	fi
> +	echo "$jsonfile"
> +}
> +
> +getUpgradePath() {
> +	jsonfile="$(getJsonPath)"
> +
> +	if [ -n "$jsonfile" ] ; then
> +		json_load "$(cat "$jsonfile")"
> +		json_select hood
> +		json_get_var upath upgrade_path
> +		echo "$upath"
> +	else
> +		return 1
> +	fi
> +	return 0
> +}
> diff --git a/src/packages/fff/fff-hoods/Makefile b/src/packages/fff/fff-hoods/Makefile
> index 11ab6d1..1a616ff 100644
> --- a/src/packages/fff/fff-hoods/Makefile
> +++ b/src/packages/fff/fff-hoods/Makefile
> @@ -1,7 +1,7 @@
> include $(TOPDIR)/rules.mk
> 
> PKG_NAME:=fff-hoods
> -PKG_VERSION:=0.0.1
> +PKG_VERSION:=2
> PKG_RELEASE:=1
> 
> PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> diff --git a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> deleted file mode 100644
> index 30963ae..0000000
> --- a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> +++ /dev/null
> @@ -1,32 +0,0 @@
> -#!/bin/sh
> -# Copyright 2017 Adrian Schmutzler
> -# License GPLv3
> -
> -. /usr/share/libubox/jshn.sh
> -
> -hoodfile="/tmp/keyxchangev2data"
> -hoodfilecopy="/www/hood/keyxchangev2data"
> -
> -getJsonPath() {
> -	jsonfile=""
> -	if [ -s "$hoodfilecopy" ] ; then
> -		jsonfile="$hoodfilecopy"
> -	elif [ -s "$hoodfile" ] ; then
> -		jsonfile="$hoodfile"
> -	fi
> -	echo "$jsonfile"
> -}
> -
> -getUpgradePath() {
> -	jsonfile="$(getJsonPath)"
> -
> -	if [ -n "$jsonfile" ] ; then
> -		json_load "$(cat "$jsonfile")"
> -		json_select hood
> -		json_get_var upath upgrade_path
> -		echo "$upath"
> -	else
> -		return 1
> -	fi
> -	return 0
> -}
> diff --git a/src/packages/fff/fff-sysupgrade/Makefile b/src/packages/fff/fff-sysupgrade/Makefile
> index 4a03652..5bccc65 100644
> --- a/src/packages/fff/fff-sysupgrade/Makefile
> +++ b/src/packages/fff/fff-sysupgrade/Makefile
> @@ -13,7 +13,7 @@ define Package/$(PKG_NAME)
>     CATEGORY:=Freifunk
>     TITLE:=Freifunk-Franken sysupgrade
>     URL:=http://www.freifunk-franken.de
> -    DEPENDS:=+fff-boardname +fff-config +fff-hoods +@BUSYBOX_CONFIG_WGET
> +    DEPENDS:=+fff-boardname +fff-config +@BUSYBOX_CONFIG_WGET
> endef
> 
> define Package/$(PKG_NAME)/description
> --
> 2.7.4
> 
> --
> franken-dev mailing list
> franken-dev@freifunk.net
> http://lists.freifunk.net/mailman/listinfo/franken-dev-freifunk.net
Fabian Blaese Nov. 22, 2017, 9:52 a.m.
Läuft in der Gatewayfirmware wunderbar.

Tested-by: Fabian Bläse <fabian@blaese.de>
Tim Niemeyer Dec. 23, 2017, 12:08 p.m.
Hi

Am Dienstag, den 21.11.2017, 17:50 +0100 schrieb Adrian Schmutzler:
> Sysupgrade implements a lib file from fff-hoods to look for the
> hood file and read the upgrade path from it.
> If fff-hoods is not desired, this poses a problem. Thus, we
> move the lib file to fff-config, as its primary purpose is
> actually providing the paths to the hood files (=config).
> 
> Signed-off-by: Adrian Schmutzler <freifunk@adrianschmutzler.de>
> ---
>  .../fff-config/files/lib/functions/fff/keyxchange  | 32
> ++++++++++++++++++++++
>  src/packages/fff/fff-hoods/Makefile                |  2 +-
>  .../fff-hoods/files/lib/functions/fff/keyxchange   | 32 ------------
> ----------
>  src/packages/fff/fff-sysupgrade/Makefile           |  2 +-
>  4 files changed, 34 insertions(+), 34 deletions(-)
>  create mode 100644 src/packages/fff/fff-
> config/files/lib/functions/fff/keyxchange
>  delete mode 100644 src/packages/fff/fff-
> hoods/files/lib/functions/fff/keyxchange
> 
> diff --git a/src/packages/fff/fff-
> config/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-
> config/files/lib/functions/fff/keyxchange
> new file mode 100644
> index 0000000..30963ae
> --- /dev/null
> +++ b/src/packages/fff/fff-config/files/lib/functions/fff/keyxchange
Wenn es nur um "config" geht, sollte die Datei nicht keyxchange heißen,
sondern vielleicht "getUpgradePath", wobei dann das getJsonPath
wiederum in eine lib vom keyXchange gehört.

fff-config müsste dann von fff-hoods abhängen, das geht aber nicht,
weil andersrum bereits eine Abhängigkeit besteht.

Wenn es trotzdem entkoppelt sein soll, muss man überlegen, ob es
abseits von fff-hoods noch eine Möglichkeit gibt den upgradepath zu
erlangen. Dann könnte man ein fff-upgradepath (oder fff-config) Package
haben, welches eine ODER Abhängigkeit hat. Fest steht, irgendwo muss
der Pfad herkommen, denn fff-sysupgrade prüft nicht, ob der Pfad
korrekt ist oder nicht.

Tim

> @@ -0,0 +1,32 @@
> +#!/bin/sh
> +# Copyright 2017 Adrian Schmutzler
> +# License GPLv3
> +
> +. /usr/share/libubox/jshn.sh
> +
> +hoodfile="/tmp/keyxchangev2data"
> +hoodfilecopy="/www/hood/keyxchangev2data"
> +
> +getJsonPath() {
> +	jsonfile=""
> +	if [ -s "$hoodfilecopy" ] ; then
> +		jsonfile="$hoodfilecopy"
> +	elif [ -s "$hoodfile" ] ; then
> +		jsonfile="$hoodfile"
> +	fi
> +	echo "$jsonfile"
> +}
> +
> +getUpgradePath() {
> +	jsonfile="$(getJsonPath)"
> +
> +	if [ -n "$jsonfile" ] ; then
> +		json_load "$(cat "$jsonfile")"
> +		json_select hood
> +		json_get_var upath upgrade_path
> +		echo "$upath"
> +	else
> +		return 1
> +	fi
> +	return 0
> +}
> diff --git a/src/packages/fff/fff-hoods/Makefile
> b/src/packages/fff/fff-hoods/Makefile
> index 11ab6d1..1a616ff 100644
> --- a/src/packages/fff/fff-hoods/Makefile
> +++ b/src/packages/fff/fff-hoods/Makefile
> @@ -1,7 +1,7 @@
>  include $(TOPDIR)/rules.mk
>  
>  PKG_NAME:=fff-hoods
> -PKG_VERSION:=0.0.1
> +PKG_VERSION:=2
>  PKG_RELEASE:=1
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/$(PKG_NAME)
> diff --git a/src/packages/fff/fff-
> hoods/files/lib/functions/fff/keyxchange b/src/packages/fff/fff-
> hoods/files/lib/functions/fff/keyxchange
> deleted file mode 100644
> index 30963ae..0000000
> --- a/src/packages/fff/fff-hoods/files/lib/functions/fff/keyxchange
> +++ /dev/null
> @@ -1,32 +0,0 @@
> -#!/bin/sh
> -# Copyright 2017 Adrian Schmutzler
> -# License GPLv3
> -
> -. /usr/share/libubox/jshn.sh
> -
> -hoodfile="/tmp/keyxchangev2data"
> -hoodfilecopy="/www/hood/keyxchangev2data"
> -
> -getJsonPath() {
> -	jsonfile=""
> -	if [ -s "$hoodfilecopy" ] ; then
> -		jsonfile="$hoodfilecopy"
> -	elif [ -s "$hoodfile" ] ; then
> -		jsonfile="$hoodfile"
> -	fi
> -	echo "$jsonfile"
> -}
> -
> -getUpgradePath() {
> -	jsonfile="$(getJsonPath)"
> -
> -	if [ -n "$jsonfile" ] ; then
> -		json_load "$(cat "$jsonfile")"
> -		json_select hood
> -		json_get_var upath upgrade_path
> -		echo "$upath"
> -	else
> -		return 1
> -	fi
> -	return 0
> -}
> diff --git a/src/packages/fff/fff-sysupgrade/Makefile
> b/src/packages/fff/fff-sysupgrade/Makefile
> index 4a03652..5bccc65 100644
> --- a/src/packages/fff/fff-sysupgrade/Makefile
> +++ b/src/packages/fff/fff-sysupgrade/Makefile
> @@ -13,7 +13,7 @@ define Package/$(PKG_NAME)
>      CATEGORY:=Freifunk
>      TITLE:=Freifunk-Franken sysupgrade
>      URL:=http://www.freifunk-franken.de
> -    DEPENDS:=+fff-boardname +fff-config +fff-hoods +@BUSYBOX_CONFIG_
> WGET
> +    DEPENDS:=+fff-boardname +fff-config +@BUSYBOX_CONFIG_WGET
>  endef
>  
>  define Package/$(PKG_NAME)/description
> -- 
> 2.7.4
>