Paul Cunningham wrote:
>
>     +
>     Lines 124 & 125 and 107 & 108, do you really need to
>     do these 'chmod's, if not remove them (might reduce
>     build time a tiny bit).
>
>
>   

Is it ok to do this? The reason that this was done is that generally 
when we first start out doing integration work for SFW we are told to 
look at what others have done and occasionally we did things without 
fully understanding why we were doing them. It seems the chmodS on the 
directories and files in the source bundle are there to avoid any 
possible problems with permissions being mangled by umask and such. When 
I'm building Ruby from source I don't chmod everything first and we are 
quite happy to drop these lines if we know for certain that after 
putback, when the gate is built it won't break horribly. I'm not 
particularly conservative except when it comes to doing something that 
might result in getting Mike up at 3AM to fix a broken build.


Thanks

Amanda


Reply via email to