Hi Gregory, I agree. The next release will have your requested changes in it. Andy ----- Original Message ----- From: "Gregory J. Sharp" <[log in to unmask]> To: "Xrootd Mailing List" <[log in to unmask]> Sent: Friday, October 08, 2004 1:16 PM Subject: Requested improvement to OSS module > Folks, > > I have to OSS enhancement requests. They allowed me to successfully > stage in data for the first time. > > 1. The mps_Stage script (and probably all the other mps_ scripts) needs > to be passed the -c configFile > option to make it work for those of us who don't install xrootd in > /opt/xrootd. > > This turns out to be easy for mps_Stage and is probably easy for the > other scripts. > In the file XrdOssStage.cc, the method int XrdOssSys::GetFile needs the > following changes: > > a) char * arglist[4]; becomes > char * arglist[6]; > b) at about line 328, change the assignments to arglist to be > arglist[0] = StageCmd; > arglist[1] = (char *) "-c"; > arglist[2] = ConfigFN; > arglist[3] = rfs_fn; > arglist[4] = lfs_fn; > arglist[5] = (char *)0; > > 2. There is a "bug" of sorts in mps_Stage in Init_Config > somewhere near line 817 is the code > while( <CONFIG> ) { > s/#.*$//; #remove comments > ($var, $val) = split(" ",$_,2); > ($x, $val) = split(" ", $val,2) if $val =~ /^= /; # remove = > > It turns out that the splits only work if you have a space between the > variable name and the = > and likewise after the =. I have only tabs after the = and it didn't > work. There are two solutions to this. The clearest is to change the " > " argument to the splits to be "\s". The perl manual indicates that you > can also change it to ' ' and split will interpret it as \s, but I > haven't tested this. No doubt this change will be needed in other mps_ > scripts that read the config file. (One could put this code into a .pm > file to share it amongst all the mps_ scripts, of course...) > > -- > Gregory J. Sharp email: [log in to unmask] > Wilson Synchrotron Laboratory url: > http://www.lepp.cornell.edu/~gregor > Dryden Rd ph: +1 607 255 4882 > Ithaca, NY 14853 fax: +1 607 255 8062 > >