Vorschlag für neuen Patch: --with-features=...

[tomworld]

wieso nicht, es muss aber Dokumentiert werden

[cs4711]

--disable-rtc enable rtc hardware support - dbox2-only

dachte immer disable =nicht vorhanden

Code: Alles auswählen

+AC_ARG_ENABLE(rtc,
+	AS_HELP_STRING(--disable-rtc,enable rtc hardware support - dbox2-only),
+	[case "${enableval}" in
+		yes) 	if ! test "$BOXTYPE" = "dbox2"; then
+				enable_rtc=no
+			fi ;;
+	esac],[	if test "$BOXTYPE" = "dbox2"; then
+			enable_rtc=yes
+		else
+			enable_rtc=no
+		fi])
+AM_CONDITIONAL(ENABLE_RTC,test "$enable_rtc" = "yes")

[tomworld]

stimmt, ich habe hwrtc für die Echtzeituhr im Image auch nicht angefordet, es wird jetzt pauschal für alle Image mitgebaut oder eingefügt

liegt es daran, das es unter "Treiber- und Bootoptionen" als Auswahl steht

[cs4711]

Schon klar, aber wäre es andersrum nicht besser ??
Ist ja zusätzliche HW wie IDE oder MMC

[tomworld]

es sollte nur bei HDD-IF aufgerufen werden, da es bei diesen Orderbar ist bei Bestellung

reine Echtzeituhr, ohne dies in Frage zu stellen, baut kein Anwender mehr ein, das zeigen mir meine Erfahrungen
ohne dies in Frage zu stellen, kann sich jeder im Tuxbox-Wiki dort einlesen und bei bedarf anwenden

ich selbst nutze ein HDD-IF mit Echtzeituhr, ohne diese anzuwenden

[Barf]

Wie gefällt Euch das? with-features2.diff

Gefällt grundsächlich. Ich habe ein Paar kleinere Bemerkingen/Verbesserungsvorschläge:

--enable-cdkvcinfo

Es ist problematisch, dass deine kleinschreibene Form von dem Programmname abweicht. Sollten wir vielleicht nicht das Program 'cdkVcInfo' endgültig in 'cdkvcinfo' umbenennen? (Ja, die Themen mit cvs-history sind mir bekannt. Ja, ich habe selbst das erste Einchecken gemacht... )

--enable-input enable input

Hoch missverständlich. Vorschlag: "enable the program named `input'"

--enable-lirc include lirc in yadds and images - dbox2-only

Hier dachte ich erstmals, es wäre dumm, Besitzer von andere Boxen das Lirc-en zu verbieten. Es wäre durchaus machbar, sogar u.U. sinnvoll, lirc zu konfigurieren, so dass Lirc-Befehle zu einem "entfernten" Lirc-Server gesendet wird. Fakt ist aber, dass die Konfiguration dbox2-spezifisch ist, so dass (z.Z.!) --enable-lirc nur bei dboxen Sinn ergibt. (Nur Bemerkung, kein Änderungsvorschlag.)

--enable-openvpn include OpenVPN in yadds and images and build tun
kernel module

Ist "tun" ein Tippfehler oder nur ein Verständnissproblem bei mir?

--enable-procps

Das Autoconfhandbuch sagt:

These options allow users to choose which optional features to build and install. --enable-feature options should never make a feature behave differently or cause one feature to replace another. They should only cause parts of the program to be built rather than left out.

Ich schlage deswegen vor --with-procps anstatt zu benutzen.

Code: Alles auswählen

+       [case "${enableval}" in
+               yes)    if ! test "$BOXTYPE" = "dbox2"; then
+                               enable_lirc=no
+                       fi ;;

Ich finde ein Abbruch wäre besser, als eine Eingabe des Benutzers leise zu ignorieren. (Betrifft auch andere Teile.)

Nach dieser Änderungen sehe ich kein Problem, das Ding einzuchecken.

[GetAway]

--enable-cdkvcinfo

Es ist problematisch, dass deine kleinschreibene Form von dem Programmname abweicht. Sollten wir vielleicht nicht das Program 'cdkVcInfo' endgültig in 'cdkvcinfo' umbenennen? (Ja, die Themen mit cvs-history sind mir bekannt. Ja, ich habe selbst das erste Einchecken gemacht... )

Etwas OT, aber ich möchte hier darauf hinweisen das möglicherweise das Tool "cdkVcInfo",
gegen das erweiterte Tool sbimgeinfo von SnowHead ausgetauscht werden sollte, sofern
dieses Tool auch zu den freigegeben Sourcen zählt, die an rhabarber1848 übergeben wurden.
Zu den normalen Imageinfo's zeigt dieses, während des Bootvorgang's, die Netzwerkeinstellungen
mit an. Ich finde es persönlich sehr nützlich.

[rhabarber1848]

--disable-rtc enable rtc hardware support - dbox2-only

dachte immer disable =nicht vorhanden

Hast Recht, ist ein Fehler in der Optionsbeschreibung, korrigiert...

stimmt, ich habe hwrtc für die Echtzeituhr im Image auch nicht angefordet, es wird jetzt pauschal für alle Image mitgebaut oder eingefügt

In einem ungepatchten CVS-Image sind /bin/hwrtc und
/lib/modules/2.4.37.7-dbox2/misc/ds1307.o immer vorhanden.
Mit der neuen Option --disable-rtc werden die Dateien nicht
mehr ins Image gepackt, was Du siehst, ist also nicht neues

es sollte nur bei HDD-IF aufgerufen werden, da es bei diesen Orderbar ist bei Bestellung

Wenn ich die Anleitung im Wiki richtig verstehe, ist kein
IDE-Interface zum Einbau einer RTC in der Dbox2 nötig.

[rhabarber1848]

--enable-cdkvcinfo

Es ist problematisch, dass deine kleinschreibene Form von dem Programmname abweicht.

Hast Recht, war ein Übertragungsfehler, der zudem die Rückwärtskompatibilität bricht, korrigiert...
@GetAway: Ich habe SnowHead bzgl. sbimgeinfo schon gefragt, er hat auch
nur das binary, aber ich habe auf einem bösen Board Sourcecode entdeckt,
von dem er der Meinung ist, dass er zu sbimgeinfo passt.
Das schaue ich mir irgendwann nochmal an
@Barf: Deshalb kümmere ich mich erstmal nicht um die Umbenennung von cdkVcInfo.

--enable-input enable input

Hoch missverständlich. Vorschlag: "enable the program named `input'"

Schon klar, ich habe die Beschreibungen bisher nur kopiert, da war noch
keine Kosmetik dran, korrigiert...

--enable-openvpn include OpenVPN in yadds and images and build tun
kernel module

Ist "tun" ein Tippfehler oder nur ein Verständnissproblem bei mir?

tun.o oder tun.ko existieren und stellen /dev/tun zur Verfügung.

Ich schlage deswegen vor --with-procps anstatt zu benutzen.

Erledigt.

Ich finde ein Abbruch wäre besser, als eine Eingabe des Benutzers leise zu ignorieren. (Betrifft auch andere Teile.)

Ich habe mich aus praktischen Gründen für meine Lösung entschieden. Die Liste der
Optionen in cdk/configure wird ja nicht kürzer Da ich häufiger für unterschiedliche
Boxtypen kompiliere, füge ich die diesbzgl. Optionen für Boxtyp/-modell in mein Skript
ein und möchte dabei nicht auch noch einen Haufen anderer Optionen mit ändern müssen,
nur weil diese für die jeweilige Box keine Bedeutung haben. Deshalb wird z.B. --enable-lirc
im nicht-Dbox2-Fall einfach ignoriert.

Ich teste den aktualisierten Patch noch ein wenig, bevor ich ihn hochlade.

[rhabarber1848]

Wenn ich tuxtxt in der "--with-features=" Liste weg lasse wird es trotzdem in der configure Ausgabe angezeigt:

Code: Alles auswählen

Internal Tuxtxt support:        yes

Erledigt.

[Striper]

Darf man schon testen? :)

[dietmarw]

...
Deshalb wird z.B. --enable-lirc
im nicht-Dbox2-Fall einfach ignoriert.
...

evtl. in der configure ausgabe dann sowas wie

Code: Alles auswählen

LIRC:        Boxtype not supported

[rhabarber1848]

LIRC: Boxtype not supported

erledigt, hier die aktuelle Version zum Testen: EDIT: Patch ist im CVS

[Striper]

Irgendwas stimmt nicht mit cdkVcInfo. Da wird nix ausgegeben wenn ich --enable-cdkVcInfo setze.

/edit

Habs gefunden einfach die verbliebenen 3x cdkvcinfo in cdkVcInfo umbenennen. ;)

[seife]

Sollten wir vielleicht nicht das Program 'cdkVcInfo' endgültig in 'cdkvcinfo' umbenennen? (Ja, die Themen mit cvs-history sind mir bekannt. Ja, ich habe selbst das erste Einchecken gemacht... )

Code: Alles auswählen

diff --git a/tuxbox/tools/cdkVcInfo/Makefile.am b/tuxbox/tools/cdkVcInfo/Makefile.am
index 2d544b6..d08070c 100644
--- a/tuxbox/tools/cdkVcInfo/Makefile.am
+++ b/tuxbox/tools/cdkVcInfo/Makefile.am
@@ -1,3 +1,3 @@
-bin_PROGRAMS = cdkVcInfo
+bin_PROGRAMS = cdkvcinfo

-cdkVcInfo_SOURCES = cdkVcInfo.cpp
+cdkvcinfo_SOURCES = cdkVcInfo.cpp

Ich sehe da kein Problem mit der CVS History

[Striper]

Pure Kosmetik, aber wenn du schon mal dran bis rhabarber...

Evtl. sollte man bei "fstab default fs" nicht Nichts ausgeben sondern analog zu "ucode directory" "NONE"?

[rhabarber1848]

-bin_PROGRAMS = cdkVcInfo
+bin_PROGRAMS = cdkvcinfo

cdk/root/etc/init.d/25cdkVcInfo nicht vergessen

[rhabarber1848]

Evtl. sollte man bei "fstab default fs" nicht Nichts ausgeben sondern analog zu "ucode directory" "NONE"?

Die in cdk/configure.ac angezeigte Variable $DEFAULT_FS_FSTAB wird
direkt in /etc/fstab übernommen, "NONE" passt dort nicht wirklich

[rhabarber1848]

Irgendwas stimmt nicht mit cdkVcInfo.

Danke für den Hinweis, Patch aktualisiert.

[Striper]

da man procps mit --with-procps aktiviert würde ich es lieber oben bei ide und dem ganzen Krempel sehen.

[Striper]

So, hat gebaut und läuft im Flash. Keine Auffälligkeiten. Hab auch ein wenig mit den Optionen gespielt, da scheint alles zu passen.

Well done!

Code: Alles auswählen

sh configure --prefix=$HOME/MY_Image/dbox2 \
	--with-cvsdir=$HOME/MY_Image/tuxbox_cvs \
	--with-customizationsdir=$HOME/MY_Image/MY_CVS/imagefiles \
	--with-archivedir=$HOME/MY_Image/Archive \
	--with-checkImage=rename \
	--with-updatehttpprefix=disable \
	--with-rootpartitionsize=0x440000 \
	--with-procps \
	--enable-maintainer-mode \
	--enable-flashrules \
	--enable-cdkVcInfo \
	--enable-movieplayer2 \
	--enable-ccache \
	--enable-dboxshot \
	--enable-dvbsnoop \
	--enable-fbshot \
	--enable-lcshot \
	--enable-sysinfo \
	--enable-tuxwetter \
	--disable-radiotext \
	--disable-fx2plugins \
	--disable-rtc \
	--disable-tuxcom \
	--disable-tuxmail \
	--disable-dvbsub

[rhabarber1848]

Fehlen noch Optionen? Mir fallen spontan noch aformat, strace und gdbserver ein.

[rhabarber1848]

da man procps mit --with-procps aktiviert würde ich es lieber oben bei ide und dem ganzen Krempel sehen.

Was haben procps und IDE miteinander zu tun? Imho soll configure eine
thematisch sortierte Liste ausgeben, nicht nach --with und --enable sortiert.

[Striper]

Fehlen noch Optionen? Mir fallen spontan noch aformat und strace ein.

Ja stimmt, aber die sind mir jetzt gar nicht abgegangen muss ich gestehen. ^^

... nicht nach --with und --enable sortiert.

So wärs für mich jetzt der Auflistung nach schlüssiger gewesen. Aber egal, das passt auch so.

[rhabarber1848]

Fehlen noch Optionen? Mir fallen spontan noch aformat, strace und gdbserver ein.

Erledigt: EDIT: Patch ist im CVS