This fix is to resolve a problem with python's os.path.basename and os.path.dirname behavior as they do not work the same as the linux basename/dirname commands.  If the path specified contains a trailing '/', basename will be empty and dirname will give you the lowest level in the path rather than the parent as linux does. . This caused problems in some of the code which was attempting to validate the path and create missing directories. . It also caused a problem with the condor switchboard when privilege separation was in effect when the valid-dirs option value of the /etc/condor/privsep_config contained a trailing '/'.  The switchboard would throw an error saying it could not create the directory.  This occured when trying to create the factory. . To resolve this, the very basic method to retrieve a value for an ini option was changed to strip all trailing '/'s from all values returned.  This will work for the glideinWMS application as it stands right now.  If this changes and some other option does require or allow a trailing '/', this will have to change. . Whew! A lot of words for a small code change, but I thought it best that all are aware of this.
This fix is to resolve a problem with python's os.path.basename and
os.path.dirname behavior as they do not work the same as the linux
basename/dirname commands.  If the path specified contains a trailing '/',
basename will be empty and dirname will give you the lowest level in the
path rather than the parent as linux does.
.
This caused problems in some of the code which was attempting to validate
the path and create missing directories.
.
It also caused a problem with the condor switchboard when privilege
separation was in effect when the valid-dirs option value of the
/etc/condor/privsep_config contained a trailing '/'.  The switchboard would
throw an error saying it could not create the directory.  This occured
when trying to create the factory.
.
To resolve this, the very basic method to retrieve a value for an ini option
was changed to strip all trailing '/'s from all values returned.  This will
work for the glideinWMS application as it stands right now.  If this changes
and some other option does require or allow a trailing '/', this will
have to change.
.
Whew! A lot of words for a small code change, but I thought it best that all
are aware of this.

bbockelm/glideinWMS,bbockelm/glideinWMS,bbockelm/glideinWMS,bbockelm/glideinWMS