From: SourceForge.net <no...@so...> - 2007-11-05 04:37:52
|
Patches item #1213103, was opened at 2005-06-01 18:16 Message generated for change (Comment added) made by nightstrike You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=302435&aid=1213103&group_id=2435 Please note that this message will contain a full copy of the comment thread, including the initial issue submission, for this request, not just the latest update. Category: w32api Group: Patch under review Status: Open Resolution: None Priority: 5 Private: No Submitted By: Julien Lecomte (julienlecomte) Assigned to: Earnie Boyd (earnie) Summary: w32api (automake) Initial Comment: This is a port of w32api to automake. Added to that, it has a simple autotest testsuite; very easy to improve an expand (testsuite is no longer in Makefile) Also, merges the DirectX files from /lib to /lib/directx, so that may be a source of bugs; but there were many duplicates of directx stuff in lib and lib/directx. Also, installs the headers in $includedir/w32api, which according to gcc spec file, is apparently the right folder. Comments welcome, Julien ---------------------------------------------------------------------- Comment By: NightStrike (nightstrike) Date: 2007-11-04 23:37 Message: Logged In: YES user_id=1864092 Originator: NO Has this port been completed? ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-11-07 09:00 Message: Logged In: YES user_id=827175 Revision 7 : - Merged with win32api 3.5 cvs version. ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-08-19 04:36 Message: Logged In: YES user_id=827175 Please only test latest revision, I'm keeping the other revision for reference. Revision 6 : * Configure.ac: Added version splitting rule * include\w32api.h: Removed. * include\w32api.h.in: Added. * tests\... Minor improvement changes * m4/msys_prog_dlltool.m4: Added value caching Revision 5: * Configure.ac: Changed version to 3.3 * test\Makefile.am: Added rules for version changes ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-08-18 10:31 Message: Logged In: YES user_id=827175 Note : AC_INIT still says version is 3.2. It should updated to 3.3 (and then run autoreconf) ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-08-17 21:59 Message: Logged In: YES user_id=827175 I've succeeded in compiling CygWin (src - 1.5.18-1) under CygWin. After that, I've succeeded to recompile CygWin (same version), under the CygWin previously created. This patch is to be applied to current CVS of w32api. If you wish to compile it under CygWin, get the src of CygWin, and replace the whole contents of /winsup/w32api by the patched version of the CVS. This will itself update CygWin to use w32api version 3.3 (instead of version 3.2) No errors were to be fixed during the CygWin build (apart the texi2html: command not found, error ignored) Now waiting for a user to test x-build. ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-06-04 05:40 Message: Logged In: YES user_id=827175 Danny, Your last comment made me realize that the w32api module (sourgeforge mingw repository) and that the w32api folder from a cygwin repository don't match ! I believe that you refer mostly to the winsup w32api; and occasionnally merge that one into the mingw32. I'll change the patch so that it will refer to the winsup w32api (and not the mingw w32api as I was currently doing) ---------------------------------------------------------------------- Comment By: Danny Smith (dannysmith) Date: 2005-06-03 03:40 Message: Logged In: YES user_id=11494 "* Merged duplicates in /lib and /lib/directx" You need to do a fresh checkout from CVS. This: 2004-03-27 Filip Navara <xn...@vo...> * include/directx: New subdir. * lib/directx: Ditto. * include/directx/(d3d9.h, d3d9caps.h, d3d9types.h, dxerr8.h, dxerr9.h): New files. * lib/directx/(Makefile.in, d3d8.def, d3d9.def, d3dim.def, d3drm.def, d3dx8d.def, d3dx9d.def, d3dxof.def, ddraw.def, dinput.def, dinput_joy.c, dinput_joy2.c, dinput_kbd.c, dinput_mouse.c, dinput_mouse2.c, dinput_private.h, dinput8.def, dmoguids.c, dplayx.def, dpnaddr.def, dpnet.def, dpnlobby.def, dpvoice.def, dsetup.def, dsound.def, dxerr.c, dxerr8.c, dxerr8w.c, dxerr9.c, dxerr9w.c, dxguid.c, ksproxy.def, ksuser.c, ksuser.def, msdmo.def, quartz.def, strmiids.c, test.c): Ditto. * lib/(d3dim.def, d3drm.def, d3dxof.def, ddraw.def, dinput.c, dinput.def, dplayx.def, dsetup.def, dsound.def, dxguid.c): Removed. Has already done the merge of the lib files. Your patch indicates that some symbols were omitted from the new directx def files in the above revision. If so, could you please document which symbols you are adding back in. Thanks Danny ---------------------------------------------------------------------- Comment By: Earnie Boyd (earnie) Date: 2005-06-02 20:40 Message: Logged In: YES user_id=15438 I mean Cygwin proper. If you prove that your changes do not break building Cygwin natively then I will ask CF or CV to check the cross build. Earnie ---------------------------------------------------------------------- Comment By: Julien Lecomte (julienlecomte) Date: 2005-06-02 16:13 Message: Logged In: YES user_id=827175 Made a couple of changes; in ChangeLog. Compile "cygwin" ? The "winsup" branch of cygwin, or just mSys DLL ? For mSys DLL, compiling works, *as long* as the "winnt.h" file in /lib/gcc-lib/i686-pc-msys/2.95.3-1/include is deleted. For DirectX files, I'll do that and submit a patch, but submitting a patch for that requires changing the current (cvs) Makefile.in. For mingw, mea culpa; I'm more fond of mSys... If target is mingw; then headers are installed in $includedir now, other targets get $includedir/w32api. Willing to work on this until accepted. J ---------------------------------------------------------------------- Comment By: Earnie Boyd (earnie) Date: 2005-06-02 07:37 Message: Logged In: YES user_id=15438 You'll have to test your configuration changes by building Cygwin. Sorry for this inconvenience but it is a long standing rule. Earnie ---------------------------------------------------------------------- Comment By: Danny Smith (dannysmith) Date: 2005-06-01 23:52 Message: Logged In: YES user_id=11494 "Also, merges the DirectX files from /lib to /lib/directx" Could you submit above as a separate patch. "Also, installs the headers in $includedir/w32api, which according to gcc spec file, is apparently the right folder." Huh? Not for mingw Danny. ---------------------------------------------------------------------- You can respond by visiting: https://sourceforge.net/tracker/?func=detail&atid=302435&aid=1213103&group_id=2435 |