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 .
