Upgrade Wizard fails on file permissions

[Kalendrinn]

As noticed in my sig, I am running Win2k3 and IIS 6. I haven't looked to see what you guys are using to do the file permissions check, but it's not doing it right. The first time I tried the upgrade wizard for the 5.0.0b patch, it came up with some notice (but no link) about file permissions and let me continue...then after I got the patch uploaded I got an error in dir_inc.php that the input was too large for chdir() when installing the patch.

So I retried it and all I get now is the installer dying saying I need to take care of file permissions. I look at the list and it's just about every file in Sugar and it shows as 0666 for permissions which I know is incorrect (I even went through and reset them). It also cannot determine the user and group.

So I am not sure what to do but it is preventing me from upgrading to the 5.0.0b patch...and I would really like to get rid of some of the bugs in the system.

Any and all help would be greatly appreciated.

[Kalendrinn]

Just for shits and giggles I changed the file perms to 777 and it let me continue to the next step finally. Seriously, everything has to be set to writable for it to work??!?!!

But I'm back to where I was in the beginning...with the following error:

Warning: chdir() [function.chdir]: Result too large (errno 34) in G:\webs\sugar\include\dir_inc.php on line 183

That's after uploading the file and going to the next step (preflight check). Anyone got any ideas?

[Kalendrinn]

I gave it another go, and at the point I get the chdir() error above I managed to get a Next button that allowed me to continue to install it. This is all on my development box which is a copy of production (just in case there was doubt).

I am not willing to install this on production unless I know it will not error, as erroring may cause problems even though it somehow succeeded on the development box.

So I still need help fixing whatever is wrong so it will install without issue.

[Kalendrinn]

I figured out the problem, the directory doesn't exist, so any of the dir functions bork and throw a "result too large" error. ERROR CHECKING! Utility functions still need it guys.

Replace the following at line 183(ish) in include/dir_inc.php:

 chdir( $the_dir );
 $the_array = findAllFiles( ".", $the_array );
 chdir( $original_dir );
 return( $the_array );

with the following:

 if(!file_exists($original_dir."/".$the_dir)) return $the_array;
 else return findAllFiles($original_dir."/".$the_dir, $the_array);

Also, as a side note, since I found the initial error with the object oriented dir class and associated read() function...I would highly suggest replacing all of them with the opendir/readdir/closedir functions I mentioned in a previous post...at least until the dir class is up to par with opendir and readdir.

I understand the desire to be object oriented and use classes...but if the classes don't work in a robust manner, don't use them.