James Tabor jimtabor.rosdev@gmail.com wrote:
Nice! I will get to test slackware 13.x and the other fedora distros soon before doing the commit.
You should be able to simplify your patch to:
Index: tools/cabman/cabinet.cxx =================================================================== --- tools/cabman/cabinet.cxx (revision 48297) +++ tools/cabman/cabinet.cxx (working copy) @@ -21,7 +21,8 @@ #if !defined(WIN32) # include <dirent.h> +# include <sys/stat.h> +# include <sys/types.h> #endif -#if defined(__FreeBSD__) || defined(__APPLE__) -# include <sys/stat.h> -#endif // __FreeBSD__
Untested and mocked up, but you should get the idea :-) Both added includes are part of the Open Group Base Specifications, so there should be no harm in including them on all modern Unix systems.
Cheers,
Colin