On Sun, Sep 09, 2007 at 07:25:37PM -0500, Karl O. Pinc wrote: > Package: mirrors > Severity: wishlist > Tags: patch > > The anonftpsync program does not take arguments, > it must be edited to be used. This is annoying > when it comes to mirroring more than one archive. > (e.g. the regular debian archive and volatile.debian.org, > or whatever.) > > Attached is a patch that fixes this problem. It does > so in the most painless way possible -- necessary variables > are taken from the environment when they exist.
Now, this is technically all right, but it touches on the issue of having the script be configured from the same file. The added visual complexity of having variables tested for existence before setting makes the upper section unsuitable for newbies. On the other hand, it was never defined exactly how non-newbie does one have to be to use it. At the same time, anonftpsync was never intended to be the end-all solution to all of people's mirroring needs - after all, it's just a shell script :) I suppose that the next step in the life of anonftpsync would be to move the configuration out of the script and into a separate file, and then the script wouldn't have to be too readable. At the same time, that would make it a notch harder to upgrade. That would be a good time to rename it, since it hasn't been using anonymous FTP for decades now :) -- Josip Rodin [EMAIL PROTECTED] -- To UNSUBSCRIBE, email to [EMAIL PROTECTED] with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]
