Message ID | 1487093650-6989-1-git-send-email-fff@chrisi01.de |
---|---|
State | Rejected |
Headers | show |
diff --git a/README.md b/README.md index 180f2a6..0e89e74 100644 --- a/README.md +++ b/README.md @@ -1,4 +1,4 @@ -firmware +Firmware ======== # Was ist Freifunk? @@ -16,7 +16,7 @@ Freifunk ist eine nicht-kommerzielle Initiative für freie Funknetzwerke. Jeder Mit Hilfe der Community-Files werden Parameter, wie die ESSID, der Kanal sowie z.B. die Netmon-IP gesetzt. Diese Einstellungen sind Community weit einheitlich und müssen i.d.R. nicht geändert werden. * `./buildscript selectcommunity community/franken.cfg` Je nach dem, für welche Hardware die Firmware gebaut werden soll muss das BSP gewählt werden: -* `./buildscript selectbsp bsp/board wr1043nd.bsp` +* `./buildscript selectbsp bsp/board_ar71xx.bsp` * `./buildscript` ## Was ist ein BSP? @@ -130,7 +130,7 @@ scp root@[ipv6ll%scope]:/etc/network.tl-wr1043nd-v2 /path/to/git/firmware/bsp/wr ``` ### BSP commiten und Patch erzeugen -Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit` checkt man sie ein. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. +Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit --signoff` checkt man sie ein (bei Änderungen git commit --amend --signoff verwenden).. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. ### Patch schicken Auf der Mailingliste franken-dev@freifunk.net kannst du natürlich jederzeit Fragen stellen, falls etwas nicht klar sein sollte.
Hi On 14.02.2017 18:49, Tobias Klaus wrote: > Hey Christian! > > Am 14. Februar 2017 18:34:10 MEZ schrieb Christian Dresel <fff@chrisi01.de>: >> Paar Fehler ausgebessert >> >> Signed-off-by: Christian Dresel <fff@chrisi01.de> >> --- >> README.md | 6 +++--- >> 1 file changed, 3 insertions(+), 3 deletions(-) >> >> diff --git a/README.md b/README.md >> index 180f2a6..0e89e74 100644 >> --- a/README.md >> +++ b/README.md >> @@ -1,4 +1,4 @@ >> -firmware >> +Firmware >> ======== >> >> # Was ist Freifunk? >> @@ -16,7 +16,7 @@ Freifunk ist eine nicht-kommerzielle Initiative für >> freie Funknetzwerke. Jeder >> Mit Hilfe der Community-Files werden Parameter, wie die ESSID, der >> Kanal sowie z.B. die Netmon-IP gesetzt. Diese Einstellungen sind >> Community weit einheitlich und müssen i.d.R. nicht geändert werden. >> * `./buildscript selectcommunity community/franken.cfg` >> Je nach dem, für welche Hardware die Firmware gebaut werden soll muss >> das BSP gewählt werden: >> -* `./buildscript selectbsp bsp/board wr1043nd.bsp` >> +* `./buildscript selectbsp bsp/board_ar71xx.bsp` >> * `./buildscript` > Guter Punkt ☺ wie oft bin ich da beim c&p schon drüber gestolpert und hab mich geärdert, das muss jetzt endlich weg :D > > >> ## Was ist ein BSP? >> @@ -130,7 +130,7 @@ scp root@[ipv6ll%scope]:/etc/network.tl-wr1043nd-v2 >> /path/to/git/firmware/bsp/wr >> ``` >> >> ### BSP commiten und Patch erzeugen >> -Nun kann man mit `git status` die Änderungen sehen. Mit `git add` >> staged man diese und mit `git commit` checkt man sie ein. `git >> format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder >> mehr) Patches. Diese schickst du dann mit `git send-email --to >> franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich >> jemand die Zeit und schaut kurz drüber und wenn alles passt finden >> deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil >> der Freifunk-Franken-Firmware. >> +Nun kann man mit `git status` die Änderungen sehen. Mit `git add` >> staged man diese und mit `git commit --signoff` checkt man sie ein (bei >> Änderungen git commit --amend --signoff verwenden).. > Prinzipiell hast du natürlich recht und wir sollten zum verwenden von 'commit --amend' ermutigen. Allerdings sollte klarer gestellt werden, dass man es nur für nachträgliche Änderungen, die logisch zum letzten patch/commit gehören verwenden sollte. > > Vielleicht so: bei nachträglichen > Änderungen, die logisch zum vorherigen commit gehören, git commit --amend --signoff verwenden. > > Schön, jetzt hab ich zweimal das gleiche geschrieben. ;-) und dennoch komm ich nicht mit. Ich muss gesetehen ich bekomm es selbst nicht richtig hin mit dem amend... Irgendwo hakt da immer irgendwas (wollte nicht mal $jemand nen git Crashkurs machen?!). Ich habs mir einfach mal aus dem Wiki [1] kopiert (was glaub ich auch wieder von mir stammt *g*). Vielleicht kann den Satz einfach mal jemand schöner machen, der auch versteht was er schreibt und es benutzt ;) am Ende vielleicht so, das sogar ich es dann sinnvoll verwenden kann :D mfg Christian [1] https://wiki.freifunk-franken.de/w/FirmwareEntwicklung#Weitere_Informationen > > Viele Grüße > Tobias >
Hey Christian! Am 14. Februar 2017 18:34:10 MEZ schrieb Christian Dresel <fff@chrisi01.de>: >Paar Fehler ausgebessert > >Signed-off-by: Christian Dresel <fff@chrisi01.de> >--- > README.md | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > >diff --git a/README.md b/README.md >index 180f2a6..0e89e74 100644 >--- a/README.md >+++ b/README.md >@@ -1,4 +1,4 @@ >-firmware >+Firmware > ======== > > # Was ist Freifunk? >@@ -16,7 +16,7 @@ Freifunk ist eine nicht-kommerzielle Initiative für >freie Funknetzwerke. Jeder >Mit Hilfe der Community-Files werden Parameter, wie die ESSID, der >Kanal sowie z.B. die Netmon-IP gesetzt. Diese Einstellungen sind >Community weit einheitlich und müssen i.d.R. nicht geändert werden. > * `./buildscript selectcommunity community/franken.cfg` >Je nach dem, für welche Hardware die Firmware gebaut werden soll muss >das BSP gewählt werden: >-* `./buildscript selectbsp bsp/board wr1043nd.bsp` >+* `./buildscript selectbsp bsp/board_ar71xx.bsp` > * `./buildscript` Guter Punkt ☺ > ## Was ist ein BSP? >@@ -130,7 +130,7 @@ scp root@[ipv6ll%scope]:/etc/network.tl-wr1043nd-v2 >/path/to/git/firmware/bsp/wr > ``` > > ### BSP commiten und Patch erzeugen >-Nun kann man mit `git status` die Änderungen sehen. Mit `git add` >staged man diese und mit `git commit` checkt man sie ein. `git >format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder >mehr) Patches. Diese schickst du dann mit `git send-email --to >franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich >jemand die Zeit und schaut kurz drüber und wenn alles passt finden >deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil >der Freifunk-Franken-Firmware. >+Nun kann man mit `git status` die Änderungen sehen. Mit `git add` >staged man diese und mit `git commit --signoff` checkt man sie ein (bei >Änderungen git commit --amend --signoff verwenden).. Prinzipiell hast du natürlich recht und wir sollten zum verwenden von 'commit --amend' ermutigen. Allerdings sollte klarer gestellt werden, dass man es nur für nachträgliche Änderungen, die logisch zum letzten patch/commit gehören verwenden sollte. Vielleicht so: bei nachträglichen Änderungen, die logisch zum vorherigen commit gehören, git commit --amend -- signoff verwenden. Schön, jetzt hab ich zweimal das gleiche geschrieben. ;-) Viele Grüße Tobias
Hi Christian, die Anmerkungen von Tobias sind berechtigt. Wie schon geschrieben, können wir uns da gern mal zusammensetzen. Unten noch was anderes... Am Dienstag, den 14.02.2017, 18:34 +0100 schrieb Christian Dresel: > Paar Fehler ausgebessert > > Signed-off-by: Christian Dresel <fff@chrisi01.de> > --- > README.md | 6 +++--- > 1 file changed, 3 insertions(+), 3 deletions(-) > > diff --git a/README.md b/README.md > index 180f2a6..0e89e74 100644 > --- a/README.md > +++ b/README.md > @@ -1,4 +1,4 @@ > -firmware > +Firmware Das Git-Projekt heißt "firmware" und ist klein geschrieben. Da dies die Erläuterung für das Git Projekt ist, würde ich es auch in der Überschrift bewusst klein lassen. Grüße Jan > ======== > > # Was ist Freifunk? > @@ -16,7 +16,7 @@ Freifunk ist eine nicht-kommerzielle Initiative für freie Funknetzwerke. Jeder > Mit Hilfe der Community-Files werden Parameter, wie die ESSID, der Kanal sowie z.B. die Netmon-IP gesetzt. Diese Einstellungen sind Community weit einheitlich und müssen i.d.R. nicht geändert werden. > * `./buildscript selectcommunity community/franken.cfg` > Je nach dem, für welche Hardware die Firmware gebaut werden soll muss das BSP gewählt werden: > -* `./buildscript selectbsp bsp/board wr1043nd.bsp` > +* `./buildscript selectbsp bsp/board_ar71xx.bsp` > * `./buildscript` > > ## Was ist ein BSP? > @@ -130,7 +130,7 @@ scp root@[ipv6ll%scope]:/etc/network.tl-wr1043nd-v2 /path/to/git/firmware/bsp/wr > ``` > > ### BSP commiten und Patch erzeugen > -Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit` checkt man sie ein. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. > +Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit --signoff` checkt man sie ein (bei Änderungen git commit --amend --signoff verwenden).. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. > > ### Patch schicken > Auf der Mailingliste franken-dev@freifunk.net kannst du natürlich jederzeit Fragen stellen, falls etwas nicht klar sein sollte. > -- > 2.1.4 >
On 02/14/2017 06:34 PM, Christian Dresel wrote: Hi Christian > Paar Fehler ausgebessert Ein guter Anfang, aber es sind noch genug Fehler in der README :) Du hast nicht alle Referenzen zum 1043er bereinigt. Inzwischen sind wir auch von OpenWrt zu LEDE gewechselt und auch unsere Verzeichnisstruktur hat sich veraendert. Und dann gibts noch einige andere Kleinigkeiten die nicht mehr aktuell sind. Keep going, die README kann es vertragen, du schaffst das! :) [...] > ### BSP commiten und Patch erzeugen > -Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit` checkt man sie ein. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. > +Nun kann man mit `git status` die Änderungen sehen. Mit `git add` staged man diese und mit `git commit --signoff` checkt man sie ein (bei Änderungen git commit --amend --signoff verwenden).. `git format-patch origin/HEAD` erzeugt dann aus deinen Commits ein (oder mehr) Patches. Diese schickst du dann mit `git send-email --to franken-dev@freifunk.net *.patch` an unsere Liste. Dort nimmt sich jemand die Zeit und schaut kurz drüber und wenn alles passt finden deine Änderungen in den Hauptentwicklungszweig und sind ab dann Teil der Freifunk-Franken-Firmware. [...] Das mit dem "git -amend" wuerde ich gar nicht erwaehnen: 1. ist dies kein Standard-Anwendungsfall 2. ist unsere README kein Git Tutorial :) Gruss -Steffen
Paar Fehler ausgebessert Signed-off-by: Christian Dresel <fff@chrisi01.de> --- README.md | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-)