timerd: Neuer Parameter --config=<file>

[rhabarber1848]

Hi,

da ich gelegentlich neu flashe und auch mal ein Yadd
benutze, vermisse ich die Möglichkeit, die Timer-Liste
auf einer anderen Maschine speichern zu können.

Der Patch in start_neutrino muss noch verändert werden,
damit der Dateiname aus neutrino.conf ausgelesen werden
kann, ebenso fehlt in Neutrino noch die Verzeichnisauswahl.

Hier der Patch: timerd_conf.diff
Als Vorbild habe ich diesen kb2rcd-Patch genommen:
http://cvs.tuxbox-cvs.sourceforge.net/l ... 00035.html

[rhabarber1848]

... und ja, ich könnte auch einen Link anlegen.
Ich bevorzuge allerdings die Lösung mit Parameter.

[mb405]

versteh ich das richtig, das du eine timer config auf verschieden maschinen nutzen willst, wenn du das auf eine mount legst ?
dafür braucht man doch keine extra startoption. wenn cfgfile = "" dann nutze die config der box, ansonsten nutze die angegebene.
das wäre einfacher zu händeln und man bräuchte keine startdatei zu ändern.
was sagste dazu ?

[rhabarber1848]

versteh ich das richtig, das du eine timer config auf verschieden maschinen nutzen willst, wenn du das auf eine mount legst ?

Nein, ich habe nur eine Dbox, die aber längere Zeit mit
Yadd und später mit den unterschiedlichsten selbstkompilierten
Flashimages läuft. Dadurch habe ich schon mehrfach
interessante Sendungen verpasst, für die ich einen
Umschalttimer gesetzt hatte. Mit diesem Patch habe ich
nun immer "meine" Timerliste, egal welches Image gerade
auf der Box läuft.

[Barf]

Ich finde die Option sinnvoll, und rhabarbers Usecase überzeugend.

Eigentlich ist timer.conf nicht eine Konfigurationsdatei, sondern ein "state"-datei, um einige Daten persistent zu speichern. Ablageplatz /var/tuxbox/conf ist deswegen eigenlich ungeignet, /var/run wäre mehr geeignet, aber jemanden hat es nach /tmp (flüchtig!) gelinkt.

Verbesserungsvorschläge:

-- mache config_file_name static und initilisiere direkt, also

static char *config_file_name = CONFIGFILE;

Bitte auch ein einzeichen-Konfigurationsoption.

Der Patch für start_neutrino ist sicherlich nicht für cvs gedacht, right?

Der Patch in start_neutrino muss noch verändert werden,
damit der Dateiname aus neutrino.conf ausgelesen werden
kann, ebenso fehlt in Neutrino noch die Verzeichnisauswahl.

Lass es lieber. Dass Expertenoptionen kein GUI hat, sondern Filefummel erfordert finde ich sogar ein Vorteil.

[rhabarber1848]

Besser so? timerd_conf3.diff

[seife]

OT:

Eigentlich ist timer.conf nicht eine Konfigurationsdatei, sondern ein "state"-datei, um einige Daten persistent zu speichern. Ablageplatz /var/tuxbox/conf ist deswegen eigenlich ungeignet, /var/run wäre mehr geeignet, aber jemanden hat es nach /tmp (flüchtig!) gelinkt.

/var/run wird auf FHS-compliant Maschinen beim booten aufgeräumt, ist also mitnichten persistent. Insofern ist es auf einem embedded system genau richtig, das auf ein tmpfs zu legen, um dem FLASH unnötige Schreibzugriffe zu ersparen

Mit "Expertenoptionen sollten vor Deppen versteckt und nicht im GUI aufgeführt werden" stimme ich dir übrigens 100% zu.

[Barf]

Besser so? timerd_conf3.diff

Fast da. Nur die "static"-Deklaration fehlt, und dadurch entfällt die extern-Deklaration in der Headerfile. (Nicht funktionell notwendig, nur (etablierte) "good manners".)

/var/run wird auf FHS-compliant Maschinen beim booten aufgeräumt, ist also mitnichten persistent. Insofern ist es auf einem embedded system genau richtig, das auf ein tmpfs zu legen, um dem FLASH unnötige Schreibzugriffe zu ersparen

OK. (Freut mich dass FHS nicht nur von mir referenziert wird. ) Der Kern meines Aussage ist aber dass timer.conf eigentlich nicht bei den Konfigurationsfiles gehört, vielleicht in /var/tmp?

[rhabarber1848]

Besser so? timerd_conf3.diff

Fast da. Nur die "static"-Deklaration fehlt, und dadurch entfällt die extern-Deklaration in der Headerfile. (Nicht funktionell notwendig, nur (etablierte) "good manners".)

Wenn ich das mache, wirken sich Änderungen der Variable
in timerd.cpp nicht auf timermanager.cpp aus.

[rhabarber1848]

Der Kern meines Aussage ist aber dass timer.conf eigentlich nicht bei den Konfigurationsfiles gehört, vielleicht in /var/tmp?

/var/tmp wird auch nach /tmp verlinkt, cdk/make/flashroot.mk, Zeile 25.

[rhabarber1848]

committed:
http://article.gmane.org/gmane.comp.vid ... x.scm/1180