[PATCH] seperated fff-web from fff-base. This should make it easier to switch to a differnet ui package at a later time.

[Tim Niemeyer]

Hi

Am Mittwoch, den 27.01.2016, 23:47 +0100 schrieb Alexander Wunschik:
> Signed-off-by: Alexander Wunschik <freifunk at wunschik.net>
> ---
>  buildscript                       | 2 +-
>  src/packages/fff/fff-web/Makefile | 5 +++--
>  src/packages/fff/fff/Makefile     | 4 ++--
>  3 files changed, 6 insertions(+), 5 deletions(-)
> 
> diff --git a/buildscript b/buildscript
> index f159f26..d26652d 100755
> --- a/buildscript
> +++ b/buildscript
> @@ -40,7 +40,7 @@ ROUTING=(routing
>  ROUTING_PKGS="alfred"
>  
>  FFF=(fff)
> -FFF_PKGS="-a"
> +FFF_PKGS="fff-base fff-nodewatcher fff-web"
Diese Änderung scheint nicht zu dem Patch zu gehören. Sollte in dem Feed
etwas drin sein, was wir nicht brauchen/wollen, sollte es aus diesem
Feed entfernt werden.

>  
>  FEEDS=(OPENWRT ROUTING BATMAN_ADV FFF)
>  
> diff --git a/src/packages/fff/fff-web/Makefile b/src/packages/fff/fff-web/Makefile
> index aefcaa6..2e3b31d 100644
> --- a/src/packages/fff/fff-web/Makefile
> +++ b/src/packages/fff/fff-web/Makefile
> @@ -2,7 +2,7 @@ include $(TOPDIR)/rules.mk
>  
>  PKG_NAME:=fff-web
>  PKG_VERSION:=0.0.3
> -PKG_RELEASE:=1
> +PKG_RELEASE:=2
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/fff-web
>  
> @@ -11,13 +11,14 @@ include $(INCLUDE_DIR)/package.mk
>  define Package/fff-web
>      SECTION:=base
>      CATEGORY:=Freifunk
> +    DEFAULT:=y
Das Paket sollte nur gewählt sein, wenn man wirklich was mit Freifunk
einbaut. Siehe dazu auch die Intention zu dem Meta-Paket fff-base.

>      TITLE:= Freifunk-Franken Webinterface
>      URL:=http://www.freifunk-franken.de
>      DEPENDS:=+uhttpd +haserl +uhttpd-mod-tls +px5g +simple-tc
Man kann fff-web aktivieren, ohne die restlichen Freifunk Komponenten
auszuwählen. Da stimmen die Abhängigkeiten nicht.

>  endef
>  
>  define Package/fff-web/description
> -    This is the Webinterface for the Freifunk Franken Firmware
> +    This is the default Webinterface for the Freifunk Franken Firmware
>  endef
>  
>  define Build/Prepare
> diff --git a/src/packages/fff/fff/Makefile b/src/packages/fff/fff/Makefile
> index e29713f..80b25a7 100644
> --- a/src/packages/fff/fff/Makefile
> +++ b/src/packages/fff/fff/Makefile
> @@ -2,7 +2,7 @@ include $(TOPDIR)/rules.mk
>  
>  PKG_NAME:=fff
>  PKG_VERSION:=0.0.1
> -PKG_RELEASE:=4
> +PKG_RELEASE:=5
>  
>  PKG_BUILD_DIR:=$(BUILD_DIR)/fff
>  
> @@ -14,7 +14,7 @@ define Package/fff-base
>      DEFAULT:=y
>      TITLE:= Freifunk-Franken Base
>      URL:=http://www.freifunk-franken.de
> -    DEPENDS:=+micrond +fff-nodewatcher +fff-web
> +    DEPENDS:=+micrond +fff-nodewatcher
Eigentlich hast du hier schon Recht. fff-base ist nicht abhängig davon.
Die Intention von diesem Paket ist aber, dass es als Meta-Paket dienen
soll. Wenn man dieses wählt werden alle benötigten Freifunk Sachen
angezogen. Diese Firmware soll aber per Default mit dem Webinterface
ausgeliefert werden, daher gehört es da rein.

An dieser Intention würde ich erstmal nichts ändern wollen, da sie
flexibel genug ist deine Vorstellungen mit aufzunehmen. Dies könnte z.B.
geschehen, in dem ein zweites Meta-Paket erstellt wird. Irgendwann
könnten in diesem Szenario sogar mal drei Meta-Pakete sein, eines ohne
Web-Interface, und zwei mit jeweils anderen Web-Interfaces, wobei
letztere vom ersteren abhängen würden.

Tim

>  endef
>  
>  define Package/fff-base/description
> -- 
> 1.8.3.2
> 

-------------- nächster Teil --------------
Ein Dateianhang mit Binärdaten wurde abgetrennt...
Dateiname   : signature.asc
Dateityp    : application/pgp-signature
Dateigröße  : 473 bytes
Beschreibung: This is a digitally signed message part
URL         : <http://lists.freifunk.net/pipermail/franken-dev-freifunk.net/attachments/20160128/dc764e78/attachment-0002.sig>