Commit-Wünsche für das CVS II

[rhabarber1848]

Dieser Thread ist die Fortsetzung dieses Threads:
http://forum.tuxbox-cvs.sourceforge.net ... =7&t=48069

Da ich nun CVS commit-Rechte habe (und schon fleißig davon Gebrauch
gemacht habe ), ist es an der Zeit, den alten Thread aufzuräumen.

Hier kommen noch einige Überbleibsel von mir aus dem alten Thread rein und
alle User können CVS-würdige Patches hier einreichen, um die Aufmerksamkeit
zu erhöhen. Gleichzeitig, und das ist eine Bitte an alle, sollten wir das ULC
http://www.tuxbox-upload.de nach Patches durchforsten, die noch nicht im
CVS sind, es aber sein sollten. Solche Patches ebenfalls bitte hier melden,
am besten mit einer kleinen Beschreibung oder einem Link zum jeweiligen
Forumsposting.

[rhabarber1848]

Optionales Soft-Standby nach dem Booten.

Patch: startstandby.diff
Beschreibung: http://forum.tuxbox-cvs.sourceforge.net ... 68#p359768

Dieser Patch ist allerdings nicht fertig und wird so nicht ins CVS kommen,

eine generische "startmodus"-Einstellung, die halt _auch_ "standby" sein kann, wäre evtl. also nicht unnütz.

Dem kann ich mich nur anschließen.

[rhabarber1848]

Netzwerkdevice lo im yadd aktivieren:
http://forum.tuxbox-cvs.sourceforge.net ... 28&start=0

[rhabarber1848]

Neutrino: Radiomode Infobar timeout=0 funktioniert nicht mehr
http://forum.tuxbox-cvs.sourceforge.net ... 11#p364611

Patch ist noch nicht fertig, verhindert die Nutzung der channel history im Radiomodus.

[rhabarber1848]

Patch "antiknarz" -> http://ulc.tuxbox-cvs.sourceforge.net/i ... tory=Diffs&

http://forum.tuxbox-cvs.sourceforge.net ... 45#p364745

[Striper]

Hatte leider noch keine Zeit mich genug mit CVS zu beschäftigen um die Sachen selber einzuchecken, deshalb mach ich einfach hier weiter. Hoffe seife nimmt es mir nicht übel das ich seine Sachen hier verlinke, aber ich denke das kann alles ins CVS:

add a "reZap()" command to libzapitclient, use it after scan
http://gitorious.org/projects/tuxbox-ap ... 8dfc91a882
-> Hab den schwarzen Bildschirm nach der Kanalsuche schon immer nervig gefunden :)


fix timeout during setStandby()
http://gitorious.org/projects/tuxbox-ap ... 0a35749c3a


prevent potential problems with RC_timeout
http://gitorious.org/projects/tuxbox-ap ... 878e9112d0


add rounded corner to the epgview scrollbar
http://gitorious.org/projects/tuxbox-ap ... a708840ad2


add rounded corners to the eventlist scrollbar
http://gitorious.org/projects/tuxbox-ap ... 7587a46395


use full screen height for the eventlist
http://gitorious.org/projects/tuxbox-ap ... 1a7ad48810


fix a potential memory leak
http://gitorious.org/projects/tuxbox-ap ... abb80f8b4c


replace float with integer arithmetic
http://gitorious.org/projects/tuxbox-ap ... 14e5a04c99


convert "vertical tab" (was displayed "S with accent") to newline
http://gitorious.org/projects/tuxbox-ap ... 74bd5e9e13


the separator line between LCD name and title started 1 pixel from left edge
http://gitorious.org/projects/tuxbox-ap ... 39306bb2e9

[seife]

Ganz im Gegenteil, deswegen hab ichs ja veröffentlicht.
Für alle, die mit meinen GIT-Trees arbeiten: ich werde sowohl den controld_zapit_merege-Branch regelmässig auf "mainline" rebasen als auch den tripledragon-Branch auf den controld_zapit_merge rebasen. Das ist nicht optimal, das weiss ich, aber noch kenne ich mich nicht gut genug mit git aus, dass mir was besseres einfallen würde. Im zweifelsfall müsst ihr halt evtl. neu clonen, wenn "git pull" nicht richtig mag. Ich lerne das noch

[rhabarber1848]

Ganz im Gegenteil, deswegen hab ichs ja veröffentlicht.

Mein Problem ist nun, dass ich nicht in allen Fällen einschätzen kann, ob ein
Patch von Dir gut im CVS aufgehoben ist, beim Radio-Infobar-Timeout=0-Patch
habe ich daneben gelegen, obwohl er so gut aussah. Könntest Du bitte kurz die hier
gezeigten Patches kommentieren, besonders im Hinblick auf deren CVS-Würdigkeit?

[seife]

Die sind alle OK (sonst hätte ich schon geschrien). Der "use full screen height for the eventlist" ist etwas Geschmackssache, aus solchen Diskussionen halte ich mich gern raus (ich habe gern möglichst viel auf einmal auf dem Bildschirm und die Fläche möglichst gut genutzt, andere finden bestimmt kleinere Fenster mit Bild aussenrum schöner).

In Zukunft werde ich auch "experimentelle" commits oder solche mit bekannten Problemen explizit benennen, aber das war halt der grosse "importiere alles was du hast"-Marathon

[rhabarber1848]

Die sind alle OK (sonst hätte ich schon geschrien).

Ok, wenn Du die Patches also nicht aus Qualitätsgründen nicht ins CVS
committed hast (Zeitmangel?), dann helfe ich Dir gerne dabei aus.

[ThulsaDoom]

Bin mir nicht sicher, ob es an die Stelle gehört, oder lieber in den eigentlichen Thread, aber den antiknarz-Patch würde ich gerne eingescheckt sehen, sodass die Funktion CVS-Standard wird! Dann ist das im nächsten JtG-Image drinne...

Soweit ich den Diskussions-Thread gelesen habe, nutzt Tommy das nun schon 1 Jahr und ist zufrieden damit! Wenn es funktioniert und keine Nebenwirkungen hat, dann könnte ich meine Box auch mit AC3-Default laufen lassen - was momentan nicht geht, wegen dem Knarzen beim Umschalten.

Wäre echt geil, wenn das weg wäre

Gruß
ThulsaDoom

[rhabarber1848]

add a "reZap()" command to libzapitclient, use it after scan
http://gitorious.org/projects/tuxbox-ap ... 8dfc91a882
-> Hab den schwarzen Bildschirm nach der Kanalsuche schon immer nervig gefunden

Positiv getestet, committed.

[rhabarber1848]

fix timeout during setStandby()
http://gitorious.org/projects/tuxbox-ap ... 0a35749c3a

prevent potential problems with RC_timeout
http://gitorious.org/projects/tuxbox-ap ... 878e9112d0

add rounded corner to the epgview scrollbar
http://gitorious.org/projects/tuxbox-ap ... a708840ad2

add rounded corners to the eventlist scrollbar
http://gitorious.org/projects/tuxbox-ap ... 7587a46395

fix a potential memory leak
http://gitorious.org/projects/tuxbox-ap ... abb80f8b4c

replace float with integer arithmetic
http://gitorious.org/projects/tuxbox-ap ... 14e5a04c99

convert "vertical tab" (was displayed "S with accent") to newline
http://gitorious.org/projects/tuxbox-ap ... 74bd5e9e13

the separator line between LCD name and title started 1 pixel from left edge
http://gitorious.org/projects/tuxbox-ap ... 39306bb2e9

committed

Offen bleibt noch der "use full screen height for the eventlist"-Patch,
der noch zu diskutieren wäre.

[rhabarber1848]

Neutrino GUI-Support für LCD-EPG-Modus mit Sendernamen nur in einer Zeile:
EDIT: Patch ist im CVS

Der Patch dient zur Konfiguration dieser Option:
http://cvs.tuxbox-cvs.sourceforge.net/c ... 64&r2=1.65
http://forum.tuxbox-cvs.sourceforge.net ... 89#p363489

[Striper]

Offen bleibt noch der "use full screen height for the eventlist"-Patch,
der noch zu diskutieren wäre.

Wegen mir sollte das sofort rein. Ich seh das wie seife. Wenn ich ein Menü aufrufe dann will ich auch was sehen davon. So ein Fummelschiss ist doch nix Halbes und nix Ganzes. ;)

[dwilx]

Wegen mir sollte das sofort rein. Ich seh das wie seife. Wenn ich ein Menü aufrufe dann will ich auch was sehen davon. So ein Fummelschiss ist doch nix Halbes und nix Ganzes.

Schon mal die Benutzer gefragt? Meine ja nur, wenn du schon so konsequent sein willst, musst du die komplette Neutrino GUI auf Fullscreen stellen und nicht nur einen Teil und spätestens dann gibts Mecker!

[rhabarber1848]

aber den antiknarz-Patch würde ich gerne eingescheckt sehen

Du meinst diesen Patch hier?
http://forum.tuxbox-cvs.sourceforge.net ... 67#p350967

Ich muss mich hier mangels AC3-Hardware seife anschließen:
http://forum.tuxbox-cvs.sourceforge.net ... 56#p350956

Einchecken tu ich das nicht, weil ich mich zuwenig mit der Hardware auskenne und deswegen die Nebenwirkungen nicht beurteilen kann, das wäre eher was für z.B. carjay.

[rhabarber1848]

Audioplayer Erweiterung für Bedienung nur über LCD (ohne TV) und
Anzeige von albumspezifischen Hintergrundbildern beim Abspielen.
http://forum.tuxbox-cvs.sourceforge.net ... 25#p365025

[rhabarber1848]

Movieplayer-Anpassungen an VideoLAN 0.9.8a:
http://forum.tuxbox-cvs.sourceforge.net ... 39#p363939

EDIT: Patch ist im CVS

[jojo]

Für den IDE-Treiber (2.6er Kernel) würde ich zumindest gerne diesen harmlosen Patch im CVS sehen:

Index: cdk/Patches/linux-2.6.26.4-dbox2.diff
===================================================================
RCS file: /cvs/tuxbox/cdk/Patches/linux-2.6.26.4-dbox2.diff,v
retrieving revision 1.2
diff -u -r1.2 linux-2.6.26.4-dbox2.diff
--- a/cdk/Patches/linux-2.6.26.4-dbox2.diff 4 Feb 2009 10:41:54 -0000 1.2
+++ b/cdk/Patches/linux-2.6.26.4-dbox2.diff 9 Feb 2009 23:19:52 -0000
@@ -536,6 +536,31 @@
NAME = Rotary Wombat

# *DOCUMENTATION*
+
+diff -uNr ./drivers/ide/ide-iops.org.c ../drivers/ide/ide-iops.c
+--- ./drivers/ide/ide-iops.org.c 2008-09-07 19:16:46.000000000 +0200
++++ ./drivers/ide/ide-iops.c 2008-09-07 19:17:36.000000000 +0200
+@@ -95,6 +95,8 @@
+ hwif->OUTB(drive->select.all, hwif->io_ports.device_addr);
+ }
+
++EXPORT_SYMBOL(SELECT_DRIVE);
++
+ static void SELECT_MASK(ide_drive_t *drive, int mask)
+ {
+ const struct ide_port_ops *port_ops = drive->hwif->port_ops;
+
+diff -u ./drivers/ide/ide.c.orig ./drivers/ide/ide.c
+--- ./drivers/ide/ide.c.orig 2009-01-28 08:26:32.378823846 +0100
++++ ./drivers/ide/ide.c 2009-01-28 08:26:43.198939460 +0100
+@@ -95,6 +95,7 @@
+ int noautodma = 0;
+
+ ide_hwif_t ide_hwifs[MAX_HWIFS]; /* master data repository */
++EXPORT_SYMBOL(ide_hwifs); /* required by dboxide */
+
+ static void ide_port_init_devices_data(ide_hwif_t *);
+

[rhabarber1848]

Für den IDE-Treiber (2.6er Kernel) würde ich zumindest gerne diesen harmlosen Patch im CVS sehen

IDE in Kernel 2.6 kompiliert doch im Moment ohnehin nicht,
spricht also etwas dagegen, anstatt dieses kleinen Codeteils
gleich Deinen kompletten Patch ins CVS zu packen?
http://forum.tuxbox-cvs.sourceforge.net ... =7&t=48499

Schlimmer als jetzt kann es nicht werden und Deine HDD funktioniert
wohl, ich kann leider mangels HDD den Patch nicht testen.

[jojo]

Schlimmer als jetzt kann es nicht werden und Deine HDD funktioniert
wohl, ich kann leider mangels HDD den Patch nicht testen.

NIchts dagegen einzuwenden

[GetAway]

add a "reZap()" command to libzapitclient, use it after scan
http://gitorious.org/projects/tuxbox-ap ... 8dfc91a882
-> Hab den schwarzen Bildschirm nach der Kanalsuche schon immer nervig gefunden

Positiv getestet, committed.

Es wäre wünschenswert diese Funktion direkt mit pzapit (z.B -rz) aufrufen zu können.

[Striper]

Es wäre wünschenswert diese Funktion direkt mit pzapit (z.B -rz) aufrufen zu können. :)

Workaround:

Code: Alles auswählen

pzapit -zi $(pzapit -gi | cut -d " " -f 1)

[GetAway]

Deswegen hatte ich "direkt" geschrieben.

Weiss aber gerade auch nicht wie groß der unterschied zwischen zapto und rezap ist.