Comment 5 for bug 602134

Revision history for this message
Gursimran singh (simar) wrote :

Thanks again Bhavani for your review. Now I am happy for better patching. I suppose ... :))

As far as MOCOS is concerned I think it is a macro for MAC operating system. MAC OS might need these headers for earlier versions, we did not require these in lucid but in maverick we do. So, to keep the source code still consistent with MAC, I have not touched this code and added a separate line..

or if we want to increase our work we may have something like this

#ifdef MAVERICK
#include <sys/stat.h>
....
..

I think by doing this in MAC also we do not have any multiple inclusions, for obvious reasons. So we can go ahead. But I'm not sure ofcourse that what exactly MACOS means and should we go like this or not..