freifunk-franken
/
firmware
9
7
Fork 13
Code Issues 48 Pull Requests 7 Releases 17 Wiki Activity

fff-web: split hood and UI #6

Closed
adschm wants to merge 3 commits from adschm:webui3 into master
pull from: adschm:webui3
merge into: freifunk-franken:master
Conversation 4 Commits 3 Files Changed 42 +74 -17
adschm commented 2020-12-12 23:25:27 +01:00
Owner
Copy Link

Im Moment macht fff-web zwei verschiedene Dinge: Das Hood-File anbieten und WebUI anzeigen. Dies gehört in zwei verschiedene Packages, dann kann man es auch viel schöner ein- und ausschalten.

Im Moment macht fff-web zwei verschiedene Dinge: Das Hood-File anbieten und WebUI anzeigen. Dies gehört in zwei verschiedene Packages, dann kann man es auch viel schöner ein- und ausschalten.
Felix reviewed 2020-12-12 23:43:17 +01:00
src/packages/fff/fff-web-hood/Makefile Outdated
@ -0,0 +10,4 @@
define Package/fff-web-hood
SECTION:=base
CATEGORY:=Freifunk
TITLE:= Freifunk-Franken web server for hood file
Felix commented 2020-12-12 23:43:10 +01:00
First-time contributor
Copy Link

Ist das Leerzeichen Zwischen "Title=" und "Freifunk" gewollt?

Ist das Leerzeichen Zwischen "Title=" und "Freifunk" gewollt?
adschm commented 2020-12-12 23:50:51 +01:00
Author
Owner
Copy Link

Das ist konsistent inkonsistent. Die Mehrheit scheint aber ohne Leerzeichen zu sein:

src/packages/fff/alfred-json/Makefile:17:       TITLE:=A.L.F.R.E.D - JSON Client
src/packages/fff/fff-alfred/Makefile:13:        TITLE:=Freifunk-Franken Alfred
src/packages/fff/fff-alfred-monitoring-proxy/Makefile:13:       TITLE:=Freifunk-Franken Alfred-Monitoring-Proxy
src/packages/fff/fff-babeld/Makefile:13:        TITLE:=Freifunk-Franken babeld configuration example
src/packages/fff/fff-batman-adv/Makefile:13:    TITLE:=Freifunk-Franken batman-adv
src/packages/fff/fff-boardname/Makefile:13:     TITLE:=Freifunk-Franken boardname
src/packages/fff/fff-config/Makefile:13:        TITLE:=Freifunk-Franken Config
src/packages/fff/fff-dhcp/Makefile:13:  TITLE:=Freifunk-Franken dhcp
src/packages/fff/fff-fastd/Makefile:13: TITLE:=Freifunk-Franken fastd configuration script
src/packages/fff/fff-firewall/Makefile:13:      TITLE:=Freifunk-Franken firewall
src/packages/fff/fff-gateway/Makefile:13:       TITLE:=Freifunk-Franken gateway configuration
src/packages/fff/fff-hoods/Makefile:13: TITLE:= Freifunk-Franken hoods
src/packages/fff/fff-hoodutils/Makefile:13:     TITLE:= Freifunk-Franken hoodutils
src/packages/fff/fff-macnock/Makefile:19:       TITLE:= Freifunk-Franken mac nocker
src/packages/fff/fff/Makefile:13:       TITLE:=Freifunk-Franken Base
src/packages/fff/fff-network/Makefile:13:       TITLE:= Freifunk-Franken network configuration
src/packages/fff/fff-node/Makefile:13:  TITLE:=Freifunk-Franken node configuration
src/packages/fff/fff-nodewatcher/Makefile:13:   TITLE:= Freifunk-Franken Nodewatcher
src/packages/fff/fff-ra/Makefile:13:    TITLE:=Freifunk-Franken Router Advertisements
src/packages/fff/fff-random/Makefile:13:        TITLE:=Freifunk-Franken Random
src/packages/fff/fff-simple-tc/Makefile:13:     TITLE:=Freifunk-Franken simple-tc config
src/packages/fff/fff-support/Makefile:13:       TITLE:=Freifunk-Franken Support Scripts
src/packages/fff/fff-sysupgrade/Makefile:13:    TITLE:=Freifunk-Franken sysupgrade
src/packages/fff/fff-timeserver/Makefile:13:    TITLE:=Freifunk-Franken timeserver
src/packages/fff/fff-uradvd/Makefile:13:        TITLE:= Freifunk-Franken uradvd
src/packages/fff/fff/variant-layer3.mk:5:       TITLE:=Freifunk-Franken Base default switcher
src/packages/fff/fff/variant-node.mk:5: TITLE:=Freifunk-Franken Base default switcher
src/packages/fff/fff-vpn-select/Makefile:13:    TITLE:= Freifunk-Franken vpn-select
src/packages/fff/fff-web-hood/Makefile:13:      TITLE:= Freifunk-Franken web server for hood file
src/packages/fff/fff-web-ui/Makefile:13:        TITLE:= Freifunk-Franken Webinterface
src/packages/fff/fff-wireguard/Makefile:13:     TITLE:=Freifunk-Franken wireguard
src/packages/fff/fff-wireless/Makefile:13:      TITLE:= Freifunk-Franken wireless configuration

Ich werds ändern ...

Das ist konsistent inkonsistent. Die Mehrheit scheint aber ohne Leerzeichen zu sein: ``` src/packages/fff/alfred-json/Makefile:17: TITLE:=A.L.F.R.E.D - JSON Client src/packages/fff/fff-alfred/Makefile:13: TITLE:=Freifunk-Franken Alfred src/packages/fff/fff-alfred-monitoring-proxy/Makefile:13: TITLE:=Freifunk-Franken Alfred-Monitoring-Proxy src/packages/fff/fff-babeld/Makefile:13: TITLE:=Freifunk-Franken babeld configuration example src/packages/fff/fff-batman-adv/Makefile:13: TITLE:=Freifunk-Franken batman-adv src/packages/fff/fff-boardname/Makefile:13: TITLE:=Freifunk-Franken boardname src/packages/fff/fff-config/Makefile:13: TITLE:=Freifunk-Franken Config src/packages/fff/fff-dhcp/Makefile:13: TITLE:=Freifunk-Franken dhcp src/packages/fff/fff-fastd/Makefile:13: TITLE:=Freifunk-Franken fastd configuration script src/packages/fff/fff-firewall/Makefile:13: TITLE:=Freifunk-Franken firewall src/packages/fff/fff-gateway/Makefile:13: TITLE:=Freifunk-Franken gateway configuration src/packages/fff/fff-hoods/Makefile:13: TITLE:= Freifunk-Franken hoods src/packages/fff/fff-hoodutils/Makefile:13: TITLE:= Freifunk-Franken hoodutils src/packages/fff/fff-macnock/Makefile:19: TITLE:= Freifunk-Franken mac nocker src/packages/fff/fff/Makefile:13: TITLE:=Freifunk-Franken Base src/packages/fff/fff-network/Makefile:13: TITLE:= Freifunk-Franken network configuration src/packages/fff/fff-node/Makefile:13: TITLE:=Freifunk-Franken node configuration src/packages/fff/fff-nodewatcher/Makefile:13: TITLE:= Freifunk-Franken Nodewatcher src/packages/fff/fff-ra/Makefile:13: TITLE:=Freifunk-Franken Router Advertisements src/packages/fff/fff-random/Makefile:13: TITLE:=Freifunk-Franken Random src/packages/fff/fff-simple-tc/Makefile:13: TITLE:=Freifunk-Franken simple-tc config src/packages/fff/fff-support/Makefile:13: TITLE:=Freifunk-Franken Support Scripts src/packages/fff/fff-sysupgrade/Makefile:13: TITLE:=Freifunk-Franken sysupgrade src/packages/fff/fff-timeserver/Makefile:13: TITLE:=Freifunk-Franken timeserver src/packages/fff/fff-uradvd/Makefile:13: TITLE:= Freifunk-Franken uradvd src/packages/fff/fff/variant-layer3.mk:5: TITLE:=Freifunk-Franken Base default switcher src/packages/fff/fff/variant-node.mk:5: TITLE:=Freifunk-Franken Base default switcher src/packages/fff/fff-vpn-select/Makefile:13: TITLE:= Freifunk-Franken vpn-select src/packages/fff/fff-web-hood/Makefile:13: TITLE:= Freifunk-Franken web server for hood file src/packages/fff/fff-web-ui/Makefile:13: TITLE:= Freifunk-Franken Webinterface src/packages/fff/fff-wireguard/Makefile:13: TITLE:=Freifunk-Franken wireguard src/packages/fff/fff-wireless/Makefile:13: TITLE:= Freifunk-Franken wireless configuration ``` Ich werds ändern ...
adschm commented 2020-12-12 23:51:11 +01:00
Author
Owner
Copy Link

BTW schnippelt Make das so oder so weg.

BTW schnippelt Make das so oder so weg.
adschm marked this conversation as resolved
adschm force-pushed webui3 from cb90718cdf to fdec8603e7 2020-12-12 23:48:25 +01:00 Compare
adschm force-pushed webui3 from fdec8603e7 to 8c323e5384 2020-12-12 23:57:06 +01:00 Compare
adschm added the
packages/fff
 label 2020-12-13 14:00:01 +01:00
adschm force-pushed webui3 from 8c323e5384 to 2a3c63b9f5 2020-12-13 14:45:00 +01:00 Compare
adschm force-pushed webui3 from 2a3c63b9f5 to af64d2a96b 2020-12-17 17:09:35 +01:00 Compare
ChristianD commented 2020-12-18 11:50:30 +01:00
Member
Copy Link

Schaut grundsätzlich ok aus, ich muss es nochmal einen Tag sacken lassen aber ich denke das müsste so passen. Eine Kleinigkeit hab ich im Code noch kommentiert.

Schaut grundsätzlich ok aus, ich muss es nochmal einen Tag sacken lassen aber ich denke das müsste so passen. Eine Kleinigkeit hab ich im Code noch kommentiert.
adschm commented 2020-12-18 13:51:04 +01:00
Author
Owner
Copy Link

ah das versemmelt hier gitea. Du hast das in 2 Patches jeweils um +1 erhöht was insgesamt dann +2 ;)

Deswegen schaue ich mir immer die Commits einzeln an und nervt es mich total, dass man nicht direkt dort inline kommentieren kann wie in GitHub. Das ist tatsächlich die einzige wirklich relevante funktionale Einschränkung von Gitea, der Rest ist primär nur lästig, aber nicht so wichtig.

> ah das versemmelt hier gitea. Du hast das in 2 Patches jeweils um +1 erhöht was insgesamt dann +2 ;) Deswegen schaue ich mir immer die Commits einzeln an und nervt es mich total, dass man nicht direkt dort inline kommentieren kann wie in GitHub. Das ist tatsächlich die einzige wirklich relevante funktionale Einschränkung von Gitea, der Rest ist primär nur lästig, aber nicht so wichtig.
rohammer approved these changes 2020-12-22 00:22:30 +01:00
rohammer left a comment
Member
Copy Link

Find ich gut!

Reviewed-by: Robert Langhammer <rlanghammer@web.de>
Find ich gut! ``` Reviewed-by: Robert Langhammer <rlanghammer@web.de> ```
ChristianD requested changes 2020-12-22 13:51:24 +01:00
src/packages/fff/fff-web-ui/Makefile
@ -0,0 +1,43 @@
include $(TOPDIR)/rules.mk
PKG_NAME:=fff-web-ui
PKG_RELEASE:=15
ChristianD commented 2020-12-18 11:47:47 +01:00
Member
Copy Link

warum +2?

warum +2?
ChristianD commented 2020-12-18 11:51:29 +01:00
Member
Copy Link

ah das versemmelt hier gitea. Du hast das in 2 Patches jeweils um +1 erhöht was insgesamt dann +2 ;)

ah das versemmelt hier gitea. Du hast das in 2 Patches jeweils um +1 erhöht was insgesamt dann +2 ;)
src/packages/fff/fff-web-ui/files/etc/uci-defaults/93-fff-uhttpd
ChristianD commented 2020-12-19 08:47:18 +01:00
Member
Copy Link

wenn ich das richtig sehe, ist das nun in beiden 93-er uci-defaults von den 2 packages oder?

  1. Ich sehe es doch richtig, das damit die uhttpd defaults gelöscht werden oder? Gefällt mir irgendwie nicht wenn wir das in jeden Package machen. Dummerweise haben wir kein uhttpd "Vorbereitungspackage" oder? Bin mir jetzt gerade auch bisschen unschlüssig wie das schöner wäre.

  2. Wenn es ein "default löschen" ist, warum nur dies und nicht der ganze batch Block danach der nach meiner Ansicht auch noch defaults weg wirft oder?

wenn ich das richtig sehe, ist das nun in beiden 93-er uci-defaults von den 2 packages oder? 1) Ich sehe es doch richtig, das damit die uhttpd defaults gelöscht werden oder? Gefällt mir irgendwie nicht wenn wir das in jeden Package machen. Dummerweise haben wir kein uhttpd "Vorbereitungspackage" oder? Bin mir jetzt gerade auch bisschen unschlüssig wie das schöner wäre. 2) Wenn es ein "default löschen" ist, warum nur dies und nicht der ganze batch Block danach der nach meiner Ansicht auch noch defaults weg wirft oder?
adschm commented 2020-12-22 13:55:50 +01:00
Author
Owner
Copy Link

uhttpd.main kommt von OpenWrt. D.h. das muss man immer löschen, egal ob nur eine Package installiert ist, oder beide. Sind beide installiert, kann man es beim zweiten Mal aber nicht mehr löschen, daher das "-q".

Alle anderen deletes im batch-Teil sind jeweils Package-spezifisch, d.h. die löschen nur Sachen die auch in der jeweiligen Package (also sicher nur einmal) erstellt werden.

Tatsächlich muss ich allerdings sagen, das z.B. uhttpd.public zu löschen eigentlich keinen Sinn macht, da das ja uci-defaults ist, es also noch gar kein uhttpd.public geben kann. Das könnte man separat mal entfernen und dort ggf. weiter diskutieren.

Aber die Situation mit httpd.main entsteht durch diesen Patch, also will ich die auch hier beheben.

uhttpd.main kommt von OpenWrt. D.h. das muss man immer löschen, egal ob nur eine Package installiert ist, oder beide. Sind beide installiert, kann man es beim zweiten Mal aber nicht mehr löschen, daher das "-q". Alle anderen deletes im batch-Teil sind jeweils Package-spezifisch, d.h. die löschen nur Sachen die auch in der jeweiligen Package (also sicher nur einmal) erstellt werden. Tatsächlich muss ich allerdings sagen, das z.B. uhttpd.public zu löschen eigentlich keinen Sinn macht, da das ja uci-defaults ist, es also noch gar kein uhttpd.public geben kann. Das könnte man separat mal entfernen und dort ggf. weiter diskutieren. Aber die Situation mit httpd.main entsteht durch diesen Patch, also will ich die auch hier beheben.
ChristianD commented 2020-12-22 13:58:41 +01:00
Member
Copy Link

uhttpd.main kommt von OpenWrt. D.h. das muss man immer löschen, egal ob nur eine Package installiert ist, oder beide. Sind beide installiert, kann man es beim zweiten Mal aber nicht mehr löschen, daher das "-q".

Besser wäre da aber wohl ein uhttpd "Vorbereitungspackage" wo das einmalig gelöscht wird. Aber gut kann ich erstmal so mit leben, kann man vllt. mal irgendwann noch verbessern

Alle anderen deletes im batch-Teil sind jeweils Package-spezifisch, d.h. die löschen nur Sachen die auch in der jeweiligen Package (also sicher nur einmal) erstellt werden.

stimmt, ist mir auch eben aufgefallen

Tatsächlich muss ich allerdings sagen, das z.B. uhttpd.public zu löschen eigentlich keinen Sinn macht, da das ja uci-defaults ist, es also noch gar kein uhttpd.public geben kann. Das könnte man separat mal entfernen und dort ggf. weiter diskutieren.

ack

Aber die Situation mit httpd.main entsteht durch diesen Patch, also will ich die auch hier beheben.

kann ich auch erstmal mit leben und wills nicht länger blockieren

> uhttpd.main kommt von OpenWrt. D.h. das muss man immer löschen, egal ob nur eine Package installiert ist, oder beide. Sind beide installiert, kann man es beim zweiten Mal aber nicht mehr löschen, daher das "-q". Besser wäre da aber wohl ein uhttpd "Vorbereitungspackage" wo das einmalig gelöscht wird. Aber gut kann ich erstmal so mit leben, kann man vllt. mal irgendwann noch verbessern > Alle anderen deletes im batch-Teil sind jeweils Package-spezifisch, d.h. die löschen nur Sachen die auch in der jeweiligen Package (also sicher nur einmal) erstellt werden. stimmt, ist mir auch eben aufgefallen > Tatsächlich muss ich allerdings sagen, das z.B. uhttpd.public zu löschen eigentlich keinen Sinn macht, da das ja uci-defaults ist, es also noch gar kein uhttpd.public geben kann. Das könnte man separat mal entfernen und dort ggf. weiter diskutieren. ack > Aber die Situation mit httpd.main entsteht durch diesen Patch, also will ich die auch hier beheben. kann ich auch erstmal mit leben und wills nicht länger blockieren
ChristianD commented 2020-12-22 13:59:05 +01:00
Member
Copy Link
Acked-by: Christian Dresel <freifunk@dresel.systems>
``` Acked-by: Christian Dresel <freifunk@dresel.systems> ```
ChristianD approved these changes 2020-12-22 13:59:45 +01:00
adschm commented 2020-12-22 14:12:10 +01:00
Author
Owner
Copy Link

Danke, wird mit gemergt.

Danke, wird mit gemergt.
adschm closed this pull request 2020-12-22 14:12:10 +01:00

Pull request closed

This pull request cannot be reopened because the branch was deleted.
Sign in to join this conversation.
Reviewers
No reviewers
rohammer
ChristianD
Labels
Clear labels   
RFC

   
RFT

   
WIP

   
blocked

   
bsp

touching bsp files or adding devices

  
bug

Something is not working

  
build/scripts/tools

touching buildscript or other tools

  
duplicate

This issue or pull request already exists

  
feature

New feature

  
fixed

Issue that got resolved

  
layer3

only for layer3 FW

  
mantis

Imported from mantis

  
more details required

Reported information is insufficient to debug the issue

  
needs changes

   
node

only for node FW

  
packages/fff

touching our packages

  
rejected

Rejected or won't fix

  
security

   
trivial

simple non-functional changes that are mostly matter of taste

  
upstream

touching upstream version, build or feed patches

No Label
RFC
RFT
WIP
blocked
bsp
bug
build/scripts/tools
duplicate
feature
fixed
layer3
mantis
more details required
needs changes
node
packages/fff
rejected
security
trivial
upstream
Milestone
Clear milestone
No items
No Milestone
Assignees
Clear assignees
No Assignees
4 Participants
Notifications
Due Date
The due date is invalid or out of range. Please use the format 'yyyy-mm-dd'.

No due date set.

Dependencies

No dependencies set.

Reference: freifunk-franken/firmware#6
No description provided.
Delete Branch "adschm:webui3"

Deleting a branch is permanent. Although the deleted branch may continue to exist for a short time before it actually gets removed, it CANNOT be undone in most cases. Continue?