Movieplayer 1 und 2 / Filmarchiv

[GetAway]

Auf eine saubere Übergabe an den mp2 warten schon einige, dann könnte mp1 eingemottet
und der Moviebrowser weiterhin verwendet werden. Mach einfach mal etwas. Reifen wirds im CVS
dann eh.

[Tahtu]

.

[rhabarber1848]

Für diese Dateien *.patch Dateien erzeugen, in eine .zip Datei packen und auf ein ULC legen?

Ja.

Wir nutzen hier hauptsächlich http://tuxbox-upload.de/ als UCL.
Einen Zugang bekommst Du von CarstenW.

[Striper]

Oder aber du holst dir CVS Zugriff. Der einfachste Weg ist wohl dort zu posten: http://forum.tuxbox-cvs.sourceforge.net ... 18&t=48498

[Tahtu]

.

[dietmarw]

als erstes solltest du dir einen unix konformen editor zulegen..

[Tahtu]

.

[rhabarber1848]

Der Patch: EDIT: Patch ist im CVS

Ich hoffe, ich habe nichts übersehen.
Änderungen:
- dos2unix *
- Dateien waren nicht auf neuestem CVS-Stand, angepasst

[dbt]

@Tahtu
Solltest du bei Cartsten noch keinen ULC-Zugang haben, nimm doch einfach den von mir. Das Werbegesockse auf den "Free"ShareDingern nervt nur und das müssen wir uns zum Glück nicht antun.

[dietmarw]

in rabarbers diff sind nur 5 files betroffen?

und tahtus diff mit configure is auch nich mehr da

[GetAway]

@Tahtu

Bleibt es dabei das du Dich nur um den MB in Verbindung mit dem MP2 kümmerst?
Dann setze ich mal eine 2. Umgebung zum testen auf.
Kompilieren tuts auch so schon.

[Tahtu]

.

[GetAway]

Dafür

[dbt]

Ich hätte noch Lust MoviePlayer2 / MovieBrowser soweit aufzuräumen, dass der Player immer vom Browser aufgerufen wird - nicht andersherum.

Das ist meiner meinung nach das was ich schon öfter angesprochen habe, so und nicht anders.

Allerdings sind diese Aufräumarbeiten schwer, solange der MoviePlayer1 noch auf den MovieBrowser zugreift.

Da ist es manchmal besser, wenn man das neu macht, MovieBrowser2 sag ich nur und den nur mit MoviePlayer2 aufgezogen. Das ganze für beide kompatibel zu machen, wäre mir das beim bloßen dran denken schon zu anstrengend. Vorher war das nur ein Kompromiss und sollte nicht noch mehr werden.

[Tahtu]

.

[MPC823]

Warum nicht wenn die 2te Generation das kann was die 1 kann und dann noch besser spricht doch nichts dagegen oder kann der Movieplayer 1 was was der 2er nicht kann ? Ich glaube es ist eher umgekehrt.


Martin

[Houdini]

PSPad von Windows

der kann bestimmt auch Unix kompatible Zeilenenden, daran sollte es nicht scheitern...

[dietmarw]

... aber dazu hätte ich nur Lust, wenn dabei der MoviePlayer1 ganz aus dem CVS fliegt und auch die beiden Menüpunkte ersatzlos gestrichen werden. Ich vermute meine radikalen Ansätze sind nicht jedem genehm...

sagen wir es mal so..
grade bei solch heißen eisen wie dem mp1 ist evtl. ein radikaler ansatz mal ganz sinnvoll.
(auch wenns ...ihm... nich passt )
denn ohne radikale ansätze wäre vieles im tuxbox projekt immer noch in den kinderschuhen..

[Tahtu]

.

[rhabarber1848]

Patch für configure.ac:

Patch aktualisiert (ungetestet): EDIT: Patch ist im CVS

[rhabarber1848]

EDIT: Patch ist im CVS

Nochmals aktualisiert, moviebrowser.cpp wurde nicht kompiliert,
Problem in src/gui/Makefile.am gelöst.

[rhabarber1848]

Mit dem letzten Patch konnte ich vom TS Filmarchiv
aus eine Datei mit dem movieplayer2 abspielen.

[seife]

Dann sieht es doch schon mal gar nicht so schlecht aus. Ich komme heute nicht mehr dazu, das zu testen, aber unter der Woche wird es wohl werden.

Achtung: der Patch macht den VLC-Fix rückgängig.

[rhabarber1848]

Achtung: der Patch macht den VLC-Fix rückgängig.

Patch aktualisiert

[seife]

Falls es funktioniert, ist es ein proof-of-concept, es gibt noch ein paar Dinge, die ich ändern würde:

Code: Alles auswählen

--- src/gui/moviebrowser.cpp   2009-09-26 15:40:06.000000000 +0200
+++ src/gui/moviebrowser.cpp   2009-10-04 21:23:11.000000000 +0200
...
+#else
+        CMoviePlayerGui* MoviePlayer = new CMoviePlayerGui();
+
+        if (MoviePlayer)
+        {
+            char URL[255];
+            sprintf(URL, "file://%s?startpos=%d", getSelectedFile()->Name.c_str(), m_currentStartPos);
+            std::string actionKey = URL;
+
+            hide();
+
+            MoviePlayer->exec(NULL, actionKey);
+            delete MoviePlayer;

a) URL läuft bei längeren Datei/Verzeichnisnamen über (nein, URL einfach vergrössern ist nicht die richtige Lösung, ich würde einfach gleich einen std::string nehmen)
b) brauchen wir wirklich ein neues CMoviePlayerGui()-Objekt? Wir haben doch schon das globale.