[abgeschlossen] Entwurf: IDE Menü

[dbt]

Ok, dann bleibt das drin. Der einzige Unterschied wird nur der sein, dass man bedarfsgerecht immer nur eine Partition formatiert und nicht wie bisher alles auf einmal. In der Regel müsste das meistens eine jungfräuliche Partition sein, dann sind Scripte eigentlich unnötig, es sei denn, man will eine Partition platt machen. Das dürfte aber nicht das Problem sein. Das muss man dann halt mal testen.

[dbt]

[träum]
wenn dbt jetzt noch mit dem ide menü um die ecke kommen würde..
[/träum]

Dann würde ich sagen, machen wir das. Ein paar diverse Image-Tests werd' ich vorraussichtlich am WE rausgeben. Ein paar Sachen haben sich inzwischen bei einigen Tests ergeben, aber da komm ich grad nicht dazu, dass zu überarbeiten, aber für den Anfang sollte es reichen.
Kleine Frage noch wegen Samba und NFS, das ist momentan noch nicht drin, würde es aber in den Netzwerkeinstellungen unterbringen. Darüber wurde eigentlich schon öfter diskutiert, aber ich denke, dass man das so festmachen sollte, natürlich in Abhängigkeit von --enable-ide. Ansonsten wenn es dann noch Ideen gibt, immer her damit.

[rhabarber1848]

Kleine Frage noch wegen Samba und NFS, das ist momentan noch nicht drin

Es geht hier um die Serverversionen dieser Protokolle?

würde es aber in den Netzwerkeinstellungen unterbringen.

Wie wäre es, wenn Du das erstmal beiseite lässt und
ein funktionierendes IDE-Menü zum Testen bereitstellst?
Oder wäre der nachträgliche Einbau der Samba/NFS-
Menüs zu kompliziert?

[Striper]

[träum]
wenn dbt jetzt noch mit dem ide menü um die ecke kommen würde.. ;)
[/träum]

Dann würde ich sagen, machen wir das. Ein paar diverse Image-Tests werd' ich vorraussichtlich am WE rausgeben. Ein paar Sachen haben sich inzwischen bei einigen Tests ergeben, aber da komm ich grad nicht dazu, dass zu überarbeiten, aber für den Anfang sollte es reichen.
Kleine Frage noch wegen Samba und NFS, das ist momentan noch nicht drin, würde es aber in den Netzwerkeinstellungen unterbringen. Darüber wurde eigentlich schon öfter diskutiert, aber ich denke, dass man das so festmachen sollte, natürlich in Abhängigkeit von --enable-ide. Ansonsten wenn es dann noch Ideen gibt, immer her damit.

Freu mich schon auf nen Patch zum testen. :)

[dietmarw]

...
Dann würde ich sagen, machen wir das...

hier der erste test des von dbt überarbeiteten(neu geschriebenen?) ide menü's

http://tuxbox.trale.de/ unter div_tests

vorsichtshalber nicht mit einer wichtigen platte??

[FlatTV]

Welches diff-file hast du benutzt?

cu FlatTV

[dietmarw]

diesen hier

http://www.tuxbox-cvs.sourceforge.net/u ... ry=Patches

wenn das partition.raw icon fehlt, ein anderes kopieren.
ich musste die pfade im patch bei mir noch anpassen, da er den nur zusammenkopiert hat.

[dbt]

Wer testen will, also für alle hier der v1a-Patch
Im configure muss

Code: Alles auswählen

 --enable-ide

und die neue Option

Code: Alles auswählen

--enable-drive-gui 

angegeben sein.
Das Menü ist wie bisher im Servicemenü untergebracht, ist aber wie schon mal angedeutet nur vorab.

vorsichtshalber nicht mit einer wichtigen platte??

Kann ich nur befürworten, auch wenn theoretisch nichts passieren sollte. Auch sonst ist alles noch pre beta und daher relativ unaufgeräumt und recht geschwätzig gehalten, aber es geht.
Baut bitte, wenn möglich die originale fdisk version ein. Für HDD geht die bb-Version, aber für mmc gehts damit defintiv in die Hose.
Wenn es Probleme gibt, wovon ich zu 100% ausgehe , bitte Logs posten, wer Ideen hat, nur zu.
Testsnaps für's JTG dürften noch folgen, aber es sollte erst mal fürs reine CVS passen.
Dann viel Spaß beim Probieren.

Edit:
Linkadresse geändert

[rhabarber1848]

Das fehlende Raw-Icon ist hier:

und jetzt auch im CVS: http://article.gmane.org/gmane.comp.vid ... ox.scm/977

[rhabarber1848]

Wer testen will, also für alle hier der v1-Patch

Hier obiger Patch (ungetestet), angepasst an das aktuelle
CVS und so umgebaut, dass er am Stück eingespielt werden
kann: ide-setup.diff

[tine1084]

Patch getestet und habe festgestellt, dass die Umlaute nicht richtig dargestellt werden.

mfg tine1084

[FlatTV]

Dieser Teil funktioniert so auch nicht, hat aber auch nichts mit dem IDE-Menü zu tun und kann komplett raus, denke ich

Code: Alles auswählen

diff -uNr ../cvs/apps/tuxbox/neutrino/daemons/timerd/timermanager.cpp ./apps/tuxbox/neutrino/daemons/timerd/timermanager.cpp
--- ../cvs/apps/tuxbox/neutrino/daemons/timerd/timermanager.cpp	2009-05-19 19:52:14.000000000 +0200
+++ ./apps/tuxbox/neutrino/daemons/timerd/timermanager.cpp	2009-08-11 12:43:35.000000000 +0200
@@ -859,10 +859,12 @@
 	else
 	{
 		time_t now = time(NULL);
+		
 		while(alarmTime <= now)
 		{
 			time_t diff = 0;
-			struct tm *t= localtime(&alarmTime);
+			struct tm *t;
+			t= localtime(&alarmTime);
 			int isdst1=t->tm_isdst;
 			switch(eventRepeat)
 			{
@@ -900,7 +902,7 @@
 							weekday_arr[4]=((weekdays & 0x8) > 0); //Do
 							weekday_arr[5]=((weekdays & 0x10) > 0); //Fr
 							weekday_arr[6]=((weekdays & 0x20) > 0); //Sa
-							struct tm *t= localtime(&alarmTime);
+							t= localtime(&alarmTime);
 							int day;
 							for(day=1 ; !weekday_arr[(t->tm_wday+day)%7] ; day++){}
 							t->tm_mday+=day;

cu FlatTV

[dbt]

Hier der Patch mit korrigierter locale und entferntem offtopic-Patch v1a-Patch

[FlatTV]

Meiner unmaßgeblichen Meinung nach, kann das so eingecheckt werden.
Das IDE-Menü ist optional und Weiterentwicklungen finden wirksamer im CVS statt, sonst bleibt das schicke Teil wieder auf der Strecke.

cu FlatTV

[tine1084]

Meiner unmaßgeblichen Meinung nach, kann das so eingecheckt werden.
Das IDE-Menü ist optional und Weiterentwicklungen finden wirksamer im CVS statt, sonst bleibt das schicke Teil wieder auf der Strecke.

cu FlatTV

Der Meinung bin ich auch.

Ich konnte bis jetzt keine weiteren fehler finden.
Das Einrichten der Festplatte und die Aufnahmen funktionieren tatellos.

mfg tine1084

[tine1084]

Irgendwie habe ich immer noch probleme mit den umlauten in der deutsch.locale im aktuellen patch.
Bin ich da der einzige wo die umlaute nicht richtig angezeigt werden?

[rhabarber1848]

Hier der Patch mit korrigierter locale und entferntem offtopic-Patch v1a-Patch

Hier ist noch mehr off-topic-Zeug drin, und zwar
bei ./apps/tuxbox/neutrino/src/gui/moviebrowser.cpp
Der Teil des Patches kann komplett entfernt werden
und kollidiert dann nicht mehr mit meinem
--disable-movieplayer-Patch

[rhabarber1848]

Hier noch ein paar Kleinigkeiten, die mir aufgefallen
sind, dbt hat sie bereits per PM von mir erhalten.
Damit möchte ich sagen, dass es für einen CVS
commit noch etwas zu früh ist
=======================================
apps/tuxbox/neutrino/data/icons/Makefile.am
am besten so patchen:

if ENABLE_DRIVE_GUI
install_DATA += \
partition.raw
endif

=======================================
In apps/tuxbox/neutrino/configure.ac nutzt Du
richtigerweise AC_DEFINE(ENABLE_IDE & Co.

Imho müsste dann der komplette Patch von
apps/tuxbox/neutrino/src/Makefile.am
überflüssig sein.

Dann sehe ich, dass Du in neutrino.cpp
#if ENABLE_DRIVE_GUI
nutzt statt
#ifdef ENABLE_DRIVE_GUI

Hängt das zusammen? Mir ist es lieber, ohne
AM_CPPFLAGS zu arbeiten, dann ist das
Kompilierlog übersichtlicher
=======================================
In cdk/configure deaktivierst Du das HDD-Menü, wenn
IDE inaktiv ist. Läuft das HDD-Menü nicht auf einem
MMC-only-System?
Wäre es nicht besser, enable_drive_gui in die Test-
funktion bei Zeile 256 packen?

Code: Alles auswählen

# if no disc filesystem is activated, deactivate IDE and MMC support

[rhabarber1848]

Soeben aufgefallen, ./cdk/make/buildenv.mk

Code: Alles auswählen

CONFIGURE_OPTS_KERNEL26 =

und

Code: Alles auswählen

CONFIGURE_OPTS_MMC =

bitte in

Code: Alles auswählen

CONFIGURE_OPTS +=

ändern, sonst merkt Neutrino nichts davon.

[rhabarber1848]

Ich verstehe den Sinn dieses Patches für cdk/root/etc/init.d/start nicht:

Code: Alles auswählen

-    exec /etc/init.d/start_neutrino
+       if [ -x /var/etc/init.d/start_neutrino ]; then
+               exec /var/etc/init.d/start_neutrino
+       else
+               exec /etc/init.d/start_neutrino
+       fi

In einem CVS-Image wird start_neutrino & Co. niemals in
/var/etc liegen, deshalb muss es auch nicht berücksichtigt
werden. Wenn ein Image-Ersteller das anders macht, dann
muss er oder sie dies mit eigenen Patches berücksichtigen.

[rhabarber1848]

Der Patch in ./cdk/make/distribution-lists.mk

Code: Alles auswählen

+GET_MD5SUM = `GET_MD5SUM -b <$$f |sed -e s/\*-//`

kann so auch nicht funktionieren, da GET_MD5SUM nirgendwo definiert wird.
Außerdem ist er off-topic

[seife]

Ich verstehe den Sinn dieses Patches für cdk/root/etc/init.d/start nicht:

Code: Alles auswählen

-    exec /etc/init.d/start_neutrino
+       if [ -x /var/etc/init.d/start_neutrino ]; then
+               exec /var/etc/init.d/start_neutrino
+       else
+               exec /etc/init.d/start_neutrino
+       fi

In einem CVS-Image wird start_neutrino & Co. niemals in
/var/etc liegen, deshalb muss es auch nicht berücksichtigt
werden. Wenn ein Image-Ersteller das anders macht, dann
muss er oder sie dies mit eigenen Patches berücksichtigen.

Sowas habe ich bei mir auch drin: ich kann start_neutrino nach /var/etc/init.d kopieren und anpassen und somit Änderungen testen, ohne dazu neu bauen / flashen zu müssen.

Ob das ins CVS muss, ist eine andere Frage. Wegen mir nicht

[rhabarber1848]

ich kann start_neutrino nach /var/etc/init.d kopieren und anpassen und somit Änderungen testen, ohne dazu neu bauen / flashen zu müssen.

Sorry, ich habe mich vorhin ungenau ausgedrückt.
Es hätte heißen müssen:

In einem CVS-Squashfs-Image wird start_neutrino & Co. niemals in /etc liegen

sondern sie liegen in /var/etc, zudem existieren Links von
/etc/init.d/start_* nach /var/etc/init.d/start_*.
In JFFS2-only-Images liegen die Skripts in /etc, was auch Sinn macht.
Besagter Patch ist damit dennoch überflüssig

[dbt]

Diverse (unnötige) Sachen sind momentan auf jeden Fall nur probehalber drin. Das muss man natürlich entsprechend ändern und Hinweise darauf sind natürlich nur recht. Einiges davon ist auch nur nebenher vom Anhänger gefallen und mach ich beim nächtsen mal weg oder überarbeite das. Dazu ist die Sache zum testen ja da und vom CVS ist das noch ein ganzes Stück weg. Um das mal klarszustellen. Mir sind da sowieso noch einige Sachen eingefallen, die man ändern muss oder sogar fehlen. Ein Größerer Feldtest mit JTG steht übrigens noch aus. Evtl. kann Riker das demnächst mal in Angriff nehmen.
Interessieren tut vorallem ob das ganze in allen Distriformen funktioniert (Yadd, jffs2, squshfs etc...). Vorallem in Bezug auf das fstab-Handling. Das Dilemma ist halt das man in /etc nicht immer schreiben kann. Bin mir nicht sicher, ob die Lösung so ok ist. Sollte es bessere Ideen geben, dann wäre das natürlich super.

[rhabarber1848]

Vorallem in Bezug auf das fstab-Handling. Das Dilemma ist halt das man in /etc nicht immer schreiben kann.

Vielleicht wäre es eine gute Idee, wenn vor dem CVS commit
des IDE-Menü-Patches fstab analog zu start_neutrino behandelt
wird, d.h. in Squashfs-Images in /var/etc zu finden und
/etc/fstab in Link dorthin ist. Dann wäre auch ein Teil des
Tuxbox-Busybox-Patches überflüssig.
Wenn das sinnvoll ist, könnte ich mich daran mal versuchen.

EDIT: Geht glaube ich nicht, ohne /etc/fstab kann /var in
einem Squashfs-Image nicht gemountet werden...