[PATCH] Neue Option Stand-by nach Boot

[rhabarber1848]

Hi,

hier ein Patch, der eine neue Bootoption für Neutrino ermöglicht.
Wenn aktiviert, geht die Box nach durchgeführtem Bootvorgang
automatisch in den Stand-by.

EDIT 10.09.2008: Patch aktualisiert, geänderter yhttpd-Port wird berücksichtigt
Patch: EDIT: Patch entfernt

[dbt]

Ist zwar ok aber evtl. etwas paradox. Die Box geht doch dann immer ins Standby solange man die Option nicht wieder abschaltet oder sehe ich das falsch. Sinnvoller wäre es, wenn sich die Box selbst merkt in welchem Modus sie war als man sie ausgeschaltet hat und sollte die mal vom Netz getrennt worden sein, dann auch wieder in den gleichen Zustand kommen wie er vorher war (z.B: wenn mal der Strom weg war). So ist das glaube ich auch bei der ollen Betanova. Korrigiere mich bitte, falls ich da was falsch verstehe.

[rhabarber1848]

Die Box geht doch dann immer ins Standby solange man die Option nicht wieder abschaltet

Richtig.

Sinnvoller wäre es, wenn sich die Box selbst merkt in welchem Modus sie war als man sie ausgeschaltet hat und sollte die mal vom Netz getrennt worden sein, dann auch wieder in den gleichen Zustand kommen wie er vorher war

Das wäre eine zweite Option, die neben der o.g. implementiert werden könnte.
Ich persönlich bevorzuge den ersten Weg, es spricht aber nichts dagegen, auch
Deinen Vorschlag einzubauen. Nur gibt es AFAIK mehrere Wege, den Standby-
Status der Box zu verändern (Fernbedienung/Neutrino, yWeb, saa -o, noch mehr?).
Bei jeder dieser Veränderungen müsste eine Flag-Datei geschrieben werden.
Machbar ist das ohne Frage, nur müsste sichergestellt sein, dass alle
o.g. Prozeduren diese Flag-Datei entsprechend erstellen/löschen.

So ist das glaube ich auch bei der ollen Betanova.

Habe ich nie benutzt, meine Box hatte ein JTG-Image drauf, als ich sie gekauft habe

[mb405]

sehr gefährlich die option.
stell mal im diverse einstellungen->allgemein->standbymodus auf aus. dann fährt die runter.
oder täuche ich mich da ?

[rhabarber1848]

stell mal im diverse einstellungen->allgemein->standbymodus auf aus. dann fährt die runter.
oder täuche ich mich da ?

Du hast Recht! Ich habe den Patch aktualisiert:
im nhttpd gibt es für control/standby? nun eine neue Option soft, die
nur den Soft-Standby einschalten soll.

Wenn "diverse einstellungen->allgemein->standbymodus auf aus" steht,
dann bewirkt control/standby?soft nichts.

Ein kurzer Test hier mit yadd-neutrino funktioniert.

[Tommy]

Du hast Recht! Ich habe den Patch aktualisiert:
im nhttpd gibt es für control/standby? nun eine neue Option soft, die
nur den Soft-Standby einschalten soll.

control/standby?on stand doch schon immer für den "Soft" Standby ansonnsten wäre ja control/standby?off at absurdum geführt?!

[rhabarber1848]

sehr gefährlich die option.
stell mal im diverse einstellungen->allgemein->standbymodus auf aus. dann fährt die runter.
oder täuche ich mich da ?

So, nachdem ist das Thema nochmals getestet habe, kann ich folgendes feststellen:

Code: Alles auswählen

wget -Y off -q -O /dev/null http://localhost/control/standby?on

bringt die Box in den Soft-Standby, unabhängig davon, wie die Option Standby-Modus
gesetzt ist, getestet mit einem ungepatchten yadd-neutrino.

[rhabarber1848]

Patch aktualisiert und mit Standbymodus ein/aus getestet, die Box
wird immer in den Soft-Standby versetzt, wenn die neue Option
"Aktiviere Soft-standby nach Boot" aktiviert ist.

[joe123456]

Hi,
kann mir jemand verraten wo und wie ich die Datei aktiviere? Bin hier noch neu und kenn mich noch nicht so aus.

Gruß Joe

[rhabarber1848]

Kleines Ping zum Hochholen des Threads:

http://forum.tuxbox-cvs.sourceforge.net ... 93#p360693

bis auf den "starte im standby"-Patch habe ich alles drin. Der besticht zwar durch seine Einfachheit, allerdings finde ich das mit dem "sleep;wget" etwas unelegant, drum überlege ich noch, ob und wie man das direkt im neutrino machen könnte.

Der Patch ist aber noch nicht CVS-reif, da die Reihenfolge in locals.h nicht stimmt

[seife]

Inzwischen würde ich das einfach in der init.end machen, das spart das sleep etc.

[rhabarber1848]

Gute Idee, ich setze mich dran.

[rhabarber1848]

UNGETESTET, wie wäre es mit diesem Code in neutrino.cpp?
EDIT: Patch entfernt

Ich habe keine Ahnung von C und dem Neutrino-Message-System,
aber als Idee wäre das doch nicht schlecht. Kompilieren werde ich
das aber erst morgen, gute Nacht.

[Barf]

Neutrinos Verhalten bei Startup ist verbesserungsbedürftig. So gibt es keine (saubere) Möglichkeit, das Programm in einem bestimmten, von Benutzer spezifizierte Zustand (z.B. TV/Radio/Internet Radio/SCART/... oder "letzte") zu bringen. (Etwas kann aber durch händische Befummeln von zapit.conf erreicht werden.) Auch der Zustand des SCART-Eingangs wird ignoriert (siehe thread) so dass der dBox von SCART aufgeweckt werden kann, aber trotzdem nicht in SCART mode geht.

Der vorliegende Patch "fixt" eine kleine Teilmenge von diesen Anforderungen. Dafür wird noch eine, von wenige benutzt, Konfigurationsoption eingeführt, und Neutrino noch größer.

Dazu kommt, dass die Option ein gewisse Werwirrpotential für unerfahrene Benutzer besitzt, die versehlich sie einschaltet, und -- der Tag danach!! -- erfahren dass die Box "nicht funktioniert".

Ausserdem gibt es ein Workaround: der Benutzer hat die Möglichkeit, entsprechende Befehl "wget ... standby?on" in init.end zu schreiben.

Ich bin deswegen der Meinung, dass der Patch nicht committed werden soll.

[seife]

Tatsächlich, wenn ich mir den Code anschaue, denke ich, dass das noch zu früh ist, da die eigntliche Applikation mit RealRun(mainMenu) gestartet wird und ich nicht weiss was passiert, wenn du vorher schon was in die rcinput-queue schiebst.
Allerdings sollte an dieser stelle schon ein "standbyMode(true)" möglich sein.
Wenn es im Neutrino gemacht wird, ist ein Markerfile auch unnütz, dann kann man es auch einfach in die Configdatei schreiben, die wird eh gelesen (die Markerfiles sind ja nur böse hacks, damit die init-Skripten keine Configfiles parsen müssen).

Generell stimme ich Barf zu, allerdings kenne ich z.B auch Leute, die neutrino so gepatched haben, dass es immer mit dem audioplayer startet und ähnliches, eine generische "startmodus"-Einstellung, die halt _auch_ "standby" sein kann, wäre evtl. also nicht unnütz.
Andererseits kann man das auch in der init.end mit ein paar "rcsim"-Aufrufen zusammenwursteln

[rhabarber1848]

Auch der Zustand des SCART-Eingangs wird ignoriert (siehe thread) so dass der dBox von SCART aufgeweckt werden kann, aber trotzdem nicht in SCART mode geht.

Das ist im CVS mittlerweile behoben.

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

Inspiriert von obigem Scart-Patch habe ich das hier zusammengebastelt:
EDIT: Patch ist im CVS

Damit kann ausgewählt, ob Neutrino nach dem Start im
TV-/Radio-/Audioplayer-/Esound-/oder Standby-Modus ist.
Internetradio habe ich auch in der Liste drin, nur weiß ich nicht,
wie ich das starten kann. Jemand eine Idee?
Esound wird hier immer angezeigt, egal, ob es im Image
vorhanden ist oder nicht, das müsste noch angepasst werden.

Alle bisher eingebauten Funktionen habe ich positiv getestet.

[dbt]

Damit kann ausgewählt, ob Neutrino nach dem Start im
TV-/Radio-/Audioplayer-/Esound-/oder Standby-Modus ist.
Internetradio habe ich auch in der Liste drin, nur weiß ich nicht,
wie ich das starten kann. Jemand eine Idee?

Auf den ersten Blick würde ich sagen, dass du der Klasse den Parameter(true) mitgeben musst.

Code: Alles auswählen

CAudioPlayerGui(true)

edit:
PS:
für sowas wie hier:
else if(g_settings.startmode == 4)
wären vernünftig bezeichnete enums mal nicht schlecht, davon gäbe es jede Menge. Blanke Zahlen sind irgendwie nichtssagend (sofern man nicht 100% weiß was die bedeuten).

[rhabarber1848]

Also im Prinzip genauso wie den Audioplayer, nur eben mit diesem Parameter. hier mal ins blaue getippt

Habe ich schon probiert, klappt nicht:

neutrino.cpp: In member function `void CNeutrinoApp::RealRun(CMenuWidget&)':
neutrino.cpp:2355: error: expected `;' before "tmpAudioPlayerGui"
neutrino.cpp:2356: error: `tmpAudioPlayerGui' was not declared in this scope

[rhabarber1848]

für sowas wie hier:
else if(g_settings.startmode == 4)
wären vernünftig bezeichnete enums mal nicht schlecht

Patch aktualisiert, besser so? Ist eine enum -1 nötig?

[dbt]

Der Tipp ins blaue war quark
-1 Sollte nichts ausmachen, schaden tuts sicher nichts. Könnte man STARTMODE_UNKNOWN betiteln!
Enums würde ich aber wie Konstanten mit Großbuchstaben versehen, das ist leider hier nicht immer so gemacht worden. Ist zwar keine Pflicht, aber ist eigentlich üblicher.

Edit: link entfernt

[rhabarber1848]

Esound wird hier immer angezeigt, egal, ob es im Image
vorhanden ist oder nicht, das müsste noch angepasst werden.

Neutrino kennt jetzt #define ENABLE_ESD und zeigt dementsprechend
den Esound-Menüpunkt bei der Startmodus-Auswahl nicht mehr an.

Der komplette Patch ist, mit den Ergänzungen von dbt, im CVS:
http://article.gmane.org/gmane.comp.vid ... ox.scm/853
http://article.gmane.org/gmane.comp.vid ... ox.scm/854
http://article.gmane.org/gmane.comp.vid ... ox.scm/855
http://article.gmane.org/gmane.comp.vid ... ox.scm/856

[dbt]

Fehlt da nicht was, um den bisherigen Modus wie vor dem letzen runterfahren zu haben? So wird das doch gnadenlos festgenagelt. Das dürfte zwar im ersten Moment nicht gleich auffallen, da die meißten sowieso im TV-Mode sind, aber wenn der gewisse Fall eintritt, gibts Haue.
@rhabarber1848
Glückwunsch zum 2000en

[rhabarber1848]

Fehlt da nicht was, um den bisherigen Modus wie vor dem letzen runterfahren zu haben?

Afaik startet Neutrino bisher immer im TV-Modus.

aber wenn der gewisse Fall eintritt, gibts Haue.

Du sprichst in Rätseln

[dbt]

Naja, wenn ich TV- schaue und irgendwann auf die Idee komme Radio zu hören schaltet man doch in den Radiomodus. Schalte ich die Box ab und dann irgendwann wieder ein zwecks "ich will weiter Radio hören" ist das Teil im TV-Mode. Das war bisher nicht so denke ich.
Edit:
Ich würde so'ne Art Option STARTMODE_LASTMODE als default Option mit reinnehmen , die im Prinzip garnichts bewirkt und den bisherigen Stand weiter am Leben hält.

[rhabarber1848]

Option STARTMODE_LASTMODE

Bin schon dabei