cURL / Mailing Lists / curl-library / Single Mail

curl-library

Re: [PATCH] some more changes to testcurl.pl

From: Guenter Knauf <eflash_at_gmx.net>
Date: Tue, 29 Jun 2004 15:18:15 +0200

Hi Daniel,
>> I've added a --setup option so that you can now specify different setup
>> files; and for NetWare I append the $targetos to the setup filename;

> Uh, why? If you support a --setup option, why append anything at all? It
> makes better sense to me if the option simply specifies the full path name to
> the alternative file.
that's what the patch already does.

> I don't see any benefit with automatically appending $targetos to the
> name.
only that you dont need a second option when you specify --target since I believe its most likely that you also want to use another setupfile name then; and if you use --setup too this wins anyway...

> ... and with my suggestion, the existing builds would continue working.
ok, I removed the line completely.

Guenter.
Received on 2004-06-29