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

README.md: Various Updates #5

Closed
Felix wants to merge 4 commits from (deleted):readme into master
pull from: (deleted):readme
merge into: freifunk-franken:master
Conversation 4 Commits 4 Files Changed 1 +15 -5
Felix commented 2020-12-12 23:14:45 +01:00
First-time contributor
Copy Link

Dies ist eine aktualisierte version des README.md. Die aktualisierungen wurden aufgeteilt auf 4 commits.

Das ersetzt Pull-Request #1

  • README.md: fix dependencies
  • README.md: change git url to new gitea host
  • README.md: explain more commands from buildscript
  • README.md: remove unnecessary empty lines
Dies ist eine aktualisierte version des README.md. Die aktualisierungen wurden aufgeteilt auf 4 commits. Das ersetzt Pull-Request #1 - README.md: fix dependencies - README.md: change git url to new gitea host - README.md: explain more commands from buildscript - README.md: remove unnecessary empty lines
Felix added 1 commit 2020-12-12 23:14:45 +01:00
ac23913389 README.md: remove unnecessary empty lines 
Signed-off-by: Felix Luber <Felix.Luber@servercreator.de>
adschm reviewed 2020-12-13 00:14:01 +01:00
README.md Outdated
@ -31,2 +30,4 @@
Das Buildscript generiert ein dynamisches sed-Script. Dies geschieht, damit die Templates mit den richtigen Werten gefüllt werden können.
### `./buildscript selectvariant`
Desweiteren wählt man hier aus ob man Node Firmware oder Layer3 Firmware bauen möchte.
adschm commented 2020-12-13 00:14:01 +01:00
Owner
Copy Link

"Des Weiteren". Ich würde das aber ganz anders machen:
"Hier wählt man aus ob man Node Firmware oder Layer3 Firmware bauen möchte:"

"Des Weiteren". Ich würde das aber ganz anders machen: "Hier wählt man aus ob man Node Firmware oder Layer3 Firmware bauen möchte:"
Felix marked this conversation as resolved
adschm reviewed 2020-12-13 00:14:20 +01:00
README.md Outdated
@ -33,0 +32,4 @@
### `./buildscript selectvariant`
Desweiteren wählt man hier aus ob man Node Firmware oder Layer3 Firmware bauen möchte.
* `./buildscript selectvariant [node/layer3]`
* Um die Verschiedenen Varianten zu sehen, kann `./buildscript selectvariant help` ausgeführt werden.
adschm commented 2020-12-13 00:14:20 +01:00
Owner
Copy Link

"verschiedenen" klein schreiben

"verschiedenen" klein schreiben
Felix marked this conversation as resolved
adschm reviewed 2020-12-13 00:21:36 +01:00
README.md Outdated
@ -51,2 +54,4 @@
* Config ins BSP zurück speichern
### `./buildscript updatefeeds`
Aktualisiert die OpenWrt Feeds, die in die Firmware eingebaut werden. Dabei werden alle Pakete, die für bestimmte Feeds angegeben sind, ins OpenWrt Buildsystem eingebunden bzw. aktualisiert. Dieser Schritt wird bereits von `./buildscript prepapre` übernommen, daher ist dies nur bei manuellen Änderungen der Feeds nötig.
adschm commented 2020-12-13 00:21:36 +01:00
Owner
Copy Link

Der zweite Satz ist misleading. Wir aktualisieren die Liste aller Feeds, installieren aber immer nur die, die war ausgewählt haben.

Weiterhin Tippfehler bei buildscript prepare.

Ich würde das anders schreiben:
"Aktualisiert die OpenWrt Feeds für zusätzliche Pakete, die in die Firmware eingebaut werden. Dabei werden die Referenzen im build/ Verzeichnis aktualisiert. Dieser Schritt wird bereits von ./buildscript prepare übernommen, daher ist dies nur bei manuellen Änderungen der Feeds nötig."

Der zweite Satz ist misleading. Wir aktualisieren die Liste _aller_ Feeds, installieren aber immer nur die, die war ausgewählt haben. Weiterhin Tippfehler bei buildscript prepare. Ich würde das anders schreiben: "Aktualisiert die OpenWrt Feeds für zusätzliche Pakete, die in die Firmware eingebaut werden. Dabei werden die Referenzen im build/ Verzeichnis aktualisiert. Dieser Schritt wird bereits von `./buildscript prepare` übernommen, daher ist dies nur bei manuellen Änderungen der Feeds nötig."
Felix marked this conversation as resolved
adschm reviewed 2020-12-13 00:22:41 +01:00
README.md Outdated
@ -53,1 +57,4 @@
Aktualisiert die OpenWrt Feeds, die in die Firmware eingebaut werden. Dabei werden alle Pakete, die für bestimmte Feeds angegeben sind, ins OpenWrt Buildsystem eingebunden bzw. aktualisiert. Dieser Schritt wird bereits von `./buildscript prepapre` übernommen, daher ist dies nur bei manuellen Änderungen der Feeds nötig.
### `./buildscript build`
Sollte man am besten immer in einem screen oder ähnliches laufen lassen um einen Abbruch des Builds bei Verbindungsproblemen oder ähnlichem zu verhindern.
adschm commented 2020-12-13 00:22:41 +01:00
Owner
Copy Link

"in einem screen"
Besser: ... mit Hilfe des Tools 'screen' o.ä. ...

"in einem screen" Besser: ... mit Hilfe des Tools 'screen' o.ä. ...
Felix marked this conversation as resolved
adschm reviewed 2020-12-13 00:23:25 +01:00
README.md Outdated
@ -64,2 +71,4 @@
* board_postbuild() wird aufgerufen
### `./buildscript buildall`
Kann verwendet werden um für alle BSPs Firmware zu bauen, das kann jedoch mehrere Stunden dauern.
adschm commented 2020-12-13 00:23:25 +01:00
Owner
Copy Link

Punkt statt Komma:

"Kann verwendet werden um für alle BSPs Firmware zu bauen. Das kann jedoch mehrere Stunden dauern."

Punkt statt Komma: "Kann verwendet werden um für alle BSPs Firmware zu bauen. Das kann jedoch mehrere Stunden dauern."
Felix marked this conversation as resolved
rohammer reviewed 2020-12-13 00:32:42 +01:00
README.md Outdated
@ -10,3 +10,2 @@
## Voraussetzungen
* `apt-get install zlib1g-dev lua5.2 build-essential unzip libncurses-dev gawk git subversion realpath libssl-dev` (Sicherlich müssen noch mehr Abhängigkeiten installiert werden, diese Liste wird sich hoffentlich nach und nach füllen. Ein erster Ansatzpunkt sind die Abhängigkeiten von OpenWrt selbst)
* `git clone https://github.com/FreifunkFranken/firmware.git`
* `apt-get install zlib1g-dev lua5.2 build-essential unzip libncurses-dev gawk git subversion manpages-pl libssl-dev` (Sicherlich müssen noch mehr Abhängigkeiten installiert werden, diese Liste wird sich hoffentlich nach und nach füllen. Ein erster Ansatzpunkt sind die Abhängigkeiten von OpenWrt selbst)
rohammer commented 2020-12-13 00:32:42 +01:00
Member
Copy Link

Hi, da kann was nicht stimmen. realpath kommt mit den coreutils und sollte da sein. Ich kann mir nicht vorstellen, wie und warum polnische manpages das ersetzen sollen???
Gruesse Robert

Hi, da kann was nicht stimmen. realpath kommt mit den coreutils und sollte da sein. Ich kann mir nicht vorstellen, wie und warum polnische manpages das ersetzen sollen??? Gruesse Robert
Felix commented 2020-12-13 14:23:11 +01:00
Author
First-time contributor
Copy Link

Hi, das ist was APT einem unter Debian 10 vorschlägt. Realpath gibts da wohl nichtmehr. Muss man eventuell nochmal genauer ansehen.

Hi, das ist was APT einem unter Debian 10 vorschlägt. Realpath gibts da wohl nichtmehr. Muss man eventuell nochmal genauer ansehen.
rohammer commented 2020-12-17 01:15:37 +01:00
Member
Copy Link

Hi,
bitte wirf die polnischen manpages raus, die will keiner installiern. Dann kann Adrian das mergen.

Das package realpath gibt es nicht mehr. realpath ist bei den coreutils dabei.

Hi, bitte wirf die polnischen manpages raus, die will keiner installiern. Dann kann Adrian das mergen. Das package realpath gibt es nicht mehr. realpath ist bei den coreutils dabei.
adschm added the
build/scripts/tools
 label 2020-12-13 14:00:39 +01:00
Felix force-pushed readme from f902a654da to d140f32805 2020-12-13 14:17:19 +01:00 Compare
adschm commented 2020-12-13 14:42:39 +01:00
Owner
Copy Link

Ich habe mal alle Commits außer dem mit den Depencies gepickt und gemergt.

Ich habe mal alle Commits außer dem mit den Depencies gepickt und gemergt.
adschm commented 2021-01-13 22:08:08 +01:00
Owner
Copy Link

Ping.

Ping.
fbl commented 2021-01-13 23:55:43 +01:00
Owner
Copy Link

realpath ist schon lange in coreutils. Auch in jessie und stretch war realpath nur ein transitional package, was keinen Inhalt hatte und nur von coreutils abhing. Man könnte diese Abhängigkeit also entfernen.

Woher dieses manpages-pl her kommt, weiß ich nicht..

realpath ist schon lange in coreutils. Auch in jessie und stretch war realpath nur ein transitional package, was keinen Inhalt hatte und nur von coreutils abhing. Man könnte diese Abhängigkeit also entfernen. Woher dieses manpages-pl her kommt, weiß ich nicht..
adschm commented 2021-01-19 15:53:28 +01:00
Owner
Copy Link

Ich werde jetzt einfach realpath in nem eigenen Patch entfernen. Damit ist dieser PR fertig.

Ich werde jetzt einfach realpath in nem eigenen Patch entfernen. Damit ist dieser PR fertig.
adschm closed this pull request 2021-01-19 15:53:28 +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
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#5
No description provided.
Delete Branch "(deleted):readme"

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?