Plan: zapit und controld zusammenlegen.

[seife]

nur im Radiomodus? seltsam...

[GetAway]

Ja, nur im Radiomodus, aber der Fehler hat nichts mit den beiden Einstellungen zu tun.
Ich habe festgestellt das es immer dann segfaultet, wenn man versucht das 4. Untermenü zu öffnen.
Das wird dir sicher helfen.

[seife]

Also "das 4. von oben"? oder wenn man sich durch 3 menüs bis ins 4. "Verschachtelte" durchgehangelt hat? Nur damit ich's dann auch prompt reproduzieren kann

[GetAway]

Du weißt ja wo Du die beiden oben genannten findest. Zähl selbst.

[seife]

Gefixt. Wobei ich mir nicht sicher bin, warum das nur im controld_zapit_merge auftritt... denn eigentlich ist es ein generisches problem.

[GetAway]

Positiv getestet.

In ganz wenigen Fällen hatte ich bei einem Image aus dem CVS einen Segfault, soweit ich weiss
war es immer im Radiomode beim durchgehen der Menü's. Ich konnte den Fehler aber nie
reproduzieren, bis zum controld_zapit_merge. Vielleicht wars das ja.

[seife]

vermutlich. Ich habe da jetzt eine Warnmeldung drin, aber man könnte auch einfach still die zu grossen Werte clippen

[rhabarber1848]

Gefixt.

http://gitorious.org/projects/tuxbox-ap ... 6d6169f8d3

Ins CVS damit?

[seife]

Ich behaupte mal "es kann nichts schaden". Aber ich bin mit dem CVS-(rück)Stand nicht mehr auf dem laufenden (das ewige zurückportieren hält ziemlich auf, insbesondere wenn ich es auch noch ausprobieren will), deswegen mag ich da keine Empfehlung abgeben.

Momentan pack ich übrigens den tripledragon und den controld-zapit-merge branch zusammen (so dass der tripledragon auch für dbox und dream verwendbar ist), dann wird der tripledragon-Branch "Hauptentwicklungszweig" (das verhindert, dass ich generische fixes ständig hin- und hermergen muss, und zum debuggen ist die TD einfach, wegen RAM und CPU, am besten geeignet).

Edit:
JFTR: Wenn noch Interesse daran besteht, den controld-zapit-merge ins CVS zu übernehmen, dann wäre übrigens jetzt die letzte Möglichkeit, das halbwegs schmerzarm zu machen... Irgendwann werde ich anfangen, den controld-"compat"-Code aus meinem neutrino zu entfernen und dann wird das zurückmergen meiner fixes ... "interessant".

[rhabarber1848]

JFTR: Wenn noch Interesse daran besteht, den controld-zapit-merge ins CVS zu übernehmen, dann wäre übrigens jetzt die letzte Möglichkeit

Eindeutiges Ja. Ich werde heute versuchen, das ganze hier zu testen.

[seife]

Ok, dann warte ich noch mit dem einpflegen der TD-Sachen und teste auch nochmals auf dream und dbox.

[rhabarber1848]

Für schnelle tests einfacher:

in deinem tuxbox-cvs-Verzeichnis

Code: Alles auswählen

mv apps apps-cvs
git clone git://gitorious.org/tuxbox-apps/dm500.git apps
git fetch origin controld_zapit_merge:controld_zapit_merge
git checkout controld_zapit_merge

Funktioniert, bis auf den letzten Befehl:

~/tuxbox/controld_zapit# git checkout controld_zapit_merge
fatal: Not a git repository
~/tuxbox/controld_zapit# cd apps/
~/tuxbox/controld_zapit/apps# git checkout controld_zapit_merge
error: pathspec 'controld_zapit_merge' did not match any file(s) known to git.

[GetAway]

mach mal so:

Code: Alles auswählen

git clone git://gitorious.org/tuxbox-apps/dm500.git apps
cd apps
git checkout -b controld_zapit_merge origin/controld_zapit_merge

[seife]

http://forum.tuxbox-cvs.sourceforge.net ... 08#p366108
Ich muss das mit dem git auch noch lernen ;-)

[rhabarber1848]

Die Unterschiede zwischen Branch controld_zapit_merge und CVS
sind etwas umfangreicher:

- Enigma: showMultiEPG, find_games, USEFREETYPELCD
- Neutrino: Ringbuffer, Framebuffer

und noch diverse andere Dinge, die mit controld/zapit nichts zu tun haben:
http://forum.tuxbox-cvs.sourceforge.net ... 08#p366108

@seife: Was schlägst Du vor? Wie sollen wir vorgehen?
Das komplette diff controld_zapit_merge <-> CVS ins CVS committen,
ist sicher nicht wünschenswert. Die vielen kleinen Patches, die andere
Themen betreffen, zu testen, ist wohl zu komplex und zeitraubend im
Moment.

Wäre es möglich, nur die controld_zapit_merge-Sachen ins CVS zu
committen und Du übernimmst dann die restlichen Patches im
controld_zapit_merge in den TD-Branch, wo sie weiter reifen können?

[seife]

Die Unterschiede zwischen Branch controld_zapit_merge und CVS
sind etwas umfangreicher:
...

Wäre es möglich, nur die controld_zapit_merge-Sachen ins CVS zu
committen und Du übernimmst dann die restlichen Patches im
controld_zapit_merge in den TD-Branch, wo sie weiter reifen können?

Ja. Ich würde erstmal nur die Sachen in apps/dvb/zapit und das Entfernen des controld in apps/tuxbox/neutrino, sowie das "controld->shutdown() -> timerd->shutdown()" in neutrino ins CVS übernehmen.
Der Rest ist erst mal "offtopic" und kann später folgen.

Eine Sache, die kaputtgehen könnte, die ich aber nicht testen kann, ist LIRC, wenn wir also noch einen Betatester mit LIRC-Setup finden, so wäre das nützlich.
Ansonsten ist das auch bei den ersten Bugreports schnell gefixt.

[rhabarber1848]

Kleiner Bug im Branch, der Patch ist überflüssig:

apps/tuxbox/neutrino/src/neutrino.cpp

Code: Alles auswählen

@@ -3481,7 +3559,7 @@
 void CNeutrinoApp::standbyMode( bool bOnOff )
 {
        //printf( ( bOnOff ) ? "mode: standby on\n" : "mode: standby off\n" );
-#ifdef HAVE_DREAMBOX_DM500
+#ifdef HAVE_DREAMBOX_DM5000
        int fd = open("/dev/dbox/fp0", O_RDWR);
        if (fd != -1) {
                int standby = bOnOff?1:0;

[rhabarber1848]

wenn wir also noch einen Betatester mit LIRC-Setup finden, so wäre das nützlich.

Da kann ich leider nicht helfen.

http://wiki.tuxbox-cvs.sourceforge.net/wiki/Lirc

Heißt das, eine nicht umgebaute Dbox kann selber IR-Codes versenden?

[seife]

Tatsächlich ist das DM5000 Absicht
Im controld_zapit_merge ist neuerer dreambox-code als im CVS. Das müsste ich mal aufräumen.

Sprich: entweder "#if 0" statt "#if dm5000" oder besser: ganz weg.

Und ja, wenn ich es richtig verstanden habe, ist das der Sinn vom LIRC-Setup: z.B. dem Fernseher "laut" und "leise" oder dem Beamer "16:9" und "4:3"-Kommandos zu senden.

[seife]

Es gibt einige Sachen, die ich in meinen branches habe, die nicht wirklich was mit dem merge zu tun haben, aber doch nützlich sind (verbesserungen im Zapit für dreambox, umbenennen von "bla_hiess_mal_fake_pat_hat_aber_nix_mit_pat_zu_tun()".

Die checke ich jetzt mal vorneweg ein, dann ist der merge nicht so gross und es wird etwas übersichtlicher.

[rhabarber1848]

es wird etwas übersichtlicher.

Gibt es eine Möglichkeit, das Folgende beim diff zwischen Git und CVS zu verhindern?

Code: Alles auswählen

- * $Id: terrestrial_delivery_system_descriptor.h,v 1.3 2008/03/31 07:49:43 mws Exp $
+ * $Id$

[seife]

nein, es sei denn man würde die CVS-IDs mit ins GIT übernehmen. Will man aber nicht, weil die dort keinen Sinn ergeben. Aber beim Committen ins CVS bekommt man wieder die Versionsnummern, ist also kein Problem.

[rhabarber1848]

Dass das Bild bei plugins schwarz wird, liegt wohl daran, dass ich vom 16bit-modus (falls er denn benutzt wird) erst auf den 8bit-Modus zurückschalte. Das könnte man #ifdef'en.

Ist mir mit dem dvbsub-Plugin auch aufgefallen, außerdem erscheint:

started subtitling filter..
reader_thread started
dvbsub_thread started
reader_thread stopped
dvbsub_thread stopped
[CPlugins] exec ...
avia_gt_gv: set_input_mode (mode=2)
avia_gt_gv: set_input_size (width=720, height=576)
paintBackgroundBoxRel:1081 invalid y (23), dy (556), sum = 579 > yRes (576)

[seife]

Dass das Bild bei plugins schwarz wird, liegt wohl daran, dass ich vom 16bit-modus (falls er denn benutzt wird) erst auf den 8bit-Modus zurückschalte. Das könnte man #ifdef'en.

Ist mir mit dem dvbsub-Plugin auch aufgefallen, außerdem erscheint:

started subtitling filter..
reader_thread started
dvbsub_thread started
reader_thread stopped
dvbsub_thread stopped
[CPlugins] exec ...
avia_gt_gv: set_input_mode (mode=2)
avia_gt_gv: set_input_size (width=720, height=576)
paintBackgroundBoxRel:1081 invalid y (23), dy (556), sum = 579 > yRes (576)

Ja, der paintBackgroundBoxRel-Aufruf übergibt zu grosse Werte. Normalerweise würde neutrino halt segfaulten, oder undefinierte Sachen machen. Jetzt meckerts.
Die Frage ist, ob wir die paintBackgroundBox-Aufrufe alle durchschaune, ob sie Mist machen, oder einfach stillschweigend die Werte auf die zulässigen Werte clippen.

Edit: und die Framebuffersache: wenn man jemals einen Modus mit mehr als 8bit benutzen wollen wird, dann sollte man diese Fixes mit reinnehmen. Wenn man das nicht will, dann kann man die weglassen.

Das Umschalten von 8bit -> 8bit vor dem Pluginstart kann man natürlich noch rausoptimieren

[rhabarber1848]

Im controld-zapit-merge-Branch funktioniert dieser Patch nicht mehr:
http://forum.tuxbox-cvs.sourceforge.net ... 82#p366182

Nach dem Beenden der Neutrino-Esound-Funktion (esd-Daemon war nicht einmal
gestartet, da nicht kompiliert) bleibt der Ton stumm, auch nach Kanalwechsel.