Some interesting, some not:
/home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function]
/home/ken/wine-git/dlls/jscript/parser.y: conflicts: 1 shift/reduce, 18 reduce/reduce
/home/ken/wine-git/dlls/msi/sql.y: conflicts: 1 reduce/reduce /home/ken/wine-git/dlls/ntdll/directory.c: In function ‘wine_getdirentries’: /home/ken/wine-git/dlls/ntdll/directory.c:1710:5: warning: passing argument 4 of ‘getdirentries’ from incompatible pointer type [enabled by default] In file included from /home/ken/wine-git/dlls/ntdll/directory.c:29:0: /usr/include/dirent.h:313:18: note: expected ‘__off_t * __restrict__’ but argument is of type ‘long int *’
/home/ken/wine-git/dlls/vbscript/parser.y: conflicts: 6 shift/reduce /home/ken/wine-git/programs/winedbg/dbg.y: conflicts: 1 shift/reduce, 1 reduce/reduce /home/ken/wine-git/programs/winedbg/dbg.y: conflicts: 1 shift/reduce, 1 reduce/reduce
On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote:
Some interesting, some not:
/home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function]
I have a patch to fix those (and another one to fix them on Mac OS), but there's a bunch of patches ahead of it in my queue. Also, the patch was designed for standard FreeBSD (it needs libprocstat), so I'm not sure how well it will work on GNU/kFreeBSD. I've attached both patches so you can test it. N.B. the Mac OS patch needs to be applied first--it was developed first, and it implements the guts of the GetExtended*Table functions that actually call those functions. Also, you'll need to run autoreconf(1) after applying: the patches contain changes to configure.ac, but not configure or include/config.h.in.
/home/ken/wine-git/dlls/ntdll/directory.c: In function ‘wine_getdirentries’: /home/ken/wine-git/dlls/ntdll/directory.c:1710:5: warning: passing argument 4 of ‘getdirentries’ from incompatible pointer type [enabled by default] In file included from /home/ken/wine-git/dlls/ntdll/directory.c:29:0: /usr/include/dirent.h:313:18: note: expected ‘__off_t * __restrict__’ but argument is of type ‘long int *’
This one has to do with the actual getdirentries(2) prototype:
int getdirentries(int, char *, int, long *);
glibc's prototype is broken. It really is a 'long *', not an 'off_t *' (don't believe me? Go read the syscalls.master file in the kernel source). Or, maybe they're aware of this, and glibc's getdirentries(2) stub extends the 32-bit long (on a 32-bit system) into an off_t--in which case, we'll need a configure check for this.
Chip
On 08/08/13 21:28, Charles Davis wrote:
On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote:
Some interesting, some not:
/home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function]
I have a patch to fix those (and another one to fix them on Mac OS), but there's a bunch of patches ahead of it in my queue. Also, the patch was designed for standard FreeBSD (it needs libprocstat), so I'm not sure how well it will work on GNU/kFreeBSD. I've attached both patches so you can test it. N.B. the Mac OS patch needs to be applied first--it was developed first, and it implements the guts of the GetExtended*Table functions that actually call those functions. Also, you'll need to run autoreconf(1) after applying: the patches contain changes to configure.ac, but not configure or include/config.h.in.
Unfortunately the build breaks with the two patches applied in Wine 1.7.0.
winegcc: File does not exist: @LIBPROCSTAT@ make[1]: *** [iphlpapi.dll.so] Error 2 make: *** [dlls/iphlpapi] Error 2
The patches were applied cleanly.
On Aug 9, 2013, at 11:01 AM, Ken Sharp wrote:
On 08/08/13 21:28, Charles Davis wrote:
On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote:
Some interesting, some not:
/home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function]
I have a patch to fix those (and another one to fix them on Mac OS), but there's a bunch of patches ahead of it in my queue. Also, the patch was designed for standard FreeBSD (it needs libprocstat), so I'm not sure how well it will work on GNU/kFreeBSD. I've attached both patches so you can test it. N.B. the Mac OS patch needs to be applied first--it was developed first, and it implements the guts of the GetExtended*Table functions that actually call those functions. Also, you'll need to run autoreconf(1) after applying: the patches contain changes to configure.ac, but not configure or include/config.h.in.
Unfortunately the build breaks with the two patches applied in Wine 1.7.0.
winegcc: File does not exist: @LIBPROCSTAT@
Did you run autoreconf like I said?
Chip
On 10/08/13 00:01, Charles Davis wrote:
Did you run autoreconf like I said?
No! I'm useless!
I'll get back to you tomorrow.
Sorry. :(
On 08/08/13 21:28, Charles Davis wrote:
On Aug 8, 2013, at 8:20 AM, Ken Sharp wrote:
Some interesting, some not:
/home/ken/wine-git/dlls/iphlpapi/ipstats.c:1898:24: warning: ‘get_pid_map’ defined but not used [-Wunused-function] /home/ken/wine-git/dlls/iphlpapi/ipstats.c:1953:21: warning: ‘find_owning_pid’ defined but not used [-Wunused-function]
I have a patch to fix those (and another one to fix them on Mac OS), but there's a bunch of patches ahead of it in my queue. Also, the patch was designed for standard FreeBSD (it needs libprocstat), so I'm not sure how well it will work on GNU/kFreeBSD. I've attached both patches so you can test it. N.B. the Mac OS patch needs to be applied first--it was developed first, and it implements the guts of the GetExtended*Table functions that actually call those functions. Also, you'll need to run autoreconf(1) after applying: the patches contain changes to configure.ac, but not configure or include/config.h.in.
/home/ken/wine-git/dlls/ntdll/directory.c: In function ‘wine_getdirentries’: /home/ken/wine-git/dlls/ntdll/directory.c:1710:5: warning: passing argument 4 of ‘getdirentries’ from incompatible pointer type [enabled by default] In file included from /home/ken/wine-git/dlls/ntdll/directory.c:29:0: /usr/include/dirent.h:313:18: note: expected ‘__off_t * __restrict__’ but argument is of type ‘long int *’
This one has to do with the actual getdirentries(2) prototype:
int getdirentries(int, char *, int, long *);
glibc's prototype is broken. It really is a 'long *', not an 'off_t *' (don't believe me? Go read the syscalls.master file in the kernel source). Or, maybe they're aware of this, and glibc's getdirentries(2) stub extends the 32-bit long (on a 32-bit system) into an off_t--in which case, we'll need a configure check for this.
Chip
All looks good here. No warnings, no failures.
$ uname -srmvpio GNU/kFreeBSD 9.0-2-686 #0 Sun Jun 23 17:53:03 UTC 2013 i386 i386 Intel(R) Pentium(R) 4 CPU 3.00GHz GNU/kFreeBSD
$ ~/wine32/wine --version wine-1.7.0
There are a few test failures but I don't think that they are related (fixmes).
fixme:iphlpapi:SetTcpEntry (pTcpRow 0x33fcc8): stub iphlpapi.c:855: Test failed: got 0, expected 87 fixme:iphlpapi:SetTcpEntry (pTcpRow 0x33fcc8): stub iphlpapi.c:859: Test failed: got 0, expected 317 fixme:iphlpapi:NotifyAddrChange (Handle 0x33fcb4, overlapped 0x33fcb8): stub iphlpapi.c:1055: Test failed: GetLastError returned 203, expected ERROR_IO_PENDING fixme:iphlpapi:CancelIPChangeNotify (overlapped 0x33fcb8): stub iphlpapi.c:1057: Test failed: CancelIPChangeNotify returned FALSE, expected TRUE fixme:iphlpapi:CancelIPChangeNotify (overlapped 0x33fcb8): stub fixme:iphlpapi:NotifyAddrChange (Handle 0x33fcb4, overlapped 0x33fcb8): stub iphlpapi.c:1068: Test failed: NotifyAddrChange returned invalid file handle fixme:iphlpapi:CancelIPChangeNotify (overlapped 0x33fcb8): stub iphlpapi.c:1074: Test failed: CancelIPChangeNotify returned FALSE, expected TRUE
-
Charles Davis -
Ken Sharp