[Top][All Lists]
[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH] Handle odd winepath failures.
From: |
Ralf Wildenhues |
Subject: |
Re: [PATCH] Handle odd winepath failures. |
Date: |
Tue, 7 Sep 2010 23:14:43 +0200 |
User-agent: |
Mutt/1.5.20 (2010-04-22) |
* Peter Rosin wrote on Tue, Aug 31, 2010 at 09:36:15PM CEST:
> Den 2010-08-31 18:57 skrev Ralf Wildenhues:
> > * Peter Rosin wrote on Tue, Aug 31, 2010 at 06:47:24PM CEST:
> >> Subject: [PATCH] Handle odd winepath failures.
> >>
> >> * lib/ar-lib (func_file_conv): winepath can exit with zero exit
> >> status but empty result, and can return a basically unconverted
> >> path, but prefixed with \\?\unix. Treat both of these as failure
> >> and retain the original file name.
> >> * lib/compile (func_file_conv): Likewise.
> >> Suggested by Ralf Wildenhues after reports of similar problems
> >> elsewhere from Charles Wilson and Roumen Petrov.
> >
> > OK for msvc if you update $scriptversion in both scripts and add a test
> > case. Ideally two, one that is effective only on systems with wine
> > installed and ensures that the output file name is not empty, the other
> > one can fake, extending ar-lib.test.
>
> That will have to wait then, because I can't test with real Wine easily
> ATM...
Can you create a test without testing it? Someone else could do that
for you. It's probably good enough even to have a stricter than
necessary test: as long as we know the intended coverage, we can always
fixup test failures later.
Thanks,
Ralf
- Re: [PATCH] Handle odd winepath failures.,
Ralf Wildenhues <=