Re: [PATCH] Cwrapper should not eat -- arguments

[Charles Wilson]

Ralf Wildenhues wrote:
>>> Other than that, the only thing this patch is missing is a testsuite
>>> test ensuring that "--" really is passed through.  (Such an addition
>>> is preapproved.)
>> I tested this manually using -DDEBUGWRAPPER, but I don't know diddly
>> about writing tests for the testsuite -- and besides, this test 
would be
>> mingw/cygwin-host specific, and would fail on all cross builds unless
>> $build has wine and binfmt...  if the test is so fragile, what's the
>> point?
>
> I don't understand.  Programs on systems != mingw, cygwin should pass
> "--" through as well.  But inexperience with Autotest granted, I'll
> write a test.

I just meant that we don't use the cwrapper at all, unless 
$host=mingw/cygwin.  And when $host is mingw/cygwin, the cwrapper is 
compiled using the $host compiler, so it won't run on $build unless 
not-cross, or unless $build has a suitable emulation environment.

Of course, if $build != $host, and $build does not have a suitable 
emulation environment, everything in the whole test suite will fail/skip 
anyway, so that concern may not be an issue...

> Please go ahead with the patch, thanks!

OK.

--
Chuck