Plan: zapit und controld zusammenlegen.

[dwilx]

Damit das hier nicht zu OT wird habe ich zu dem Thema hier einen Thread aufgemacht und hoffe, dass hier nur noch über zapit-controld Thema verhandelt wird.

[Striper]

controld ist ein Stück ziemlich Hitchcock-reifer Code; besonderes ärgerlich ist es, wenn ein Variabel plötzlich seine Bedeutung wechselt!! Hier ist ein etwas aufgeräumte Version von controld.cpp, von der Zeit etwa 2006, wahrscheinlich mit weniger Bugs als die Aktuelle.

Ich wollte deine controld mal antesten, aber leider lässt sie sich nicht kompilieren.

Code: Alles auswählen

make[5]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino/daemons/controld/driver'
make[5]: Entering directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino/daemons/controld'
powerpc-tuxbox-linux-gnu-g++ -DHAVE_CONFIG_H -I. -I../.. -I../../lib -I/home/Striper/Image/dbox2/cdkroot/include/tuxbox/connection -I/home/Striper/Image/dbox2/cdkroot/include/tuxbox -I/home/Striper/Image/dbox2/cdkroot/include -I/home/Striper/Image/tuxbox_cvs/driver/include -I/home/Striper/Image/dbox2/cdkroot/include/tuxbox    -I/home/Striper/Image/tuxbox_cvs/driver/dvb/include -I/home/Striper/Image/tuxbox_cvs/driver/include -Wextra  -Wall -pipe -Os -MT controld.o -MD -MP -MF .deps/controld.Tpo -c -o controld.o controld.cpp
controld.cpp: In function `void setvideooutput(int, bool)':
controld.cpp:422: error: `no_video_output_formats' is not a member of `CControldClient'
controld.cpp: In function `void setVideoFormat(int, bool)':
controld.cpp:506: error: `ASPECT_RATIO_16_9' is not a member of `CControldClient'
controld.cpp:507: error: `ASPECT_RATIO_221_1' is not a member of `CControldClient'
make[5]: *** [controld.o] Fehler 1
make[5]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino/daemons/controld'
make[4]: *** [all-recursive] Fehler 1
make[4]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino/daemons/controld'
make[3]: *** [all-recursive] Fehler 1
make[3]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino/daemons'
make[2]: *** [all-recursive] Fehler 1
make[2]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino'
make[1]: *** [all] Fehler 2
make[1]: Leaving directory `/home/Striper/Image/tuxbox_cvs/apps/tuxbox/neutrino'
make: *** [/home/Striper/Image/dbox2/cdkflash/root-neutrino] Fehler 2

[doc]

Und schon der erste der Offtopic antwortet ...

[seife]

Die ersten Versuche sind hier:
http://gitorious.org/projects/tuxbox-ap ... apit_merge
(dm500.git ist momentan mein Entwicklungsrepo, auch meine dboxen versorge ich daraus (aus master). Der controld_zapit_merge-Branch ist der, in dem ich den neuen zapit probiere. Ich teste es momentan auf der dm500, bin mir auch nicht sicher, ob der controld_zapit_merge-Branch auf der dbox sauber kompiliert, sollte aber relativ trivial zu fixen sein, falls nicht).

[Barf]

controld ist ein Stück ziemlich Hitchcock-reifer Code; besonderes ärgerlich ist es, wenn ein Variabel plötzlich seine Bedeutung wechselt!! Hier ist ein etwas aufgeräumte Version von controld.cpp, von der Zeit etwa 2006, wahrscheinlich mit weniger Bugs als die Aktuelle.

Ich wollte deine controld mal antesten, aber leider lässt sie sich nicht kompilieren.

...

Um mir nicht des Thread-napping schuldig zu machen, antworte ich hier.

[seife]

So. Hier mal als diff gegen's aktuelle CVS:

controld-zapit-merge.diff.gz

(Da sind noch andere sachen mit drin, wie der "--with-boxtype"-Umbau, das war mir jetzt aber zu viel Arbeit, das komplett auseinanderzudröseln).

Code: Alles auswählen

 libs/libconnection/basicsocket.cpp             |    2
 libs/liblcddisplay/lcddisplay.cpp              |    1
 neutrino/daemons/controld/.cvsignore           |    3
 neutrino/daemons/controld/avs_settings.cpp     |   15
 neutrino/daemons/controld/controld.cpp         | 1339 -------------------------
 neutrino/daemons/controld/driver/.cvsignore    |    1
 neutrino/daemons/controld/driver/audio.cpp     |   78 -
 neutrino/daemons/controld/driver/audio.h       |   39
 neutrino/daemons/controld/eventwatchdog.cpp    |  363 ------
 neutrino/daemons/controld/eventwatchdog.h      |   92 -
 neutrino/daemons/controld/scart.conf           |    7
 neutrino/lib/controldclient/.cvsignore         |    5
 neutrino/lib/controldclient/controldMsg.h      |  169 ---
 neutrino/lib/controldclient/controldc.cpp      |  176 ---
 neutrino/lib/controldclient/controldclient.cpp |  300 -----
 neutrino/lib/controldclient/controldclient.h   |  196 ---
 neutrino/lib/controldclient/controldtypes.h    |   62 -
 neutrino/src/driver/bigclock.h                 |    1
 zapit/include/controldclient/controldMsg.h     |  169 +++
 zapit/include/controldclient/controldclient.h  |  196 +++
 zapit/include/zapit/audio.h                    |   12
 zapit/include/zapit/client/msgtypes.h          |    2
 zapit/include/zapit/client/zapitclient.h       |    1
 zapit/include/zapit/client/zapittypes.h        |    7
 zapit/include/zapit/controldtypes.h            |   67 +
 zapit/include/zapit/frontend.h                 |    7
 zapit/include/zapit/settings.h                 |    2
 zapit/include/zapit/types.h                    |   23
 zapit/include/zapit/video.h                    |    2
 zapit/lib/controldclient.cpp                   |  300 +++++
 zapit/lib/zapitclient.cpp                      |    8
 zapit/src/avs_settings.cpp                     |   15
 zapit/src/controld.cpp                         |  825 +++++++++++++++
 zapit/src/controld.h                           |   19
 zapit/src/controldc.cpp                        |  176 +++
 zapit/src/eventwatchdog.cpp                    |  385 +++++++
 zapit/src/eventwatchdog.h                      |   92 +
 zapit/src/getservices.cpp                      |    4
 zapit/src/irsend/irsend.cpp                    |  120 ++
 zapit/src/irsend/irsend.h                      |   31
 zapit/src/pzapit.cpp                           |    8
 zapit/src/scan.cpp                             |   15
 zapit/src/scart.conf                           |    7
 zapit/src/zapit.cpp                            |  351 +++++-
 zapit/src/zapost/audio.cpp                     |  123 ++
 zapit/src/zapost/frontend.cpp                  |  145 ++
 zapit/src/zapost/video.cpp                     |  159 ++
 zapit/tuxbox-zapit.pc.in                       |    2
 48 files changed, 3170 insertions(+), 2952 deletions(-)

Dieser diffstat ist um die acinclude.m4 und configure.ac bereinigt und zeigt, dass hauptsächlich Code verschoben wurde (wenn ich alles aufdrösele vermute ich, dass es jetzt etwas weniger code ist).

Barf: ja, ich weiss, dass das nicht übertrieben schön ist, es sind auch entsprechende Kommentare drin, wo es was zu fixen gibt ;-)

Das Langzeitziel ist es, die controld.cpp-Datei komplett loszuwerden und den controldclient irgendwann auch und nur noch einen zapitclient zu haben. Andererseits habe ich jetzt keine Änderungen in Neutrino machen müssen, theoretisch könnte sogar ein altes Neutrino-Binary noch laufen (mit neuer libzapitclient und libcontroldclient), das habe ich aber nicht probiert.
Wenn man das dann auch noch Stück für Stück aufräumt, dann wird sicher nochmal eine Menge redundanter Code eingespart.

Wenn also an diesem Projekt noch Interesse besteht, dann bitte testen, bei Erfolg würde ich es Stück für Stück ins CVS einarbeiten.
Wenn die "Stabilität" wichtiger ist, dann spiele ich halt in meinem GIT-repo weiter

[dietmarw]

So. Hier mal als diff gegen's aktuelle CVS:

controld-zapit-merge.diff.gz

ergibt

Code: Alles auswählen

checking for linux/dvb/version.h... yes
configure: found dvb version 3
checking for pkg-config... /usr/bin/pkg-config
checking for package tuxbox-configfile... yes
checking for package tuxbox-connection... yes
checking for package tuxbox-eventserver... yes
checking for package tuxbox-lircdclient... no
configure: error: could not find package tuxbox-lircdclient
make[2]: *** [/home/dietmarw/tux_test/20090221/tuxbox-cvs/apps/dvb/zapit/config.status] Fehler 1
make[2]: Leaving directory `/home/dietmarw/tux_test/20090221/tuxbox-cvs/cdk'
make[1]: *** [/home/dietmarw/tux_test/20090221/tuxbox-cvs/apps/dvb/zapit/src/zapit] Fehler 2
make[1]: Leaving directory `/home/dietmarw/tux_test/20090221/tuxbox-cvs/cdk'
make: *** [/home/dietmarw/tux_test/20090221/dbox2/cdkflash/root] Fehler 2 

[rhabarber1848]

Hilft das?

Code: Alles auswählen

--- cdk/make/zapit.mk    2009-01-06 22:40:15.000000000 +0100
+++ cdk/make/zapit.mk 2009-02-21 17:18:32.000000000 +0100
@@ -3,7 +3,7 @@
 $(appsdir)/dvb/zapit/src/zapit:
        $(MAKE) zapit

-$(appsdir)/dvb/zapit/config.status: bootstrap $(targetprefix)/lib/pkgconfig/tuxbox-xmltree.pc
+$(appsdir)/dvb/zapit/config.status: bootstrap $(targetprefix)/lib/pkgconfig/tuxbox-xmltree.pc $(targetprefix)/lib/pkgconfig/tuxbox-tuxtxt.pc
        cd $(appsdir)/dvb/zapit && $(CONFIGURE)

 zapit: $(appsdir)/dvb/zapit/config.status

weil liblircdclient Teil von apps/tuxbox/libs ist und cdk/make/tuxbox_libs.mk:

Code: Alles auswählen

# This file serves as a marker for tuxbox_libs
$(targetprefix)/lib/pkgconfig/tuxbox-tuxtxt.pc:

[dietmarw]

So. Hier mal als diff gegen's aktuelle CVS:

controld-zapit-merge.diff.gz

Hilft das?

jo, ergebnisse http://dietmarw.trale.de/ unter div_tests

[rhabarber1848]

@seife: Könntest Du den Patch aktualisieren, er passt nicht mehr zum CVS.

[seife]

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

Allerdings empfiehlt es sich vermutlich, das in einem frischen Checkout zu machen.
Später einfach mit

Code: Alles auswählen

cd apps
git pull origin controld_zapit_merge

updaten um meine letzten Fixes zu bekommen.

[jojo]

Für schnelle tests einfacher:

in deinem tuxbox-cvs-Verzeichnis

Code: Alles auswählen

mv apps apps-cvs
git pull git://gitorious.org/tuxbox-apps/dm500.git apps

Hmm...

Code: Alles auswählen

jojo@elwood:/home/Playground/tuxbox-clean/tuxbox-cvs_K26> git pull git://gitorious.org/tuxbox-apps/dm500.git apps
fatal: Not a git repository

[seife]

sorry, "git clone..." (oben gefixt).

[seife]

controld_zapit_merge jetzt mit:

• verbessertem ringbuffer-Code in stream2file (Aufnahme auf NFS/Festplatte)
• stark verbessertem Movieplayer2 (unterbrechungsfreies Abspielen von gesplitteten Aufnahmen und VDR-Aufnahmen inklusive rückwärts-springen über den Split, also so als wäre es nur eine Datei)
• potenziellen fixes für den 16bit-Modus (aber nur auf 32bit getestet)
• crashfixes in MP2 und imageinfo
• vielen gefixten memleaks in neutrino

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

(eine bessere Kommandozeile als oben zum auschecken habe ich auch noch gefunden )

Später updaten einfach mit

Code: Alles auswählen

cd apps
git pull

[GetAway]

Da fehlt mir wohl etwas. Alles in einem leeren Ordner ausgecheckt.
Nicht an "2.6" stören. Ordner wird gerade missbraucht. Kompiliert wird mit K2.4

Code: Alles auswählen

checking for pkg-config... /usr/bin/pkg-config
checking for package tuxbox-configfile... yes
checking for package tuxbox-connection... yes
checking for package tuxbox-eventserver... yes
checking for package tuxbox-lircdclient... no
configure: error: could not find package tuxbox-lircdclient
make[2]: *** [/home/getaway/tuxbox/tuxbox-cvs-2.6/apps/dvb/zapit/config.status] Fehler 1
make[2]: Verlasse Verzeichnis '/home/getaway/tuxbox/tuxbox-cvs-2.6/cdk'
make[1]: *** [/home/getaway/tuxbox/tuxbox-cvs-2.6/apps/dvb/zapit/src/zapit] Fehler 2
make: *** [/home/getaway/tuxbox/dbox2-2.6/cdkflash/root] Fehler 2
make[1]: Verlasse Verzeichnis '/home/getaway/tuxbox/tuxbox-cvs-2.6/cdk'
Ready .... press Enter

Edit:
Oops, sehe gerade das Problem gabs oben schon. Dann werde ich mal zapit.mk bearbeiten.

[seife]

Mach mal "make tuxbox_libs", dann nochmal dein ursprüngliches make. Zapit braucht jetzt ein paar sachen, die vorher erst später gebraucht wurden (nämlich vom controld).

Ich bin grad am probieren, aber es wird noch ein paar minuten dauern, bis der bootstrap durch ist.

[GetAway]

Aso, deswegen auch Null Ausgabe über Scart? ... aber Ton OK

Hier mal etwas, das mir aufgefallen ist:

Code: Alles auswählen

[getservices] dup transponder id 11 onid 85
[getservices] dup transponder id 2 onid 85
[getservices] dup transponder id 4 onid 85
[getservices] dup transponder id 1 onid 85
/tmp/currentservices.xml: No such file or directory
[controld] Boxtype detected from mID: unknown (0)        <------------- !!!
/var/etc/init.d/start_neutrino: line 38: controld: not found
[controld]: ROUTEVIDEO v1 = 1 a1 = 1 v2 = 1 a2 = 1 v3 = 1 a3 = 1 fblk=1
void controld_main():782                                   <------------- !!!
[CVideo::setVideoFormat] setting VideoFormat to auto
[CVideo::setVideoFormat] output format: 4:3(LB)
[CVideo::setCroppingMode] setting to letterbox

[seife]

Nein, da sind nur die Abhängigkeiten beim Bauen nicht mehr ganz richtig: tuxbox_libs müsste bei zapit in den dependencies drinstehen.

Dass es nicht funktioniert, wundert mich nun doch etwas, denn meine Philips daheim läuft problemlos damit.
Allerdings habe ich die auch am 12.3. das letzte mal geflashed, evtl. habe ich ja am Wochenende was kaputtgemacht... Schau ich mir nächstes Wochenende mal an.

Prinzipiell müsste es (wg. Movieplayer2) gehen, wenn du einfach die movieplayer2.cpp in deinen CVS checkout kopierst, ich glaube nicht, dass die irgendwas anderes braucht.

Du hast aber schon den controld_zapit_merge branch ausgecheckt und nicht den tripledragon? (siehst du mit "git status")

EDIT:

OOPS.

Code: Alles auswählen

#ifdef HAVE_DBOX_HARDWARE
#define USE_LIBTUXBOX 1
#endif
#define CONFIG_FILE
//#define TRACE
#include <config.h>

HAVE_DBOX_HARDWARE wird in config.h definiert... das kann nicht gehen.
Probier mal das:

Code: Alles auswählen

diff --git a/dvb/zapit/src/controld.cpp b/dvb/zapit/src/controld.cpp
index 6325216..46678e5 100644
--- a/dvb/zapit/src/controld.cpp
+++ b/dvb/zapit/src/controld.cpp
@@ -22,12 +22,12 @@
   Foundation, Inc., 675 Mass Ave, Cambridge, MA 02139, USA.
 */

+#include <config.h>
 #ifdef HAVE_DBOX_HARDWARE
 #define USE_LIBTUXBOX 1
 #endif
 #define CONFIG_FILE
 //#define TRACE
-#include <config.h>

 #include <fcntl.h>
 #include <signal.h>

(sollte trivial von Hand zu machen sein, einfach das #include über das #ifdef)

Aber warum's auf meiner philips funktioniert hat.... keine Ahnung.

[GetAway]

sieht so aus... mp2 ist eh mit drin.

Code: Alles auswählen

getaway@jars-desktop:~/tuxbox/tuxbox-cvs-2.6/apps$ git status
# On branch controld_zapit_merge
nothing to commit (working directory clean)
getaway@jars-desktop:~/tuxbox/tuxbox-cvs-2.6/apps$ 

[seife]

Ich habe den post oben editiert. Da ist noch ein fehlerchen

[GetAway]

Mit den Änderungen funktioniert es jetzt. Ich werd's mal ne Zeit lang testen.

Da ich auch das shellexec Plugin verbaue, ist mir gleich das jetzt wieder inkompatible
Farbpalettenhandling aufgefallen.

Liegt wohl daran. Eventuell ist es jetzt sauber programmiert.

controld_zapit_merge jetzt mit:

• potenziellen fixes für den 16bit-Modus (aber nur auf 32bit getestet)

[seife]

Ja, auf closed-source out-of-tree-plugins kann ich beim Bugfixen leider keine Rücksicht nehmen

[GetAway]

Mir ist aufgefallen das beim starten von Plugins jetzt immer erst der Bildschirm schwarz wird.
Das war vorher nicht so. Kann man beim Kalender, Email und Newsticker Plugin nachvollziehen.
Beim Kalender bleibt das Bild sogar ganz weg.

Dbox-Taste während des MP betriebs, blendet nicht mehr die Spielzeit ein.

[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.
Das hat nichts mit dem controld-zapit-merge an sich zu tun, sondern damit, dass ich natürlich auch andere Bugs in diesem Branch fixe, da er mein Hauptentwicklungszweig ist.

Das mit der Zeit im Movieplayer funktioniert bei mir, ich habe allerdings nur noch den MP2 im Einsatz. Und ich muss zugeben, dass ich es seit einer Woche nur noch auf der TD probiert habe. Ich sehe aber auch nicht, wo das maschinenspezifisch wäre...
Schau mal im bootlog, ob die Start-pts gefunden wurden. Da muss eine Meldung "startpts = xxx" oder "PTS at file start: xxx" (je nach filetyp) kommen.
Oder mach mal in Zeile 3035 (vor dem "if (StreamTime.IsVisible())") ein "INFO("startpts: %d\n", g_startpts);" hin, damit du siehst, ob die dbox-Taste ausgewertet und die startpts schon vorhanden sind (sonst kommt "-1")

[GetAway]

@seife

Mal wieder eine kleine Rückmeldung.
Neutrino segfaultet, wenn im Radiomodus unter Einstellungen/Aufnahme versucht wird das Menü
"Voreinstellung für Tonspuren" oder "Direktaufnahme Einstellungen" zu öffnen.