February 2016 - Wine-devel - list.winehq.org

Re: [PATCH 1/2] msvcp110: Add tr2_sys__Unlink implementation and test.(resend)
by Marvin 19 Feb '16

19 Feb '16

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=20725 Your paranoid android. === build (build) === Patch failed to apply

2 1

Re: [PATCH 2/2] msvcp110: Add tr2_sys__Stat_wchar and tr2_sys__Lstat_wchar implementation and test.(resend)
by Marvin 19 Feb '16

19 Feb '16

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=20726 Your paranoid android. === build (build) === Patch failed to apply

1 0

Re: msctf/tests: Use todo_wine_if() in tests
by Marvin 19 Feb '16

19 Feb '16

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=20718 Your paranoid android. === w1064 (32 bit inputprocessor) === inputprocessor.c:787: Test failed: Deactivation came unexpectedly

1 0

[RFC PATCH 2/2] Load device drivers of the same group into the same winedevice process
by Aric Stewart 19 Feb '16

19 Feb '16

Signed-off-by: Aric Stewart <aric(a)codeweavers.com> --- programs/services/services.c | 94 +++++++++++++++++++++++++++++-- programs/winedevice/device.c | 128 ++++++++++++++++++++++++++++++++++++++----- 2 files changed, 204 insertions(+), 18 deletions(-)

1 0

[RFC PATCH 1/2] programs/services: Break services into Loader groups
by Aric Stewart 19 Feb '16

19 Feb '16

Sebastian was requesting that multiple drivers be able to be loaded into the same winedevice process. Discussing this with Alexandre he expressed that instead of my approach to the plug and play manager as a service it may be more proper to have it be a kernel driver that then depends on other drivers being loaded into the same process. So here is a proposed process to allow this to happen. Right now these 2 patches depend on my moving the driver loading code, but that is not a hard dependency at all, it would be relatively straight forward to be able to keep all the driver loading code into winedevice and still have these patches. This is a request for comment before submitting it to wine-patches. Also to decide if we should move the driver loading or not, as that does affect what set of patches are submitted. -aric Sort the groups by the service's tag to start in order --- programs/services/services.c | 101 ++++++++++++++++++++++++++++++++----------- 1 file changed, 75 insertions(+), 26 deletions(-)

1 0

Re: [PATCH] kernel32: Use setprogname(3) to set the program name if available.
by Alexandre Julliard 19 Feb '16

19 Feb '16

Charles Davis <cdavis5x(a)gmail.com> writes: > @@ -1118,21 +1118,29 @@ static void set_process_name( int argc, char *argv[] ) > { > #ifdef HAVE_SETPROCTITLE > setproctitle("-%s", argv[1]); > -#endif > - > -#ifdef HAVE_PRCTL > +#elif defined(HAVE_PRCTL) || defined(HAVE_SETPROGNAME) > int i, offset; > char *p, *prctl_name = argv[1]; > char *end = argv[argc-1] + strlen(argv[argc-1]) + 1; > - > -#ifndef PR_SET_NAME > -# define PR_SET_NAME 15 > +#ifdef HAVE_SETPROGNAME > + static char progname[PATH_MAX]; > #endif > > if ((p = strrchr( prctl_name, '\\' ))) prctl_name = p + 1; > if ((p = strrchr( prctl_name, '/' ))) prctl_name = p + 1; > > +#ifdef HAVE_SETPROGNAME > + lstrcpynA( progname, prctl_name, sizeof(progname)/sizeof(progname[0]) ); > + progname[PATH_MAX-1] = 0; > + setprogname( progname ); > + if (1) > +#elif defined(HAVE_PRCTL) > +#ifndef PR_SET_NAME > +# define PR_SET_NAME 15 > +#endif > + > if (prctl( PR_SET_NAME, prctl_name ) != -1) > +#endif This is becoming hard to follow, please try to clean up these #ifdefs a little. Also you shouldn't need to make a copy, it can point directly into argv. -- Alexandre Julliard julliard(a)winehq.org

1 0

Re: advapi32: add stub for AddAuditAccessObjectAce
by Nikolay Sivov 19 Feb '16

19 Feb '16

On 19.02.2016 11:40, Piet Jankbal wrote: > For bug https://bugs.winehq.org/show_bug.cgi?id=39918 > <https://bugs.winehq.org/show_bug.cgi?id=29862> > > > Signed-off-by: Louis Lenders <xerox.xerox2000x(a)gmail.com> > > > > This should be marked as a part of series, as they depend on each other. Also I think it's better to do a little bit more and forward them to ntdll Rtl* calls, like some existing calls already do, it's safe to assume prototype are the same between the two.

1 0

Re: advapi32: add stub for AddAuditAccessObjectAce
by Austin English 19 Feb '16

19 Feb '16

On Fri, Feb 19, 2016 at 2:40 AM, Piet Jankbal <xerox.xerox2000x(a)gmail.com> wrote: > For bug https://bugs.winehq.org/show_bug.cgi?id=39918 > > > Signed-off-by: Louis Lenders <xerox.xerox2000x(a)gmail.com> The name on your email, the named on the signed off by, and the name on the patch are all different. Why? -- -Austin

1 0

Re: [4/4] ntdll: Set correct number of arguments for breakpoint exceptions.
by Marvin 19 Feb '16

19 Feb '16

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=20712 Your paranoid android. === wvistau64 (32 bit exception) === exception.c:490: Test failed: 42: Unexpected eip 0x2e0000/0x2e0001 exception: unhandled exception 80000004 at 002E0000 exception.c:490: Test failed: 42: Unexpected eip 0x77baed97/0x2e0001 === w7u (32 bit exception) === exception.c:490: Test failed: 42: Unexpected eip 0x3f0000/0x3f0001 exception: unhandled exception 80000004 at 003F0000 exception.c:490: Test failed: 42: Unexpected eip 0x77cde34f/0x3f0001 === w1064 (32 bit exception) === Failure running script in VM: network read timed out (Connect:AgentVersion.h:0/9) === w1064 (64 bit exception) === Failure running script in VM: network read timed out (Connect:AgentVersion.h:0/9)

1 0

Re: [3/4] ntdll: Fix up instruction pointer in context for EXCEPTION_BREAKPOINT on x86_64.
by Marvin 19 Feb '16

19 Feb '16

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=20711 Your paranoid android. === wvistau64 (32 bit exception) === exception.c:490: Test failed: 42: Unexpected eip 0x3c0000/0x3c0001 exception: unhandled exception 80000004 at 003C0000 exception.c:490: Test failed: 42: Unexpected eip 0x77baed97/0x3c0001 === w7u (32 bit exception) === exception.c:490: Test failed: 42: Unexpected eip 0x2f0000/0x2f0001 exception: unhandled exception 80000004 at 002F0000 exception.c:490: Test failed: 42: Unexpected eip 0x77cde34f/0x2f0001 === w1064 (32 bit exception) === Failure running script in VM: network read timed out (Connect:AgentVersion.h:0/9) === w1064 (64 bit exception) === Failure running script in VM: network read timed out (Connect:AgentVersion.h:0/9)

1 0