Commit-Wünsche für das CVS II

[Gaucho316]

Ich habe mal in CStringInputSMS die drei Zeichen ' " und & zu den Zeichen hinzugefügt, die auf der Taste 1 liegen. Außerdem kann man jetzt alle Zeichen, die die Klasse CStringInputSMS hergibt, für die Änderung der Bouquetnamen in der Bouquetverwaltung, für die Bearbeitung der Film-Informationen im Moviebrowser und für die EPG-Suche verwenden.

Neutrino: add some special chars to CStringInputSMS
Link entfernt, da Patch in Git

[Gaucho316]

Moviebrowser: show message when deleting files, ported from Coolstream GIT
Link entfernt, da Patch in Git

[Gaucho316]

Ich habe mich heute gefragt, warum im Spiele- und im Skripte-Menü nicht der aktuell ausgewählte Menüeintrag wie in anderen Menüs auf dem LCD angezeigt wird. Der folgende Patch sorgt dafür.

Plugin list: show selected list entry on LCD
Link entfernt, da Patch in Git


In der Bouquetliste fehlt auch noch die Anzeige des gewählten Eintrags auf dem LCD.

Bouquet list: show selected list entry on LCD
Link entfernt, da Patch in Git

[Gaucho316]

Einen im DBox2Word-Forum beschriebenen Fehler kann ich auf der d-box auch nachvollziehen. Deshalb habe ich den passenden Patch aus dem Coolstream GIT portiert.

nhttpd: fix segfault if logging is on, ported from Coolstream GIT
Link entfernt, da Patch in Git


Beim Testen des Patches ist mir aufgefallen, dass in der Timerliste im yWeb ein schließendes td-Tag fehlt.

yWeb: close td tag in timer list
Link entfernt, da Patch in Git

[doc]

Da das Backend des VCS ja nun auf Git umgezogen ist sollte dies das Arbeiten insgesamt einfacher machen. Man kann Commits direkt cherry-picken oder über einen Zwischenschritt in direkt commitfähige Patche ausgeben. Beides erfordert aber final im Tuxbox Git Nacharbeit, wenn nicht im Source dann mindestens bei den Commit Messages.
Da man sich ja üblicher Weise nicht mit fremden Federn schmückt ist der Weg über cherry-picking oder format-patch ein guter Weg den Urheber auch zu würdigen da er als Verfasser des Commits erhalten bleibt. Ein wichtiger Punkt wenn man später Fehler oder Ansprechpartner sucht

Mit git log den [BRANCH] (oder die) Commits suchen die man benutzen möchte. Wenn man cherry-picken will einfach die Commit IDs benutzen.
Will man einen Patch erstellen im [BRANCH] dann einfach

Code: Alles auswählen

git format-patch [SHA1ID~1]..[SHA1ID]

ausführen. Dabei entsteht ein in der Regel je nach Codebasis ein direkt commitfähiger Patch. Siehe auch Development:Git_Patch_erstellen#Patch_von_einem_Einzel-Commit_erstellen und Development:Git_Patch_erstellen#einen_Patch_einspielen

Das selbe geht natürlich auch rein für einen Diff.

Code: Alles auswählen

git diff [SHA1ID~1]..[SHA1ID] > diff_name.diff

Will man nicht alles aus dem originalen Commit übernehmen muss nach gearbeitet werden und der Commit per git commit --amend dann angepasst werden. Jetzt ist dann auch die Möglichkeit gegeben in den Erläuterungen den fremden Originalverfasser zu erwähnen (nein, der gehört nicht in die Zusammenfassung/Überschrift des Commits, ->1.Zeile) und aber auch die Veränderungen zu erklären.

[dbt]

Mit dem Picken ist es aber momentan noch so eine Sache, weil die Treestruktur eigentlich nicht zusammen passt. Ich bin mir noch nicht sicher, wie man das am besten lösen kann (Stichwort: Submodule, Subtree) um das kompatibel zu machen, in beide Richtungen. Aber das fällt eher in das Thema: http://www.tuxbox-cvs.sourceforge.net/f ... 51&t=50224

[Gaucho316]

Ich habe mich jetzt mal ein wenig mit Git auseinandergesetzt. Lohnt es sich, dass ich meine noch nicht eingecheckten Patches als Git-formatierte Patches veröffentliche?

[dbt]

Natürlich, genau so wäre das erwünscht.

[Gaucho316]

Also hier nun die Patches im Git-Format:

Links entfernt, da Patches in Git

[Gaucho316]

nhttpd: don't decode URL parameters twice
Link zum Patch entfernt

[dbt]

Ui, bei so vielen Patches wärs besser du schickst die einfach per Mail oder hängst ein Archiv dran wo man das dann auf einmal runter zieht Und danke, werds gleich mal pushen.

[Gaucho316]

Ach komm, reicht's nicht auch so?

[dbt]

Geht schon, war nur erschrocken, was da zusammen gekommen ist.

Edit: Erledigt

[Gaucho316]

nhttpd: don't decode URL parameters twice
nhttpd_dont-decode-url-params-twice_2012-12-21_1816.patch

Ich musste den Patch überarbeiten, weil einiges überhaupt nicht mehr dekodiert wurde. Jetzt sollte er aber nichts mehr kaputt machen.

nhttpd: don't decode URL parameters multiple times
Link entfernt, da Patch in Git

[GetAway]

Folgender Patch ermöglicht es über Konsole bzw. Skript die Plugins neu zu laden.

Code: Alles auswählen

wget -q -O /dev/null http://localhost/control/reloadplugins

0001-nhttpd-add-command-reloadplugins-to-controlapi.patch

[Gaucho316]

Dieser Patch geht auf eine Idee von micha_bbg aus dem DBox2World-Forum zurück.

yWeb: use 'textarea' for boxcontrol messages
Link entfernt, da Patch in Git

[doc]

Hallo, ein paar Anmerkungen wieder.

Lässt sich der originale Patch nicht cherry picken? Kann mir nicht wirklich vorstellen das das groß an die dbox2 angepasst werden muss. Und wenn wäre eine generische Adaption besser.

Und eine Erläuterung was der Patch verbessert wäre ebenfalls noch wünschenswert, erschließt sich mir jetzt so noch nicht. Als letztes wäre die Angabe des Realnames im Patch noch sinnvolle Angabe, auch wenn ich mich wiederhole, ist alles Open Source und gerade nach den letzten Querelen wegen der Markennamen Eintragung eines Distributors wäre es um so wichtiger nicht nur wegen Lizenzfragen einen Ansprechpartner im Falle der Nachfrage zu haben.

Ich selbst werde nichts pushen wo ein Synonym zu einer Mailadresse hinterlegt ist.

[Mourice]

@Gaucho316

Hi,

kannst Du mir etwas sagen zu dem Patch "neutrino_fix-wrong-infos-on-lcd-on-subchannels_2012-12-23_2253.patch" ?

Ins Git ist er wohl nicht gewandert.

MfG

[Gaucho316]

Lässt sich der originale Patch nicht cherry picken? Kann mir nicht wirklich vorstellen das das groß an die dbox2 angepasst werden muss. Und wenn wäre eine generische Adaption besser.

Der ist noch gar nicht im Coolstream Git und außerdem doch ein klein wenig anders. Das alte und neue yWeb unterscheiden sich eben doch ein wenig. Das neue yWeb ist bspw. mehrsprachig.

[Gaucho316]

Ich habe meine Patches aktualisiert. Jetzt sollte alles passen, hoffe ich zumindest.

Die drei Patches gibt's immer noch hier:
http://forum.tuxbox.org/forum/viewtopic ... 76#p385076
http://forum.tuxbox.org/forum/viewtopic ... 77#p385077
http://forum.tuxbox.org/forum/viewtopic ... 67#p385167

[Mourice]

Folgender Patch ermöglicht es über Konsole bzw. Skript die Plugins neu zu laden.

Code: Alles auswählen

wget -q -O /dev/null http://localhost/control/reloadplugins

0001-nhttpd-add-command-reloadplugins-to-controlapi.patch

Hi,

funktioniert Super.
Danke. Empfinde ich als sinnvolle Ergänzung und sollte ins Git wandern.

Ich kann es auch jeden Fall gut gebrauchen.

MfG

[doc]

Ich habe meine Patches aktualisiert. Jetzt sollte alles passen, hoffe ich zumindest.

Die drei Patches gibt's immer noch hier:
http://forum.tuxbox.org/forum/viewtopic ... 76#p385076
http://forum.tuxbox.org/forum/viewtopic ... 77#p385077
http://forum.tuxbox.org/forum/viewtopic ... 67#p385167

pushed
Thx!

[Gaucho316]

Ich habe bemerkt, dass die Aufnahmehinweisbox, die drei Minuten vor einer Aufnahme angezeigt wird, durch zu lange EPG-Titel kaputt dargestellt wird. Mir ist das schon vor einigen Tagen bei dem Film "Das größte Muppet Weihnachtsspektakel aller Zeiten (It’s a Very Merry Muppet Christmas Movie)" auf MGM aufgefallen. Damals hatte ich aber den Rechner nicht an. Heute konnte ich das dann nochmal nachstellen und korrigieren.

Neutrino hint box: don't overflow screen size
Link entfernt, da Patch in Git

Edit: Ich habe noch eine Kleinigkeit geändert, da ich den Schatten nicht beachtet hatte.

Edit: by doc
pushed mit kleiner Anpassung der Commit Message

[Gaucho316]

Hier nun die gestern Abend von mir angesprochenen Patches, bei deren Erstellung mir aufgefallen ist, dass Code im Git fehlt.

Neutrino: use ShowLocalizedHint() if possible
Link entfernt, da Patch in Git

Neutrino: use ShowLocalizedMessage() if possible
Link entfernt, da Patch in Git

Edit: by doc
Es waren noch ein paar Whitespaces in den Commits, habe diese noch gefixt, ebenfalls habe ich die entsprechenden Codepassagen so umgebrochen das der Code besser lesbar wird.
Ein weiterer Hint, es ist nicht notwendig die Patche anders zu benennen wie diese aus 'git format-pach' kommen, im Gegenteil, meistens bauen Patche aufeinander auf, ohne die Nummerierung die Git vergibt wird es dann etwas mühselig das zu applien.

[Gaucho316]

@doc

Wenn du meine Patches schon änderst, dann mach es bitte richtig. Jetzt fehlt eine Änderung in der Klasse CNetworkSetup. Die kannst du jetzt aber schön alleine suchen.

Du hättest besser getrennte Commits machen sollen - zuerst meine Patches und dann deine Codeverschönerungen. Dann wäre das nicht passiert.