[v1,1/2] packages/fff: add layer3 variant

Submitted by Tim Niemeyer on Feb. 2, 2019, 1:25 p.m.

Details

Message ID 20190202132545.13940-1-tim@tn-x.org
State Superseded
Headers show

Commit Message

Tim Niemeyer Feb. 2, 2019, 1:25 p.m.
Signed-off-by: Tim Niemeyer <tim@tn-x.org>
---

Changes in v1:
- First version of this patch

 src/packages/fff/fff/Makefile | 57 +++++++++++++++++++++++++++++--------------
 1 file changed, 39 insertions(+), 18 deletions(-)

Patch hide | download patch | download mbox

diff --git a/src/packages/fff/fff/Makefile b/src/packages/fff/fff/Makefile
index dce33f9..c135fa6 100644
--- a/src/packages/fff/fff/Makefile
+++ b/src/packages/fff/fff/Makefile
@@ -8,11 +8,10 @@  PKG_BUILD_DIR:=$(BUILD_DIR)/fff
 
 include $(INCLUDE_DIR)/package.mk
 
-define Package/fff-base
+define Package/fff/base
     SECTION:=base
     CATEGORY:=Freifunk
-    DEFAULT:=y
-    TITLE:= Freifunk-Franken Base
+    TITLE:=Freifunk-Franken Base
     URL:=http://www.freifunk-franken.de
     DEPENDS:=+iptables \
              +ip6tables \
@@ -21,37 +20,59 @@  define Package/fff-base
              +fff-config \
              +fff-nodewatcher \
              +fff-web \
-             +fff-uradvd \
              +fff-batman-adv \
-             +fff-firewall \
              +fff-support \
              +fff-network \
              +fff-sysupgrade \
              +fff-wireless \
              +fff-timeserver \
-             +fff-vpn-select \
-             +simple-tc \
-             +fff-hoods
+             +simple-tc
 endef
 
-define Package/fff-base/description
+define Package/fff/base/description
     This is the Freifunk Franken Firmware Base package
 endef
 
-define Build/Prepare
-	echo "all: " > $(PKG_BUILD_DIR)/Makefile
+define Package/fff-node
+    $(call Package/fff/base)
+    DEFAULT:=y
+    TITLE+=(Home node)
+    DEPENDS+=+fff-fastd \
+             +fff-firewall \
+             +fff-vpn-select \
+             +fff-uradvd \
+             +fff-hoods
+    VARIANT:=homenode
 endef
 
-define Build/Configure
-	# nothing
+define Package/fff-node/description
+    $(call Package/fff/base/description)
 endef
 
-define Build/Compile
-	# nothing
+define Package/fff-layer3
+    $(call Package/fff/base)
+    TITLE+=(Layer 3 Gateway)
+    DEPENDS+=+fff-dhcp \
+             +fff-babeld \
+             +fff-openvpn \
+             +arptables \
+             +ebtables \
+             +ebtables-utils \
+             +kmod-ebtables-ipv4 \
+             +kmod-ebtables-ipv6 \
+             +iptables-mod-filter \
+             +iptables-mod-ipopt \
+             +iptables-mod-conntrack-extra \
+             +gre \
+             +@PACKAGE_grev4 \
+             +@PACKAGE_grev6 \
+             +fff-alfred-monitoring-proxy
+    VARIANT:=layer3
 endef
 
-define Package/fff-base/install
-    # nothing
+define Package/fff-layer3/description
+    $(call Package/fff/base/description)
 endef
 
-$(eval $(call BuildPackage,fff-base))
+$(eval $(call BuildPackage,fff-node))
+$(eval $(call BuildPackage,fff-layer3))

Comments

Adrian Schmutzler Feb. 3, 2019, 2:21 a.m.
Hallo,

siehe unten.

> -----Original Message-----
> From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On Behalf
> Of Tim Niemeyer
> Sent: Samstag, 2. Februar 2019 14:26
> To: franken-dev@freifunk.net
> Subject: [PATCH v1 1/2] packages/fff: add layer3 variant
> 
> Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> ---
> 
> Changes in v1:
> - First version of this patch
> 
>  src/packages/fff/fff/Makefile | 57 +++++++++++++++++++++++++++++----
> ----------
>  1 file changed, 39 insertions(+), 18 deletions(-)
> 
> diff --git a/src/packages/fff/fff/Makefile b/src/packages/fff/fff/Makefile
> index dce33f9..c135fa6 100644
> --- a/src/packages/fff/fff/Makefile
> +++ b/src/packages/fff/fff/Makefile
> @@ -8,11 +8,10 @@ PKG_BUILD_DIR:=$(BUILD_DIR)/fff
> 
>  include $(INCLUDE_DIR)/package.mk
> 
> -define Package/fff-base
> +define Package/fff/base
>      SECTION:=base
>      CATEGORY:=Freifunk
> -    DEFAULT:=y
> -    TITLE:= Freifunk-Franken Base
> +    TITLE:=Freifunk-Franken Base
>      URL:=http://www.freifunk-franken.de
>      DEPENDS:=+iptables \
>               +ip6tables \
> @@ -21,37 +20,59 @@ define Package/fff-base
>               +fff-config \
>               +fff-nodewatcher \
>               +fff-web \
> -             +fff-uradvd \
>               +fff-batman-adv \
> -             +fff-firewall \
>               +fff-support \
>               +fff-network \
>               +fff-sysupgrade \
>               +fff-wireless \
>               +fff-timeserver \
> -             +fff-vpn-select \
> -             +simple-tc \
> -             +fff-hoods
> +             +simple-tc
>  endef
> 
> -define Package/fff-base/description
> +define Package/fff/base/description
>      This is the Freifunk Franken Firmware Base package  endef
> 
> -define Build/Prepare
> -	echo "all: " > $(PKG_BUILD_DIR)/Makefile
> +define Package/fff-node
> +    $(call Package/fff/base)
> +    DEFAULT:=y
> +    TITLE+=(Home node)
> +    DEPENDS+=+fff-fastd \
> +             +fff-firewall \
> +             +fff-vpn-select \

vpn-select würde ich hier rauswerfen. Das ist inzwischen ein reines Unterpaket von fff-hoods und wird durch dieses auch selektiert.

> +             +fff-uradvd \
> +             +fff-hoods
> +    VARIANT:=homenode
>  endef
> 
> -define Build/Configure
> -	# nothing
> +define Package/fff-node/description
> +    $(call Package/fff/base/description)
>  endef
> 
> -define Build/Compile
> -	# nothing
> +define Package/fff-layer3
> +    $(call Package/fff/base)
> +    TITLE+=(Layer 3 Gateway)
> +    DEPENDS+=+fff-dhcp \
> +             +fff-babeld \
> +             +fff-openvpn \

fff-openvpn gibt es im Moment nicht.

Dafür könnte man fff-ra und ggf. fff-odhcpd mit reinnehmen.

Grüße

Adrian

> +             +arptables \
> +             +ebtables \
> +             +ebtables-utils \
> +             +kmod-ebtables-ipv4 \
> +             +kmod-ebtables-ipv6 \
> +             +iptables-mod-filter \
> +             +iptables-mod-ipopt \
> +             +iptables-mod-conntrack-extra \
> +             +gre \
> +             +@PACKAGE_grev4 \
> +             +@PACKAGE_grev6 \
> +             +fff-alfred-monitoring-proxy
> +    VARIANT:=layer3
>  endef
> 
> -define Package/fff-base/install
> -    # nothing
> +define Package/fff-layer3/description
> +    $(call Package/fff/base/description)
>  endef
> 
> -$(eval $(call BuildPackage,fff-base))
> +$(eval $(call BuildPackage,fff-node))
> +$(eval $(call BuildPackage,fff-layer3))
> --
> 2.11.0
Tim Niemeyer Feb. 3, 2019, 12:34 p.m.
Am Sonntag, den 03.02.2019, 03:21 +0100 schrieb
mail@adrianschmutzler.de:
> Hallo,
> 
> siehe unten.
> 
> > -----Original Message-----
> > From: franken-dev [mailto:franken-dev-bounces@freifunk.net] On
> > Behalf
> > Of Tim Niemeyer
> > Sent: Samstag, 2. Februar 2019 14:26
> > To: franken-dev@freifunk.net
> > Subject: [PATCH v1 1/2] packages/fff: add layer3 variant
> > 
> > Signed-off-by: Tim Niemeyer <tim@tn-x.org>
> > ---
> > 
> > Changes in v1:
> > - First version of this patch
> > 
> >  src/packages/fff/fff/Makefile | 57 +++++++++++++++++++++++++++++
> > ----
> > ----------
> >  1 file changed, 39 insertions(+), 18 deletions(-)
> > 
> > diff --git a/src/packages/fff/fff/Makefile
> > b/src/packages/fff/fff/Makefile
> > index dce33f9..c135fa6 100644
> > --- a/src/packages/fff/fff/Makefile
> > +++ b/src/packages/fff/fff/Makefile
> > @@ -8,11 +8,10 @@ PKG_BUILD_DIR:=$(BUILD_DIR)/fff
> > 
> >  include $(INCLUDE_DIR)/package.mk
> > 
> > -define Package/fff-base
> > +define Package/fff/base
> >      SECTION:=base
> >      CATEGORY:=Freifunk
> > -    DEFAULT:=y
> > -    TITLE:= Freifunk-Franken Base
> > +    TITLE:=Freifunk-Franken Base
> >      URL:=http://www.freifunk-franken.de
> >      DEPENDS:=+iptables \
> >               +ip6tables \
> > @@ -21,37 +20,59 @@ define Package/fff-base
> >               +fff-config \
> >               +fff-nodewatcher \
> >               +fff-web \
> > -             +fff-uradvd \
> >               +fff-batman-adv \
> > -             +fff-firewall \
> >               +fff-support \
> >               +fff-network \
> >               +fff-sysupgrade \
> >               +fff-wireless \
> >               +fff-timeserver \
> > -             +fff-vpn-select \
Für Kommentar unten: War schon da..

> > -             +simple-tc \
> > -             +fff-hoods
> > +             +simple-tc
> >  endef
> > 
> > -define Package/fff-base/description
> > +define Package/fff/base/description
> >      This is the Freifunk Franken Firmware Base package  endef
> > 
> > -define Build/Prepare
> > -	echo "all: " > $(PKG_BUILD_DIR)/Makefile
> > +define Package/fff-node
> > +    $(call Package/fff/base)
> > +    DEFAULT:=y
> > +    TITLE+=(Home node)
> > +    DEPENDS+=+fff-fastd \
> > +             +fff-firewall \
> > +             +fff-vpn-select \
> 
> vpn-select würde ich hier rauswerfen. Das ist inzwischen ein reines
> Unterpaket von fff-hoods und wird durch dieses auch selektiert.
Mag stimmen, aber ich möchte mit diesem Patch möglichst wenig Impact
machen, was nichts mit diesem Umbau direkt zu tun hat. Daher möchte ich
das gern zunächst 1:1 übernehmen. Du kannst es dann gern mit einem
zusätzlichen Patch entfernen. Falls es später (warum auch immer)
Probleme gibt, dann findet man das schneller und kann es leichter
reverten.

> > +             +fff-uradvd \
> > +             +fff-hoods
> > +    VARIANT:=homenode
> >  endef
> > 
> > -define Build/Configure
> > -	# nothing
> > +define Package/fff-node/description
> > +    $(call Package/fff/base/description)
> >  endef
> > 
> > -define Build/Compile
> > -	# nothing
> > +define Package/fff-layer3
> > +    $(call Package/fff/base)
> > +    TITLE+=(Layer 3 Gateway)
> > +    DEPENDS+=+fff-dhcp \
> > +             +fff-babeld \
> > +             +fff-openvpn \
> 
> fff-openvpn gibt es im Moment nicht.
Ops. Ja.

> Dafür könnte man fff-ra und ggf. fff-odhcpd mit reinnehmen.
Es wird fff-ra werden. fff.odhcpd ist nur der alte Name für fff-ra, das
hatte ich so auch erst eben gerade mitbekommen.

Ich schick gleich ne v2.

Tim

> Grüße
> 
> Adrian
> 
> > +             +arptables \
> > +             +ebtables \
> > +             +ebtables-utils \
> > +             +kmod-ebtables-ipv4 \
> > +             +kmod-ebtables-ipv6 \
> > +             +iptables-mod-filter \
> > +             +iptables-mod-ipopt \
> > +             +iptables-mod-conntrack-extra \
> > +             +gre \
> > +             +@PACKAGE_grev4 \
> > +             +@PACKAGE_grev6 \
> > +             +fff-alfred-monitoring-proxy
> > +    VARIANT:=layer3
> >  endef
> > 
> > -define Package/fff-base/install
> > -    # nothing
> > +define Package/fff-layer3/description
> > +    $(call Package/fff/base/description)
> >  endef
> > 
> > -$(eval $(call BuildPackage,fff-base))
> > +$(eval $(call BuildPackage,fff-node))
> > +$(eval $(call BuildPackage,fff-layer3))
> > --
> > 2.11.0