L.D. Best wrote:
>
> Batch files for Arachne, particularly ones which can result in deletion
> of files, *must* be bulletproof.
Right !
> CLR.BAT as included with the Arachne package is FAR from bullet proof.
Right !
> I strongly urge you to test the revised CLR.BAT I've included in this
> message [users -- do this only if you have a mirror of your installation
> or a full backup, please!], and use it to replace the CLR.BAT file in
> all Arachne distributions which use it.
Right !
Except, please fix THIS line:
> if not "%3"=="" for %%f in (%3*.*) do del %%%f
While I don't know what damage the THREE "%" signs will do in various OSs,
they aren't proper. TWO "%" is the correct number, and given the sensitivity
of this particular .BAT, correct is best.
> I believe I tested carefully. But please don't take my word on this --
> I don't want anyone else yelling at me!!! If you are going to use the
> new file, TEST FIRST or wait for someone you trust to say s/he has run
> the tests and I am correct.
I agree. The onus should ALWAYS be on the testers to be careful and cross
their fingers and do whatever else is logical and necessary or people like
LD that spend a great deal of their own personal time pushing the limits
and passing on their experiences (good or bad) won't bother with us anymore.
Even Michael can make mistakes, (see the three upload messages) but we have
to live with the risks or go play somewhere else. This means don't blame
Michael for the oversight, and don't blame LD if the new .bat blows up.
I'm saying "good job", LD, just in case no one else steps up.
BTW, LD's new batch file looks OK to me, certainly better than the original.
Just so you know I looked seriously, I think the first 2 and last "if" tests
are unnecessary overkill but I don't see them hurting.
BTW2: I didn't test. I just looked.
- Clarence Verge
--
- Help stamp out FATWARE. As a start visit: http://home.arachne.cz/
--
