Some comments below. On Sat, 12 Mar 2011, Brian Bockelman wrote: > Hi, > > Took the chance to review Lukasz's new RPMs. Here are my items of concern: > - libXrdPss.so is in the server, but depends on libXrdFfs.so. Hence, xrootd-server is dependent on xrootd-fuse is dependent on fuse. > - In general, we probably want to remove the dependency on libfuse from libXrdPss. This then was improperly built. No individual shared libary or executable should directly depend on another xrootd-generated shared library. This is to make it easy to move things around and greatly aids in debugging. > - I notice you have three xrootd config files > ({xrootd,cmsd,frmd}-main.cf) - I thought it was the general style of the > project to have one config file? That absolutely true and quite essential. There should be only *one* configuration file. It is generally unwrokable to split them this way. > - What about the "cnsd" daemon? No started up script is given for that. We postponed his to first see how the essentials went. Anyway, we discourage people from running cnsd. > - I'm not sure it's really plausible to have multiple xrootds run in > this setup, but I think that's a larger issue It should have been possible. > Overall, a great improvement! Thanks Lukasz! Same here. Andy