I did that once. Only I modified the current --allow-root option to allow it to accept a semicolon-delimited list. I mailed it to this list and was promptly notified that there is a simpler way ... duh: write a shell script and call that instead of directly calling CVS from the inetd configuration. Then you can have all the --allow-root arguments you like, and - bonus here too - you can change and add repositories to the allowed list without reinitializing inetd. That said, I still think that it is nice to have a means of specifying a repository list to make things more concise. -- Matthew Brown ----- Original Message ----- From: "Rob" <[EMAIL PROTECTED]> To: <[EMAIL PROTECTED]> Sent: Tuesday, March 14, 2000 4:32 PM Subject: feature to deal with multiple "--allow-root=foo" lines > > Hi, > > I am trying to setup a single cvs server to server multiple (10+) repositories > (via "pserver"). From the info page, I am apparently supposed to have one > "--allow-root" line per repository, but that exceeds inetd's internal line > limit, and the solution of having inetd call a shell script seems crufty. > > So, what I have done is created an extra option to cvs to take the list of > allowable root directories from a file. I have called this option > "--allow-root-list", and have a patch available for it. > > The reason why I am mailing this to the list is I am fairly new to cvs, and I > don't want to be re-implementing old features that I may have missed, or > falling into old pit falls that others have fallen into before me. Reading > through the HACKING file, it recommends posting to a mailing list/news group > basically to bounce the idea off of someone. Well... here it is. > > I also added a check to make sure that anything passed to > src/root.c:root_allow_add() is in fact a directory, and does in fact exist > (with non-fatal errors if either check fails). Any comments on that would > also be appreciated. > > The patch is against cvs-1.10.8, and can be retrieved at: > > http://www.cs.umd.edu/~capveg/cvs.allow-root-list.patch > > Any feed back would be most useful. If there is anything else that I can do > to increase the chances of it being added to the main distribution, please let > me know. > > TIA, > > - Rob > . > >
