[torquedev] qsub -W umask

[Garrick Staples]

On Fri, Sep 29, 2006 at 05:23:07PM -0600, Garrick Staples alleged:
> On Fri, Sep 29, 2006 at 08:15:02AM -0400, Glen Beane alleged:
> > On 9/28/06, Garrick Staples <garrick at clusterresources.com> wrote:
> > >On Thu, Sep 28, 2006 at 09:02:53PM -0400, Glen Beane alleged:
> > >> I added support for changing the umask used while creating the stdout
> > >> and stderr spool files.
> > >>
> > >> There was a request for a behavior similar to PBS Pro, so I
> > >> implemented that in the trunk.
> > >>
> > >> Please test and provide me with feedback.
> > >
> > >Is the intent to set the umask for the spooled job files, or for the
> > >final delivered files?
> > >
> > >Setting umask for files in $PBS_SERVER_HOME/spool doesn't seem all that
> > >useful.
> > 
> > 
> > Thats true,  they should be set for the delivered files.  I didn't
> > check to see if the permissions are preserved when the file is
> > delivered (oops!).   although I suppose setting the umask for files in
> > $PBS_SERVER_HOME/spool could be useful to allow other users to check
> > the progress of your job...
> 
> Now that I think about it, didn't we already set '-p' as an arg to 'cp'
> for this case?  I remember doing it, but it doesn't actually appear to
> be true.
> 
> src/resmom/requests.c:sys_copy():
>   if (rmtflg == 0)
>     {
>     /* local copy */
> 
>     ag0 = "/bin/cp";
>     ag1 = "-r";
>     }

I've (re?)added the missing '-p' to svn trunk and 2.1-fixes.