Building winetest-latest.exe?
Francois Gouget
fgouget at codeweavers.com
Wed Sep 5 04:29:11 CDT 2012
On Tue, 4 Sep 2012, Dan Kegel wrote:
> On Tue, Sep 4, 2012 at 3:18 PM, Francois Gouget <fgouget at codeweavers.com> wrote:
> >> (But as it turns out, the code I needed to fix was in buildbot,
> >> not winetest.)
> >
> > Maybe the WineTestBot code needs a similar fix?
> > In testbot/src/TestLauncher/TestLauncher.c?
>
> Yeah, um, ^buildbot^testbot
> I have a patch at
> http://bugs.winehq.org/show_bug.cgi?id=31609
> for your review.
Sorry about that. I forgot about you other email when replying to this
one. From the bug report (so others may comment):
You're calling the functions directly instead of using the function
pointers. Also it might be worth splitting the patch in two parts: one
to introduce the 'goto done' which I feel is worth it on its own and the
other for the activation context stuff. But then others might say this
is overkill.
Otherwise I can't test the patch right now but it looks good.
--
Francois Gouget <fgouget at codeweavers.com>,
More information about the wine-devel
mailing list