Re: [PATCH 1/5] dwrite/tests: Added GetFontSignature() test

[Marvin]

Hi,

While running your changed tests on Windows, I think I found new failures.
Being a bot and all I'm not very good at pattern recognition, so I might be
wrong, but could you please double-check?
Full results can be found at
https://testbot.winehq.org/JobDetails.pl?Key=22328

Your paranoid android.


=== build (build) ===
Patch failed to apply

[PATCH 1/5] dwrite/tests: Added GetFontSignature() test

[Nikolay Sivov]

On 19.04.2016 19:40, Marvin wrote:
> Hi,
>
> While running your changed tests on Windows, I think I found new failures.
> Being a bot and all I'm not very good at pattern recognition, so I might be
> wrong, but could you please double-check?
> Full results can be found at
> https://testbot.winehq.org/JobDetails.pl?Key=22328
>
> Your paranoid android.
>
> === build (build) ===
> Patch failed to apply

> Applying patch
> + git apply --verbose /home/winetest/tools/testbot/var/../var/staging/patch.diff
> Checking patch dlls/dwrite/tests/font.c...
> error: while searching for:
>     DELETE_FONTFILE(path);
> }
>
> START_TEST(font)
> {
>     IDWriteFactory *factory;
>
> error: patch failed: dlls/dwrite/tests/font.c:5840
> error: dlls/dwrite/tests/font.c: patch does not apply
> Build: Patch failed to apply

DELETE_FONTFILE() line was added with today commits, looks like it's not
picked up for some reason. I'll resend, let's see if it helps.