Posts by pbrb

gggggg : leider kann ich dazu nichts (mehr) beisteuern, habe die eHD aktuell schlafen gelegt (in der Test-Reelbox ist sie noch drin...) - Hab mal vor einiger Zeit rumgefragt, wer denn eine lauffähige Buildumgebung für das MIPS eHD-Image hat...leider nichts erfahren.

Die Images, die mir über den Weg gelaufen sind, habe ich hier abgespeichert:

https://github.com/pbiering/Re…tree/main/bin-reelvdr/eHD

Einzig das Linux-Kernel-Modul habe ich mal mit etlichen Debug-Optionen versehen und 64-bit Kernel-Support eingebaut.

https://github.com/pbiering/Re…/src-reelvdr/utils/hdshm3

Falls aber auf Linux-Kernel-Modulseite nichts zu sehen ist, müßte man mal ins MIPS-Image Debug-Optionen reinstreuen.

Ggf. findet man aber schon Hinweise, wenn man sich per Telnet auf eHD einloggt und mal da im Kernel Log (dmesg) guckt...vielleicht sind irgendwelche Puffer zu klein.

0.9.7 released: https://github.com/vdr-projects/vdr-plugin-mcli/releases

Hoffe, daß da keine anderen Patches mehr fehlen...sonst gibt's halt 0.9.7.1

Quote from gggggg

letzter Feinschliff:

wenn --netcvupdate-enable-debug NICHT gesetzt ist, sollte auch die Ausgabe des Datentransfers weg sein:

die FTP-Ausgabe wird mit "-q" unterdrückt....eingebaut in cam_menu.c:

https://github.com/pbiering/vd…f867cb149ed74382155f09fd9

Quote from gggggg

Wenn beim Download der File lokal schon existiert kommt folgende Meldung:

Code

get: /root/netceiver.conf: Datei existiert bereits und xfer:clobber ist nicht gesetzt
---- Schließe die Datenverbindung
---- Schließe den Kontroll - Socket
Download failed (ret=256)

Das war aber denke ich schon immer so !

Kann auch sein, daß der ursprüngliche "ftp" client da einfach gnadenloser war und die lokale Datei überschrieben hat ohne zu Meckern.

Hab mal bei "lftp" den "get" um Option "-a" erweitert, damit meckert nix mehr.

https://github.com/pbiering/vd…5746259a4853e8f1cef1f6e33

Bitte mal final testen, dann merge ich den Branch nach "master".

Ist sonst noch was offen? Weil sonst erstelle ich einen Release.

"netcvupdate" ist aber nicht reinkompiliert ins Plugin (wäre ja sonst eine Art Bibliotheksaufruf notwendig), sondern wird als externes Kommando aufgerufen durch "SystemExec(c)" mit dem vorher zusammenstellten String "c"....und Ihr wolltet da die Zusatzoption "-o" reinschmuggeln....das hat nicht geklappt (war fast erwartbar).

Dewegen hat "netcvupdate" jetzt 2 neue Schalter, die im String "c" entsprechend gesetzt werden.

Alles, was man sieht in

dsyslog("EXEC1 %s", (const char *)c);

dsyslog("EXEC2 %s", (const char *)c);

wird aufgerufen als externes Kommando.

Die Kette ist also: MCLI-Plugin-Menü -> Triggert Aktion -> ruft "netcvupdate" mit Optionen auf -> das wiederum ruft den ftp-Client (auch als externes Kommando) auf mit einem zusammengestellten Script.

Man könnte das natürlich alles mit einer Art "libftp" all-in-one machen...aber der Code ist halt historisch so zusammen verknüpft.

Quote from cinfo

aus dem Git bekomme ich diese hier -- wenn eine andere Version benötigt wird dann hänge sie bitte hier an

Code

netcvupdate.c
add various default flags for lftp to avoid unsupported commands on v…
3 hours ago

die paßt...aber gggggg benötigt die Binaries irgendwie, bisher war im tar.bz2 wohl immer nur das Plugin drin und nicht die 3 externen Binaries: netcvdiag, netcvlogview, netcvupdate -> mindestens "netcvupdate" muß aktualisiert werden.

Zur "neuesten" Plugin-Version brauchst Du auch das neueste "netcvupdate"...sonst kommst nicht weiter.

Anbei mal statisch kompilierte Versionen, aber bitte nur zum Testen verwenden...mußt das aktuell genutzte "netcvupdate" überschreiben (und natürlich "-static" entfernen...

der "make" eines Plugins wird ohne entsprechende vdr-devel nicht gehen...da braucht's schon bisserl Vorbereitung.

Aber der Branch "lftp-fixes" müßte eigentlich nun funktionieren, wenn "netcvupdate" auch auf dem Stand vom Branch ist, weil das ist Voraussetzung, daß "-o" aktiv wird.

Und daß sich "-o" nicht via cam_menu.c "reinschmuggeln" ließ, dann daran liegen, weil ein implizites Escaping der Argumente beim Aufruf von "netcvupdate" ggf. passiert...d.h. eine Zusatzoption für's finale FTP-Script geht über diesen Weg nicht rein.

funktioniert auch mit "Pfad":

./mcast/tool/netcvupdate -n  -e -i fe80::208:54ff:fe**:**** -d enp4s0 -U /tmp/netceiver.conf
INFO  : enable options for FTP client 'lftp'
DEBUG : enable debugging
UUID fe80::208:54ff:fe**:****: Uploading /tmp/netceiver.conf ...

DEBUG : execute FTP command: lftp --norc -d
DEBUG : execute FTP script
------
set ftp:use-site-utime off
set ftp:use-site-utime2 off
set ftp:use-feat off
set ftp:ssl-allow false
open fe80::208:54ff:fe**:****%enp4s0
user root root
cd /mmc/etc/
put /tmp/netceiver.conf -o netceiver.conf
quit
------

Die Version beruht nun auf letztem Commit, der setzt bei "lftp" paar Flags, um nicht unterstützte Kommands auf Serverseite gar nicht erst zu versuchen.

Also hier funktioniert das:

./mcast/tool/netcvupdate -n  -e -i fe80::208:54ff:fe**:**** -d enp4s0 -U netceiver.conf 
INFO  : enable options for FTP client 'lftp'
DEBUG : enable debugging
UUID fe80::208:54ff:fe**:****: Uploading netceiver.conf ... 

DEBUG : execute FTP command: lftp --norc -d 
DEBUG : execute FTP script
------
open fe80::208:54ff:fe**:****%enp4s0
user root root
cd /mmc/etc/
put netceiver.conf -o netceiver.conf
quit
------
...

Kann es sein, daß da das "netcvupdate", was auch einen Patch braucht, irgendwie nicht aktualisiert wurde...

Kannst Du mal die Debug-Logzeilen mit EXEC1 und EXEC2 schicken?

sorry für das recht interaktive "DevOps"...das Option-Handing in netcvupdate ist ja leider etwas sehr angestaubt...habe cam_menu.c grad eben gefixt und die Optionen an den Anfang verschoben.

Quote from gggggg

Deine Frage geht an cinfo

pbrb Kannst du bitte das -e in deinen patch integrieren

-e ist nun aktivierbar, aber für den Inhalt von (vdr-)mcli.conf ist der Endanwender zuständig...

Braucht wohl in etwa folgende Erweiterung:

PLUGIN_OPTIONS="$PLUGIN_OPTIONS --netcvupdate-enable-debug --netcvupdate-use-lftp"

Versucht aktualisierten Diff

https://github.com/pbiering/vd…mpare/master...lftp-fixes

Plugin kennt jetzt 2 neue Optionen:

• --netcvupdate-enable-debug -> fügt Schalter "-e" dran bei netcvupdate Aufrufe
• --netcvupdate-use-lftp -> fügt Schalter "-n" dran bei netcvupdate Aufrufe

Quote from gggggg

pbrb  cinfo Danke,

1 Da nutzt es kein lftp weil -q error kommt:

Feb 6 07:55:04 BM2LTSnativeMC vdr: [1977] EXEC1 rm -f /tmp/netceiver.conf; cd /tmp; netcvupdate -i fe80::208:54ff:fe54:b261 -d eth0.2 -D

Feb 6 07:55:04 BM2LTSnativeMC vdr[1980]: UUID fe80::208:54ff:fe54:b261: Downloading netceiver.conf ...

Feb 6 07:55:04 BM2LTSnativeMC vdr[1982]: ftp: q: unknown option

2 Bitte beim Aufruf in allen Zeilen -e einbauen

netcvupdate -e -i fe80::208:54ff:fe54:b261 -d eth0.2 -D

Display More

hast Du die mcli.conf Plugin-Optionen erweitert mit "--use-lftp" ?

versucht mal folgenden Diff:

https://github.com/pbiering/vd…mpare/master...lftp-fixes

damit hat das Plugin eine neue Option "--use-lftp", die sich weiter-"vererbt" auf die Aufrufe von "netcvupdate". Paar Debug-Statements habe ich auch aktiviert.

es fehlt ev. Ein Blank nach dem -o

Ist doch eins drin... "-o "

das mit dem "%s%s" war Absicht, damit ohne lftp keine 2 Blanks auftauchen (wäre nur kosmetisch...aber soll doch schön sein).

Der Patch ist noch "hübscher":

diff --git a/mcast/tool/netcvupdate.c b/mcast/tool/netcvupdate.c
index 54368d1..ba12783 100644
--- a/mcast/tool/netcvupdate.c
+++ b/mcast/tool/netcvupdate.c
@@ -335,10 +335,10 @@ int do_single_upload( char *uuid, char *device, char *remote_path, char *fname,
                "open %s%%%s\n"
                "user %s %s\n"
                "cd %s\n"
-               "put %s %s\n"
+               "put %s %s%s\n"
 //             "site exec killall -HUP mserv\n"
                "quit",
-               uuid,device,username,password,remote_path,fname,remote_file);
+               uuid,device,username,password,remote_path,fname,((ftp_client_lftp == 1) ? "-o ": ""),remote_file);
        ret=run_ftp("", script, 120,"");
        return ret;
 }

Langsam versteh ich das Problem...würde Euch das helfen?

--- a/mcast/tool/netcvupdate.c
+++ b/mcast/tool/netcvupdate.c
@@ -331,7 +331,17 @@ int do_single_update(char *tmpdir, char *uuid, char *device)
 int do_single_upload( char *uuid, char *device, char *remote_path, char *fname, char *remote_file)
 {
        int ret;
-       sprintf(script,
+       if (ftp_client_lftp == 1) {
+           sprintf(script,
+               "open %s%%%s\n"
+               "user %s %s\n"
+               "cd %s\n"
+               "put %s -o %s\n"
+//             "site exec killall -HUP mserv\n"
+               "quit",
+               uuid,device,username,password,remote_path,fname,remote_file);
+       } else {
+           sprintf(script,
                "open %s%%%s\n"
                "user %s %s\n"
                "cd %s\n"
@@ -339,6 +349,7 @@ int do_single_upload( char *uuid, char *device, char *remote_path, char *fname,
 //             "site exec killall -HUP mserv\n"
                "quit",
                uuid,device,username,password,remote_path,fname,remote_file);
+       } // ftp_client_lftp
        ret=run_ftp("", script, 120,"");
        return ret;
 }

Hilfreich wäre ggf. auch, den kompletten String "c" vor dem "if(SystemExec(c))" mal mit isyslog auszugeben, dann kann man den 1:1 mal per Hand nachstellen.