curl-library
Re: [PATCH] some more changes to testcurl.pl
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