busybox-1.8.x

Ich habe hier noch ein diff für diesen Zeilenumbruch bei df.

diff -Naur busybox-1.8.2/coreutils/df.c busybox-1.8.2-tuxbox/coreutils/df.c_patch
--- busybox-1.8.2/coreutils/df.c	2007-11-10 02:40:51.000000000 +0100
+++ busybox-1.8.2/coreutils/df.c_patch	2007-12-19 15:10:26.000000000 +0100
@@ -124,7 +124,7 @@
 				}
 			}
 
-			if (printf("\n%-20s" + 1, device) > 20)
+			if (printf("\n%-20s" + 1, device) > 50)
 				    printf("\n%-20s", "");
 #if ENABLE_FEATURE_HUMAN_READABLE
 			printf(" %9s ",

Scheinbar glauben die BB Jungs nicht, dass es auch mal Devicenamen gibt die mehr als 20 Zeichen haben.
50 sollte für die Box reichen...

Es sollte reichen wenn man das Diff an das busybox.diff anhängt.

Gruß

Beitrag von **mb405** » Mittwoch 19. Dezember 2007, 19:24

klasse

ich hätt mir nen wolf gesucht warum de zeilenumbruch da war

Auch wenn es nervt ! Man sollte ja beide seiten beachten Neutrino & Enigma ! Denn bei Enigma läßt sich in verbindung der neueren Busyboxen nichts mehr mounten per Menü ( cifs etc )! Per Console kein Thema aber nicht per Enigma Mount Menü ! Wie auch schon in den anderen Bus Treads angesprochen ! Hat eventuell jemand mal getestet bzw. hat nen Lösungvorschlag ? Im Neutrino soll es ja gehen aber leider nicht Unter E1 !

ich kenn enigma nicht, aber der slash muss nur umgestellt werden
ich hab mir mal enigma angeguggt
enigma_mount.cpp ab zeile 168 dürfte das sein.

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
						{
							cmd = "mount -t cifs //";
							cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o user=";
							cmd += (mp.userName) ? mp.userName : "anonymous";
							cmd += (mp.password) ? (",pass=" + mp.password) : "";
							cmd += ",unc=//" + ip + "/" + mp.mountDir;
							cmd += (mp.options) ? ("," + mp.options) : "";
						}

bei neutrino hab ich das so

Code: Alles auswählen

else if(fstype == CIFS)
	{
		cmd = "mount -t cifs \\";
		cmd += ip;
		cmd += '/';
		cmd += dir;
		cmd += ' ';
		cmd += local_dir;
		cmd += " -o username=";
		cmd += username;
		cmd += ",password=";
		cmd += password;
		cmd += ",unc=//";
		cmd += ip;
		cmd += '/';
		cmd += dir;
		cmd += ',';
		cmd += options1;
	}

enigma entdröselt

sollte es so gehen. mal testen bitte, dann kann das wer einchecken.

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
						{
							cmd = "mount -t cifs \\";
							cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o user=";
							cmd += (mp.userName) ? mp.userName : "anonymous";
							cmd += (mp.password) ? (",pass=" + mp.password) : "";
							cmd += ",unc=//" + ip + "/" + mp.mountDir;
							cmd += (mp.options) ? ("," + mp.options) : "";
						}

Ich danke dir ! Werde es heue abend mal antesten und bescheid geben !

mb405 hat geschrieben:ich kenn enigma nicht, aber der slash muss nur umgestellt werden
ich hab mir mal enigma angeguggt
enigma_mount.cpp ab zeile 168 dürfte das sein.

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
						{
							cmd = "mount -t cifs //";
							cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o user=";
							cmd += (mp.userName) ? mp.userName : "anonymous";
							cmd += (mp.password) ? (",pass=" + mp.password) : "";
							cmd += ",unc=//" + ip + "/" + mp.mountDir;
							cmd += (mp.options) ? ("," + mp.options) : "";
						}

bei neutrino hab ich das so

Code: Alles auswählen

else if(fstype == CIFS)
	{
		cmd = "mount -t cifs \";
		cmd += ip;
		cmd += '/';
		cmd += dir;
		cmd += ' ';
		cmd += local_dir;
		cmd += " -o username=";
		cmd += username;
		cmd += ",password=";
		cmd += password;
		cmd += ",unc=//";
		cmd += ip;
		cmd += '/';
		cmd += dir;
		cmd += ',';
		cmd += options1;
	}

enigma entdröselt

sollte es so gehen. mal testen bitte, dann kann das wer einchecken.

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
						{
							cmd = "mount -t cifs \";
							cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o user=";
							cmd += (mp.userName) ? mp.userName : "anonymous";
							cmd += (mp.password) ? (",pass=" + mp.password) : "";
							cmd += ",unc=//" + ip + "/" + mp.mountDir;
							cmd += (mp.options) ? ("," + mp.options) : "";
						}

Ne leider geht es damit noch nicht !

Hi

Versuch das mal so:

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
                  {
                     cmd = "mount -t cifs \\";
                     cmd += ip + "\" + mp.mountDir + " " + mp.localDir + " -o username=";
                     cmd += (mp.userName) ? mp.userName : "anonymous";
                     cmd += (mp.password) ? (",password=" + mp.password) : "";
                     cmd += ",unc=//" + ip + "/" + mp.mountDir;
                     cmd += (mp.options) ? ("," + mp.options) : "";
                  }

oder so:

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
                  {
                     cmd = "mount -t cifs //";
                     cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o username=";
                     cmd += (mp.userName) ? mp.userName : "anonymous";
                     cmd += (mp.password) ? (",password=" + mp.password) : "";
                     cmd += ",unc=//" + ip + "/" + mp.mountDir;
                     cmd += (mp.options) ? ("," + mp.options) : "";
                  }

In Neutrino heisst es nämlich username und password nicht user und pass.
Kann bei Enigma eventuell daran liegen.

Gruß

Ne leider auch nicht ! Mist ! Siehe unten. Der unterschied sind die beiden // !

Busy 1.2.1
------------
mount -t cifs -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream //192.168.1.1/Mp3-Stream /mnt/hdd/

Busy 1.8.2
------------
mount -t cifs -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream 192.168.1.1/Mp3-Stream /mnt/hdd/

Beitrag von **flasher** » Freitag 21. Dezember 2007, 11:37

Hi

Der Sourcecode macht ja nichts anderes als einzelne Values zu einem String zusammen zu backen.
Wenn jetzt in der BB 1.8.2 das // zu viel ist, dann brauchst Du das doch eigentlich nur aus dem Source zu nehmen.
Die Zeile zerhackt würde dann so aussehen:

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
                  {
                     cmd = "mount -t cifs ";
                     cmd += ip + "/" + mp.mountDir + " " + mp.localDir + " -o user=";
                     cmd += (mp.userName) ? mp.userName : "anonymous";
                     cmd += (mp.password) ? (",password=" + mp.password) : "";
                     cmd += ",unc=//" + ip + "/" + mp.mountDir;
                     cmd += (mp.options) ? ("," + mp.options) : "";
                  }

Das müsste dann so aussehe:

mount -t cifs 192.168.1.1/Mp3-Stream /mnt/hdd/ -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream

Und hier so wie in Deinem Post...

Code: Alles auswählen

if (fileSystemIsSupported("cifs"))
                  {
                     cmd = "mount -t cifs ";
                     cmd += "-o user=";
                     cmd += (mp.userName) ? mp.userName : "anonymous";
                     cmd += (mp.password) ? (",password=" + mp.password) : "";
                     cmd += ",unc=//" + ip + "/" + mp.mountDir;
                     cmd += (mp.options) ? ("," + mp.options) : " ";
                     cmd += ip + "/" + mp.mountDir + " " + mp.localDir;
                  }

mount -t cifs -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream 192.168.1.1/Mp3-Stream /mnt/hdd/

Ob es jetzt user= oder username= heissen musst, sollte man noch probieren.

Gruß

Beitrag von **geko37** » Freitag 21. Dezember 2007, 19:19

Leider nein !

/ # mount -t cifs -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream //192.168.1.1/Mp3-Stream /mnt/hdd/
mount: mounting \\192.168.1.1\Mp3-Stream on /mnt/hdd/ failed: Invalid argument
/ #

--

per console geht nur so !
/ # mount -t cifs -o ro,soft,user=geko37,password=xxx,unc=//192.168.1.1/Mp3-Stream 192.168.1.1/Mp3-Stream /mnt/hdd/

Was macht den dieser part ??

Code: Alles auswählen

void eMountMgr::addMountedFileSystems()
......
...... ab Teile 505
......

			if (mountType.upper() == "CIFS")
			{
				sscanf(mountDev.c_str(), "//%d.%d.%d.%d/%*s", &mp.ip[0], &mp.ip[1], &mp.ip[2], &mp.ip[3]);
				mountDev = mountDev.right(mountDev.length() - 2); //strip off leading slashes
				mp.mountDir = getRight(mountDev, '/');
				mp.localDir = mountOn;
				mp.fstype = 1;
				mp.password = "";
				mp.userName = "";
				mp.automount = 0;
				mp.options = "";
				mp.description = "";
				mp.mounted = true;
				mp.id = -1; //don't care
				addMountPoint(mp);
			}

Beitrag von **geko37** » Mittwoch 26. Dezember 2007, 10:24

Kann den sich keiner mal erbarmen und dieses mit testen ?? NIcht das der Fehler hier liegt ! Mounten klappt nicht per Webif und auch nicht per Enigma mount !

Beitrag von **dbt** » Sonntag 30. Dezember 2007, 01:17

kompletter Patch aufs CDK:
busybox.1.8.2_cdk-diff-2007-12-30-00-17-01.patch

Beitrag von **mb405** » Sonntag 30. Dezember 2007, 10:05

hehe

hab grad die 1.9.0 entdeckt auf busybox seite.

Beitrag von **dbt** » Sonntag 30. Dezember 2007, 12:11

Habe ich schon gesehen, aber wart erst mal sehen ob 1.9.x auch schon sauber geht

Beitrag von **mb405** » Sonntag 30. Dezember 2007, 12:52

da werden wohl wieder die ganzen makes nich passen denke ich.

Beitrag von **kroki** » Sonntag 30. Dezember 2007, 13:30

Also ich hab die 1.9.0 am laufen, bis jetzt noch keine Probleme gefunden ...

cu Kroki

Beitrag von **ingrid** » Sonntag 30. Dezember 2007, 16:36

kroki hat geschrieben:Also ich hab die 1.9.0 am laufen, bis jetzt noch keine Probleme gefunden ...

dito

Beitrag von **geko37** » Sonntag 30. Dezember 2007, 17:31

kroki hat geschrieben:Also ich hab die 1.9.0 am laufen, bis jetzt noch keine Probleme gefunden ...

cu Kroki

Eventuell auch enigma damit getestet, ob man cifs mounten kann ?? Das ist ja derzeit das haupproblem, in Neutrino wurde es sofort gefixt aber Enigma noch immer nicht !

Beitrag von **Houdini** » Sonntag 30. Dezember 2007, 20:03

habt ihr den patch für init.c rausgenommen oder portiert?
wofür ist der überhaupt?

Beitrag von **ingrid** » Sonntag 30. Dezember 2007, 20:06

Houdini hat geschrieben:habt ihr den patch für init.c rausgenommen oder portiert?

Ich für meinen Teil hab den portiert.

Houdini hat geschrieben:wofür ist der überhaupt?

Keine Ahnung

, läuft jedenfalls ohne Probleme oder Fehlermeldungen.

Beitrag von **kroki** » Sonntag 30. Dezember 2007, 20:42

Der Patch für init.c muss portiert werden, sonst gibt es Zombie-Prozesse !

Code: Alles auswählen

diff -Naur busybox-1.9.0/init/init.c busybox-1.9.0-dbox2/init/init.c
--- busybox-1.9.0/init/init.c   2007-12-21 23:00:32.000000000 +0100
+++ busybox-1.9.0-dbox2/init/init.c     2007-12-30 13:20:48.000000000 +0100
@@ -497,10 +497,12 @@
        int status, wpid;

        while (1) {
-               wpid = waitpid(runpid, &status, 0);
-               if (wpid == -1 && errno == EINTR)
+               wpid = wait(&status);
+               if (wpid >0 && wpid != runpid) {
                        continue;
-               break;
+               }
+               if (wpid == runpid)
+                       break;
        }
        return wpid;
 }

Kroki

Beitrag von **Houdini** » Sonntag 30. Dezember 2007, 20:44

aber warum gibts das nur bei uns (dbox)?

Beitrag von **kroki** » Sonntag 30. Dezember 2007, 20:46

Kann ich dir auch nicht sagen .... habs nur mal ohne laufen lassen und dann gibt es halt immer Zombies, wenn du auf der dbox Prozesse killst ...

Kroki

Beitrag von **mb405** » Freitag 4. Januar 2008, 12:29

ich hab mal meinen versuch des diffs hochgeladen hier.
bitte mal gegenchecken mit euren diff. danke

Beitrag von **Houdini** » Freitag 4. Januar 2008, 20:21

ich habe den init.c patch mal in die bb mailing list eingekippt,
das wurde so erst mal committed

Tuxbox-Board

busybox-1.8.x

Re: busybox-1.8.x

Re: busybox-1.8.x