February 2010 - Wine-Commits

Hans Leidekker : msi: Implement the RemoveODBC standard action.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 28bf8e1f0db2268ee3f8aa98497e06ea7e949f90 URL: http://source.winehq.org/git/wine.git/?a=commit;h=28bf8e1f0db2268ee3f8aa984… Author: Hans Leidekker <hans(a)codeweavers.com> Date: Thu Feb 11 13:13:41 2010 +0100 msi: Implement the RemoveODBC standard action. --- dlls/msi/action.c | 120 ++++++++++++++++++++++++++++++++++++++++++++++++++--- 1 files changed, 114 insertions(+), 6 deletions(-) diff --git a/dlls/msi/action.c b/dlls/msi/action.c index b949fc7..31870bd 100644 --- a/dlls/msi/action.c +++ b/dlls/msi/action.c @@ -5086,6 +5086,120 @@ static UINT ACTION_InstallODBC( MSIPACKAGE *package ) return rc; } +static UINT ITERATE_RemoveODBCDriver( MSIRECORD *rec, LPVOID param ) +{ + DWORD usage; + LPCWSTR desc; + + desc = MSI_RecordGetString( rec, 3 ); + if (!SQLRemoveDriverW( desc, FALSE, &usage )) + { + WARN("Failed to remove ODBC driver\n"); + } + else if (!usage) + { + FIXME("Usage count reached 0\n"); + } + + return ERROR_SUCCESS; +} + +static UINT ITERATE_RemoveODBCTranslator( MSIRECORD *rec, LPVOID param ) +{ + DWORD usage; + LPCWSTR desc; + + desc = MSI_RecordGetString( rec, 3 ); + if (!SQLRemoveTranslatorW( desc, &usage )) + { + WARN("Failed to remove ODBC translator\n"); + } + else if (!usage) + { + FIXME("Usage count reached 0\n"); + } + + return ERROR_SUCCESS; +} + +static UINT ITERATE_RemoveODBCDataSource( MSIRECORD *rec, LPVOID param ) +{ + LPWSTR attrs; + LPCWSTR desc, driver; + WORD request = ODBC_REMOVE_SYS_DSN; + INT registration; + DWORD len; + + static const WCHAR attrs_fmt[] = { + 'D','S','N','=','%','s',0 }; + + desc = MSI_RecordGetString( rec, 3 ); + driver = MSI_RecordGetString( rec, 4 ); + registration = MSI_RecordGetInteger( rec, 5 ); + + if (registration == msidbODBCDataSourceRegistrationPerMachine) request = ODBC_REMOVE_SYS_DSN; + else if (registration == msidbODBCDataSourceRegistrationPerUser) request = ODBC_REMOVE_DSN; + + len = strlenW( attrs_fmt ) + strlenW( desc ) + 1 + 1; + attrs = msi_alloc( len * sizeof(WCHAR) ); + if (!attrs) + return ERROR_OUTOFMEMORY; + + FIXME("Use ODBCSourceAttribute table\n"); + + len = sprintfW( attrs, attrs_fmt, desc ); + attrs[len + 1] = 0; + + if (!SQLConfigDataSourceW( NULL, request, driver, attrs )) + { + WARN("Failed to remove ODBC data source\n"); + } + msi_free( attrs ); + + return ERROR_SUCCESS; +} + +static UINT ACTION_RemoveODBC( MSIPACKAGE *package ) +{ + UINT rc; + MSIQUERY *view; + + static const WCHAR driver_query[] = { + 'S','E','L','E','C','T',' ','*',' ','F','R','O','M',' ', + 'O','D','B','C','D','r','i','v','e','r',0 }; + + static const WCHAR translator_query[] = { + 'S','E','L','E','C','T',' ','*',' ','F','R','O','M',' ', + 'O','D','B','C','T','r','a','n','s','l','a','t','o','r',0 }; + + static const WCHAR source_query[] = { + 'S','E','L','E','C','T',' ','*',' ','F','R','O','M',' ', + 'O','D','B','C','D','a','t','a','S','o','u','r','c','e',0 }; + + rc = MSI_DatabaseOpenViewW( package->db, driver_query, &view ); + if (rc != ERROR_SUCCESS) + return ERROR_SUCCESS; + + rc = MSI_IterateRecords( view, NULL, ITERATE_RemoveODBCDriver, package ); + msiobj_release( &view->hdr ); + + rc = MSI_DatabaseOpenViewW( package->db, translator_query, &view ); + if (rc != ERROR_SUCCESS) + return ERROR_SUCCESS; + + rc = MSI_IterateRecords( view, NULL, ITERATE_RemoveODBCTranslator, package ); + msiobj_release( &view->hdr ); + + rc = MSI_DatabaseOpenViewW( package->db, source_query, &view ); + if (rc != ERROR_SUCCESS) + return ERROR_SUCCESS; + + rc = MSI_IterateRecords( view, NULL, ITERATE_RemoveODBCDataSource, package ); + msiobj_release( &view->hdr ); + + return rc; +} + #define ENV_ACT_SETALWAYS 0x1 #define ENV_ACT_SETABSENT 0x2 #define ENV_ACT_REMOVE 0x4 @@ -6290,12 +6404,6 @@ static UINT ACTION_RemoveExistingProducts( MSIPACKAGE *package ) return msi_unimplemented_action_stub( package, "RemoveExistingProducts", table ); } -static UINT ACTION_RemoveODBC( MSIPACKAGE *package ) -{ - static const WCHAR table[] = { 'O','D','B','C','D','r','i','v','e','r',0 }; - return msi_unimplemented_action_stub( package, "RemoveODBC", table ); -} - static UINT ACTION_RemoveRegistryValues( MSIPACKAGE *package ) { static const WCHAR table[] = { 'R','e','m','o','v','e','R','e','g','i','s','t','r','y',0 };

1 0

Jacek Caban : mshtml: Added IHTMLStyle::get_border*Color implementation.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 6e116f521585331876f02df3ed8abdbc363cb3b0 URL: http://source.winehq.org/git/wine.git/?a=commit;h=6e116f521585331876f02df3e… Author: Jacek Caban <jacek(a)codeweavers.com> Date: Thu Feb 11 01:19:58 2010 +0100 mshtml: Added IHTMLStyle::get_border*Color implementation. --- dlls/mshtml/htmlstyle.c | 18 ++++++++++++------ dlls/mshtml/tests/dom.c | 12 ++++++------ 2 files changed, 18 insertions(+), 12 deletions(-) diff --git a/dlls/mshtml/htmlstyle.c b/dlls/mshtml/htmlstyle.c index f96bee2..cb86b79 100644 --- a/dlls/mshtml/htmlstyle.c +++ b/dlls/mshtml/htmlstyle.c @@ -1533,8 +1533,10 @@ static HRESULT WINAPI HTMLStyle_put_borderTopColor(IHTMLStyle *iface, VARIANT v) static HRESULT WINAPI HTMLStyle_get_borderTopColor(IHTMLStyle *iface, VARIANT *p) { HTMLStyle *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + return get_nsstyle_attr_var(This->nsstyle, STYLEID_BORDER_TOP_COLOR, p, 0); } static HRESULT WINAPI HTMLStyle_put_borderRightColor(IHTMLStyle *iface, VARIANT v) @@ -1563,8 +1565,10 @@ static HRESULT WINAPI HTMLStyle_put_borderBottomColor(IHTMLStyle *iface, VARIANT static HRESULT WINAPI HTMLStyle_get_borderBottomColor(IHTMLStyle *iface, VARIANT *p) { HTMLStyle *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + return get_nsstyle_attr_var(This->nsstyle, STYLEID_BORDER_BOTTOM_COLOR, p, 0); } static HRESULT WINAPI HTMLStyle_put_borderLeftColor(IHTMLStyle *iface, VARIANT v) @@ -1577,8 +1581,10 @@ static HRESULT WINAPI HTMLStyle_put_borderLeftColor(IHTMLStyle *iface, VARIANT v static HRESULT WINAPI HTMLStyle_get_borderLeftColor(IHTMLStyle *iface, VARIANT *p) { HTMLStyle *This = HTMLSTYLE_THIS(iface); - FIXME("(%p)->(%p)\n", This, p); - return E_NOTIMPL; + + TRACE("(%p)->(%p)\n", This, p); + + return get_nsstyle_attr_var(This->nsstyle, STYLEID_BORDER_LEFT_COLOR, p, 0); } static HRESULT WINAPI HTMLStyle_put_borderWidth(IHTMLStyle *iface, BSTR v) diff --git a/dlls/mshtml/tests/dom.c b/dlls/mshtml/tests/dom.c index dc37a5e..f4f673c 100644 --- a/dlls/mshtml/tests/dom.c +++ b/dlls/mshtml/tests/dom.c @@ -4522,8 +4522,8 @@ static void test_default_style(IHTMLStyle *style) */ V_BSTR(&v) = NULL; hres = IHTMLStyle_get_borderTopColor(style, &v); - todo_wine ok(hres == S_OK, "get_borderTopColor failed: %08x\n", hres); - todo_wine ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); + ok(hres == S_OK, "get_borderTopColor failed: %08x\n", hres); + ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); VariantClear(&v); V_BSTR(&v) = NULL; @@ -4556,8 +4556,8 @@ static void test_default_style(IHTMLStyle *style) */ V_BSTR(&v) = NULL; hres = IHTMLStyle_get_borderBottomColor(style, &v); - todo_wine ok(hres == S_OK, "get_borderBottomColor failed: %08x\n", hres); - todo_wine ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); + ok(hres == S_OK, "get_borderBottomColor failed: %08x\n", hres); + ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); VariantClear(&v); V_BSTR(&v) = NULL; @@ -4590,8 +4590,8 @@ static void test_default_style(IHTMLStyle *style) */ V_BSTR(&v) = NULL; hres = IHTMLStyle_get_borderLeftColor(style, &v); - todo_wine ok(hres == S_OK, "get_borderLeftColor failed: %08x\n", hres); - todo_wine ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); + ok(hres == S_OK, "get_borderLeftColor failed: %08x\n", hres); + ok(!strcmp_wa(V_BSTR(&v), "red"), "str=%s\n", wine_dbgstr_w(V_BSTR(&v))); VariantClear(&v); V_BSTR(&v) = NULL;

1 0

Alexandre Julliard : makefiles: Add a __tooldeps__ fake dependency to rebuild only the tools.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 1431e24feeb20dac8a2e791f63cb0a7d4c32c1bf URL: http://source.winehq.org/git/wine.git/?a=commit;h=1431e24feeb20dac8a2e791f6… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Thu Feb 11 12:53:53 2010 +0100 makefiles: Add a __tooldeps__ fake dependency to rebuild only the tools. --- Makefile.in | 7 ++++--- 1 files changed, 4 insertions(+), 3 deletions(-) diff --git a/Makefile.in b/Makefile.in index 059fb6c..8aa498f 100644 --- a/Makefile.in +++ b/Makefile.in @@ -198,9 +198,10 @@ implib: $(ALL_STATICLIB_DIRS) $(IMPORT_LIBS) # Dependencies between directories # dependencies needed to build any dll or program -__builddeps__: libs/port libs/wine libs/wpp $(ALL_TOOL_DIRS) include $(ALL_STATICLIB_DIRS) $(IMPORT_LIBS) -__buildcrossdeps__: libs/port libs/wine $(ALL_TOOL_DIRS) include $(STATIC_LIBS:.a=.cross.a) $(IMPORT_LIBS:.def=.cross.a) -.PHONY: __builddeps__ __buildcrossdeps__ +__tooldeps__: libs/port libs/wine libs/wpp $(ALL_TOOL_DIRS) +__builddeps__: __tooldeps__ include $(ALL_STATICLIB_DIRS) $(IMPORT_LIBS) +__buildcrossdeps__: __tooldeps__ include $(STATIC_LIBS:.a=.cross.a) $(IMPORT_LIBS:.def=.cross.a) +.PHONY: __tooldeps__ __builddeps__ __buildcrossdeps__ loader server: libs/port libs/wine tools fonts: tools

1 0

Alexandre Julliard : makefiles: Simplify the WINE_CONFIG_MAKEFILE macro by removing the no longer needed prefix arg .
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: c6e62dda1c1d71ebcaf5a6fc347f0e26ef43bebd URL: http://source.winehq.org/git/wine.git/?a=commit;h=c6e62dda1c1d71ebcaf5a6fc3… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 19:37:48 2010 +0100 makefiles: Simplify the WINE_CONFIG_MAKEFILE macro by removing the no longer needed prefix arg. --- aclocal.m4 | 24 +++++++++++++----------- configure.ac | 26 +++++++++++++------------- tools/make_makefiles | 4 ++-- 3 files changed, 28 insertions(+), 26 deletions(-) diff --git a/aclocal.m4 b/aclocal.m4 index 7e8e208..62f73da 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -165,21 +165,23 @@ AC_CONFIG_FILES([$1])]) dnl **** Create a makefile from config.status **** dnl -dnl Usage: WINE_CONFIG_MAKEFILE(file,deps,prefix,var,enable) +dnl Usage: WINE_CONFIG_MAKEFILE(file,deps,var,enable) dnl AC_DEFUN([WINE_CONFIG_MAKEFILE], -[m4_pushdef([ac_dir],m4_bpatsubst([$1],[^$$3/?\(.*$/\)?Makefile$],[\2]))dnl +[m4_pushdef([ac_dir],m4_bpatsubst([$1],[^$\(.*$/\)?Makefile$],[\2]))dnl +m4_pushdef([ac_name],m4_bpatsubst(ac_dir,[.*/$.*$$],[\1]))dnl m4_ifval(ac_dir,[ALL_MAKEFILES="$ALL_MAKEFILES \\ $1"]) -AS_VAR_PUSHDEF([ac_enable],m4_default([$5],[enable_]ac_dir))dnl -m4_ifval([$4],[test "x$ac_enable" != xno]m4_foreach([ac_var],[$4],[ && ac_var="$ac_var \\ +AS_VAR_PUSHDEF([ac_enable],m4_default([$4],[enable_]ac_name))dnl +m4_ifval([$3],[test "x$ac_enable" != xno]m4_foreach([ac_var],[$3],[ && ac_var="$ac_var \\ ac_dir"])) AS_VAR_POPDEF([ac_enable])dnl ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS -m4_ifval(ac_dir,m4_ifval([$3],,[\$(RECURSE_TARGETS:%=ac_dir/%) ac_dir: $1 \$(MAKEDEP) -]))[$1: ]m4_ifval([$2],[$1.in $2],[$1.in]) config.status" +m4_ifval(ac_dir,[\$(RECURSE_TARGETS:%=ac_dir/%) ac_dir: $1 \$(MAKEDEP) +])[$1: ]m4_ifval([$2],[$1.in $2],[$1.in]) config.status" AC_CONFIG_FILES([$1])dnl -m4_popdef([ac_dir])]) +m4_popdef([ac_dir])dnl +m4_popdef([ac_name])]) dnl **** Create a dll makefile from config.status **** dnl @@ -209,7 +211,7 @@ dlls/$1/lib$3.$STATIC_IMPLIBEXT dlls/$1/lib$3.cross.a: dlls/$1/Makefile dummy [dlls/$1/lib$3.$IMPLIBEXT dlls/$1/lib$3.cross.a: dlls/$1/$1.spec dlls/$1/Makefile @cd dlls/$1 && \$(MAKE) \`basename \$[@]\`]) ])dlls/$1 dlls/$1/__install__ dlls/$1/__install-lib__ dlls/$1/__install-dev__: __builddeps__" -WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makedll.rules],[],[ALL_DLL_DIRS],m4_default([$2],[enable_$1]))]) +WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makedll.rules],[ALL_DLL_DIRS],[$2])]) dnl **** Create a program makefile from config.status **** dnl @@ -218,7 +220,7 @@ dnl AC_DEFUN([WINE_CONFIG_PROGRAM], [ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS programs/$1 programs/$1/__install__ programs/$1/__install-lib__: __builddeps__" -WINE_CONFIG_MAKEFILE([programs/$1/Makefile],[programs/Makeprog.rules],[],[$2],m4_default([$3],[enable_$1]))]) +WINE_CONFIG_MAKEFILE([programs/$1/Makefile],[programs/Makeprog.rules],[$2],[$3])]) dnl **** Create a test makefile from config.status **** dnl @@ -238,7 +240,7 @@ ac_name.exe: \$(TOPOBJDIR)/$1/ac_name.exe$DLLEXT ac_name.rc: echo \"ac_name.exe TESTRES \\\"ac_name.exe\\\"\" >\$[@] || (\$(RM) \$[@] && false) ac_name.res: ac_name.rc ac_name.exe" -WINE_CONFIG_MAKEFILE([$1/Makefile],[Maketest.rules],[],[ALL_TEST_DIRS],[enable_tests])dnl +WINE_CONFIG_MAKEFILE([$1/Makefile],[Maketest.rules],[ALL_TEST_DIRS],[enable_tests])dnl m4_popdef([ac_suffix])dnl m4_popdef([ac_name])]) @@ -254,7 +256,7 @@ dlls/$1: tools/widl tools/winebuild tools/winegcc include dlls/$1/__install__ dlls/$1/__install-dev__: dlls/$1 dlls/$1/lib$1.cross.a: dlls/$1/Makefile tools/widl tools/winebuild tools/winegcc include dummy @cd dlls/$1 && \$(MAKE) \`basename \$[@]\`" -WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makeimplib.rules],[],[ALL_STATICLIB_DIRS],[enable_$1])]) +WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makeimplib.rules],[ALL_STATICLIB_DIRS])]) dnl **** Add a message to the list displayed at the end **** dnl diff --git a/configure.ac b/configure.ac index 68954d5..ed0fa0a 100644 --- a/configure.ac +++ b/configure.ac @@ -2607,14 +2607,14 @@ WINE_CONFIG_TEST(dlls/xinput1_3/tests) WINE_CONFIG_DLL(xinput9_1_0) WINE_CONFIG_DLL(xmllite) WINE_CONFIG_TEST(dlls/xmllite/tests) -WINE_CONFIG_MAKEFILE([documentation/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) -WINE_CONFIG_MAKEFILE([fonts/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) -WINE_CONFIG_MAKEFILE([include/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([documentation/Makefile],[Make.rules],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([fonts/Makefile],[Make.rules],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([include/Makefile],[Make.rules],[ALL_TOP_DIRS]) WINE_CONFIG_MAKEFILE([libs/port/Makefile],[Make.rules]) WINE_CONFIG_MAKEFILE([libs/wine/Makefile],[Make.rules]) WINE_CONFIG_MAKEFILE([libs/wpp/Makefile],[Make.rules]) -WINE_CONFIG_MAKEFILE([loader/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) -WINE_CONFIG_MAKEFILE([programs/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([loader/Makefile],[Make.rules],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([programs/Makefile],[Make.rules],[ALL_TOP_DIRS]) WINE_CONFIG_PROGRAM(cacls,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) WINE_CONFIG_PROGRAM(clock,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) WINE_CONFIG_PROGRAM(cmd,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) @@ -2670,14 +2670,14 @@ WINE_CONFIG_PROGRAM(winver,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) WINE_CONFIG_PROGRAM(wordpad,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) WINE_CONFIG_PROGRAM(write,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) WINE_CONFIG_PROGRAM(xcopy,[ALL_PROGRAM_DIRS,ALL_INSTALL_DIRS]) -WINE_CONFIG_MAKEFILE([server/Makefile],[Make.rules],[],[ALL_TOP_DIRS]) -WINE_CONFIG_MAKEFILE([tools/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/widl/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/winebuild/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/winedump/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/winegcc/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/wmc/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) -WINE_CONFIG_MAKEFILE([tools/wrc/Makefile],[Make.rules],[],[ALL_TOOL_DIRS],[enable_tools]) +WINE_CONFIG_MAKEFILE([server/Makefile],[Make.rules],[ALL_TOP_DIRS]) +WINE_CONFIG_MAKEFILE([tools/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/widl/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/winebuild/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/winedump/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/winegcc/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/wmc/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) +WINE_CONFIG_MAKEFILE([tools/wrc/Makefile],[Make.rules],[ALL_TOOL_DIRS],enable_tools) dnl End of auto-generated output commands dnl Don't install in bin dir for Windows build diff --git a/tools/make_makefiles b/tools/make_makefiles index 5762b52..04a351c 100755 --- a/tools/make_makefiles +++ b/tools/make_makefiles @@ -391,11 +391,11 @@ sub update_makefiles(@) } elsif ($file =~ /^tools.*\/Makefile$/) { - push @lines, "WINE_CONFIG_MAKEFILE([$file],[$rules],[],[ALL_TOOL_DIRS],[enable_tools])\n"; + push @lines, "WINE_CONFIG_MAKEFILE([$file],[$rules],[ALL_TOOL_DIRS],enable_tools)\n"; } elsif ($file =~ /^[^\/]*\/Makefile$/) { - push @lines, "WINE_CONFIG_MAKEFILE([$file],[$rules],[],[ALL_TOP_DIRS])\n"; + push @lines, "WINE_CONFIG_MAKEFILE([$file],[$rules],[ALL_TOP_DIRS])\n"; } else {

1 0

Alexandre Julliard : makefiles: Simplify the WINE_CONFIG_DLL macro by defining some local variables.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 3be8b409ae360e87fed6795616a6c9904da17471 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3be8b409ae360e87fed679561… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 19:39:38 2010 +0100 makefiles: Simplify the WINE_CONFIG_DLL macro by defining some local variables. --- aclocal.m4 | 40 +++++++++++++++++++++++----------------- configure | 32 ++++++++++++-------------------- 2 files changed, 35 insertions(+), 37 deletions(-) diff --git a/aclocal.m4 b/aclocal.m4 index 62f73da..d7b41c1 100644 --- a/aclocal.m4 +++ b/aclocal.m4 @@ -188,30 +188,36 @@ dnl dnl Usage: WINE_CONFIG_DLL(name,enable,implib,implibsrc) dnl AC_DEFUN([WINE_CONFIG_DLL], -[m4_ifval([$3],[m4_ifval([$2],[test "x$[$2]" = xno || ])ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ - dlls/$1/lib$3.$IMPLIBEXT[]dnl -m4_if($1,$3,,[ \\ - dlls/lib$3.$IMPLIBEXT])[]dnl -m4_ifval([$4],[ \\ - dlls/$1/lib$3.$STATIC_IMPLIBEXT])" +[m4_pushdef([ac_implib],m4_ifval([$3],[dlls/$1/lib$3.$IMPLIBEXT]))dnl +m4_pushdef([ac_crosslib],m4_ifval([$3],[dlls/$1/lib$3.cross.a]))dnl +m4_pushdef([ac_staticlib],m4_ifval([$4],[dlls/$1/lib$3.$STATIC_IMPLIBEXT]))dnl +m4_pushdef([ac_symlink],m4_if([$1],[$3],,[dlls/lib$3.$IMPLIBEXT]))dnl +m4_ifval(ac_implib,[m4_ifval([$2],[test "x$[$2]" = xno || ])ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ + ac_implib[]dnl +m4_ifval(ac_symlink,[ \\ + ac_symlink])[]dnl +m4_ifval(ac_staticlib,[ \\ + ac_staticlib])" ])ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS -m4_ifval([$3],[dlls/$1/lib$3.$IMPLIBEXT m4_ifval([$4],[dlls/$1/lib$3.$STATIC_IMPLIBEXT ])dlls/$1/lib$3.cross.a: tools/widl tools/winebuild tools/winegcc include -m4_if($1,$3,,[dlls/lib$3.a: dlls/$1/lib$3.a - \$(RM) \$[@] && \$(LN_S) $1/lib$3.a \$[@] -dlls/lib$3.cross.a: dlls/$1/lib$3.cross.a +m4_ifval(ac_implib,[ac_implib m4_ifval(ac_staticlib,[ac_staticlib ])ac_crosslib: tools/widl tools/winebuild tools/winegcc include +m4_ifval(ac_symlink,[ac_symlink: ac_implib + \$(RM) \$[@] && \$(LN_S) $1/lib$3.$IMPLIBEXT \$[@] +dlls/lib$3.cross.a: ac_crosslib \$(RM) \$[@] && \$(LN_S) $1/lib$3.cross.a \$[@] -dlls/lib$3.def: dlls/$1/lib$3.def - \$(RM) \$[@] && \$(LN_S) $1/lib$3.def \$[@] clean:: - \$(RM) dlls/lib$3.def -])m4_ifval([$4],[dlls/$1/lib$3.def: dlls/$1/$1.spec dlls/$1/Makefile + \$(RM) ac_symlink +])m4_ifval(ac_staticlib,[dlls/$1/lib$3.def: dlls/$1/$1.spec dlls/$1/Makefile @cd dlls/$1 && \$(MAKE) \`basename \$[@]\` -dlls/$1/lib$3.$STATIC_IMPLIBEXT dlls/$1/lib$3.cross.a: dlls/$1/Makefile dummy +ac_staticlib ac_crosslib: dlls/$1/Makefile dummy @cd dlls/$1 && \$(MAKE) \`basename \$[@]\`], -[dlls/$1/lib$3.$IMPLIBEXT dlls/$1/lib$3.cross.a: dlls/$1/$1.spec dlls/$1/Makefile +[ac_implib ac_crosslib: dlls/$1/$1.spec dlls/$1/Makefile @cd dlls/$1 && \$(MAKE) \`basename \$[@]\`]) ])dlls/$1 dlls/$1/__install__ dlls/$1/__install-lib__ dlls/$1/__install-dev__: __builddeps__" -WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makedll.rules],[ALL_DLL_DIRS],[$2])]) +WINE_CONFIG_MAKEFILE([dlls/$1/Makefile],[dlls/Makedll.rules],[ALL_DLL_DIRS],[$2])dnl +m4_popdef([ac_implib])dnl +m4_popdef([ac_crosslib])dnl +m4_popdef([ac_staticlib])dnl +m4_popdef([ac_symlink])]) dnl **** Create a program makefile from config.status **** dnl diff --git a/configure b/configure index ac049b5..1062066 100755 --- a/configure +++ b/configure @@ -14919,14 +14919,12 @@ ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ dlls/libd3dx9.$IMPLIBEXT" ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS dlls/d3dx9_36/libd3dx9.$IMPLIBEXT dlls/d3dx9_36/libd3dx9.cross.a: tools/widl tools/winebuild tools/winegcc include -dlls/libd3dx9.a: dlls/d3dx9_36/libd3dx9.a - \$(RM) \$@ && \$(LN_S) d3dx9_36/libd3dx9.a \$@ +dlls/libd3dx9.$IMPLIBEXT: dlls/d3dx9_36/libd3dx9.$IMPLIBEXT + \$(RM) \$@ && \$(LN_S) d3dx9_36/libd3dx9.$IMPLIBEXT \$@ dlls/libd3dx9.cross.a: dlls/d3dx9_36/libd3dx9.cross.a \$(RM) \$@ && \$(LN_S) d3dx9_36/libd3dx9.cross.a \$@ -dlls/libd3dx9.def: dlls/d3dx9_36/libd3dx9.def - \$(RM) \$@ && \$(LN_S) d3dx9_36/libd3dx9.def \$@ clean:: - \$(RM) dlls/libd3dx9.def + \$(RM) dlls/libd3dx9.$IMPLIBEXT dlls/d3dx9_36/libd3dx9.$IMPLIBEXT dlls/d3dx9_36/libd3dx9.cross.a: dlls/d3dx9_36/d3dx9_36.spec dlls/d3dx9_36/Makefile @cd dlls/d3dx9_36 && \$(MAKE) \`basename \$@\` dlls/d3dx9_36 dlls/d3dx9_36/__install__ dlls/d3dx9_36/__install-lib__ dlls/d3dx9_36/__install-dev__: __builddeps__" @@ -16431,14 +16429,12 @@ test "x$enable_win16" = xno || ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ dlls/libkernel.$IMPLIBEXT" ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS dlls/krnl386.exe16/libkernel.$IMPLIBEXT dlls/krnl386.exe16/libkernel.cross.a: tools/widl tools/winebuild tools/winegcc include -dlls/libkernel.a: dlls/krnl386.exe16/libkernel.a - \$(RM) \$@ && \$(LN_S) krnl386.exe16/libkernel.a \$@ +dlls/libkernel.$IMPLIBEXT: dlls/krnl386.exe16/libkernel.$IMPLIBEXT + \$(RM) \$@ && \$(LN_S) krnl386.exe16/libkernel.$IMPLIBEXT \$@ dlls/libkernel.cross.a: dlls/krnl386.exe16/libkernel.cross.a \$(RM) \$@ && \$(LN_S) krnl386.exe16/libkernel.cross.a \$@ -dlls/libkernel.def: dlls/krnl386.exe16/libkernel.def - \$(RM) \$@ && \$(LN_S) krnl386.exe16/libkernel.def \$@ clean:: - \$(RM) dlls/libkernel.def + \$(RM) dlls/libkernel.$IMPLIBEXT dlls/krnl386.exe16/libkernel.$IMPLIBEXT dlls/krnl386.exe16/libkernel.cross.a: dlls/krnl386.exe16/krnl386.exe16.spec dlls/krnl386.exe16/Makefile @cd dlls/krnl386.exe16 && \$(MAKE) \`basename \$@\` dlls/krnl386.exe16 dlls/krnl386.exe16/__install__ dlls/krnl386.exe16/__install-lib__ dlls/krnl386.exe16/__install-dev__: __builddeps__" @@ -20229,14 +20225,12 @@ ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ dlls/libwinspool.$IMPLIBEXT" ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS dlls/winspool.drv/libwinspool.$IMPLIBEXT dlls/winspool.drv/libwinspool.cross.a: tools/widl tools/winebuild tools/winegcc include -dlls/libwinspool.a: dlls/winspool.drv/libwinspool.a - \$(RM) \$@ && \$(LN_S) winspool.drv/libwinspool.a \$@ +dlls/libwinspool.$IMPLIBEXT: dlls/winspool.drv/libwinspool.$IMPLIBEXT + \$(RM) \$@ && \$(LN_S) winspool.drv/libwinspool.$IMPLIBEXT \$@ dlls/libwinspool.cross.a: dlls/winspool.drv/libwinspool.cross.a \$(RM) \$@ && \$(LN_S) winspool.drv/libwinspool.cross.a \$@ -dlls/libwinspool.def: dlls/winspool.drv/libwinspool.def - \$(RM) \$@ && \$(LN_S) winspool.drv/libwinspool.def \$@ clean:: - \$(RM) dlls/libwinspool.def + \$(RM) dlls/libwinspool.$IMPLIBEXT dlls/winspool.drv/libwinspool.$IMPLIBEXT dlls/winspool.drv/libwinspool.cross.a: dlls/winspool.drv/winspool.drv.spec dlls/winspool.drv/Makefile @cd dlls/winspool.drv && \$(MAKE) \`basename \$@\` dlls/winspool.drv dlls/winspool.drv/__install__ dlls/winspool.drv/__install-lib__ dlls/winspool.drv/__install-dev__: __builddeps__" @@ -20559,14 +20553,12 @@ ALL_IMPORT_LIBS="$ALL_IMPORT_LIBS \\ dlls/libxinput.$IMPLIBEXT" ALL_MAKEFILE_DEPENDS="$ALL_MAKEFILE_DEPENDS dlls/xinput1_3/libxinput.$IMPLIBEXT dlls/xinput1_3/libxinput.cross.a: tools/widl tools/winebuild tools/winegcc include -dlls/libxinput.a: dlls/xinput1_3/libxinput.a - \$(RM) \$@ && \$(LN_S) xinput1_3/libxinput.a \$@ +dlls/libxinput.$IMPLIBEXT: dlls/xinput1_3/libxinput.$IMPLIBEXT + \$(RM) \$@ && \$(LN_S) xinput1_3/libxinput.$IMPLIBEXT \$@ dlls/libxinput.cross.a: dlls/xinput1_3/libxinput.cross.a \$(RM) \$@ && \$(LN_S) xinput1_3/libxinput.cross.a \$@ -dlls/libxinput.def: dlls/xinput1_3/libxinput.def - \$(RM) \$@ && \$(LN_S) xinput1_3/libxinput.def \$@ clean:: - \$(RM) dlls/libxinput.def + \$(RM) dlls/libxinput.$IMPLIBEXT dlls/xinput1_3/libxinput.$IMPLIBEXT dlls/xinput1_3/libxinput.cross.a: dlls/xinput1_3/xinput1_3.spec dlls/xinput1_3/Makefile @cd dlls/xinput1_3 && \$(MAKE) \`basename \$@\` dlls/xinput1_3 dlls/xinput1_3/__install__ dlls/xinput1_3/__install-lib__ dlls/xinput1_3/__install-dev__: __builddeps__"

1 0

Alexandre Julliard : makefiles: Move all the recursion rules to the top-level makefile.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 93f03393875d241d339289ac559921684b76efc6 URL: http://source.winehq.org/git/wine.git/?a=commit;h=93f03393875d241d339289ac5… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 20:27:52 2010 +0100 makefiles: Move all the recursion rules to the top-level makefile. --- Make.rules.in | 82 ++------------------------ Makefile.in | 135 ++++++++++++++++++++++++++++++++++---------- Maketest.rules.in | 2 + dlls/Makedll.rules.in | 14 +---- programs/Makeprog.rules.in | 7 -- 5 files changed, 116 insertions(+), 124 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=93f03393875d241d33928…

1 0

Alexandre Julliard : makefiles: Build the programs directly from the top-level makefile.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 9463687f78977058e6f114fdbd56f8a2bd754a47 URL: http://source.winehq.org/git/wine.git/?a=commit;h=9463687f78977058e6f114fdb… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 19:33:39 2010 +0100 makefiles: Build the programs directly from the top-level makefile. --- Makefile.in | 14 +- aclocal.m4 | 4 +- configure | 400 +++++++++++++++++++++++++++++++++++--------------- configure.ac | 104 +++++++------- programs/Makefile.in | 4 +- tools/make_makefiles | 2 +- 6 files changed, 345 insertions(+), 183 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=9463687f78977058e6f11…

1 0

Alexandre Julliard : makefiles: Auto-generate the list of all subdirectories too.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 52c400e97e031b8c7d0556e83119c67c2576e314 URL: http://source.winehq.org/git/wine.git/?a=commit;h=52c400e97e031b8c7d0556e83… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 19:57:02 2010 +0100 makefiles: Auto-generate the list of all subdirectories too. --- Makefile.in | 20 +- aclocal.m4 | 4 +- configure | 2158 +++++++++++++++++++++++++++++----------------------------- configure.ac | 2 +- 4 files changed, 1085 insertions(+), 1099 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=52c400e97e031b8c7d055…

1 0

Alexandre Julliard : makefiles: Build the dlls directly from the top-level makefile.
by Alexandre Julliard Feb. 11, 2010

Feb. 11, 2010

Module: wine Branch: master Commit: 5acb431511727317f0274bba82b21a74967e9750 URL: http://source.winehq.org/git/wine.git/?a=commit;h=5acb431511727317f0274bba8… Author: Alexandre Julliard <julliard(a)winehq.org> Date: Wed Feb 10 19:32:31 2010 +0100 makefiles: Build the dlls directly from the top-level makefile. --- Makefile.in | 19 +- aclocal.m4 | 10 +- configure | 1920 +++++++++++++++++++++++++++++++++------------ configure.ac | 1 - dlls/Makefile.in | 23 - documentation/Makefile.in | 8 +- 6 files changed, 1435 insertions(+), 546 deletions(-) Diff: http://source.winehq.org/git/wine.git/?a=commitdiff;h=5acb431511727317f0274…

1 0

Michael Stefaniuc : kernel32: Avoid using HIWORD on types that are 64bit on Win64.
by Alexandre Julliard Feb. 10, 2010

Feb. 10, 2010

Module: wine Branch: master Commit: 3f37e33497ab6f61cffee53a0f479516404c2dd7 URL: http://source.winehq.org/git/wine.git/?a=commit;h=3f37e33497ab6f61cffee53a0… Author: Michael Stefaniuc <mstefani(a)redhat.de> Date: Wed Feb 10 11:43:37 2010 +0100 kernel32: Avoid using HIWORD on types that are 64bit on Win64. --- dlls/kernel32/atom.c | 3 ++- dlls/kernel32/except.c | 2 +- dlls/kernel32/module.c | 2 +- 3 files changed, 4 insertions(+), 3 deletions(-) diff --git a/dlls/kernel32/atom.c b/dlls/kernel32/atom.c index d6caec2..a8deb5e 100644 --- a/dlls/kernel32/atom.c +++ b/dlls/kernel32/atom.c @@ -37,6 +37,7 @@ #include "kernel_private.h" #define MAX_ATOM_LEN 255 +#define IS_INTATOM(x) (((ULONG_PTR)(x) >> 16) == 0) /****************************************************************** * get_local_table @@ -88,7 +89,7 @@ BOOL WINAPI InitAtomTable( DWORD entries ) */ static inline BOOL check_integral_atom( const void* ptr, ATOM* patom) { - if (HIWORD( ptr )) return FALSE; + if (!IS_INTATOM( ptr )) return FALSE; if ((*patom = LOWORD( ptr )) >= MAXINTATOM) { SetLastError( ERROR_INVALID_PARAMETER ); diff --git a/dlls/kernel32/except.c b/dlls/kernel32/except.c index d6dea99..6ed88ad 100644 --- a/dlls/kernel32/except.c +++ b/dlls/kernel32/except.c @@ -133,7 +133,7 @@ static int format_exception_msg( const EXCEPTION_POINTERS *ptr, char *buffer, in rec->ExceptionInformation[0]); break; case EXCEPTION_WINE_STUB: - if (HIWORD(rec->ExceptionInformation[1])) + if ((ULONG_PTR)rec->ExceptionInformation[1] >> 16) len = snprintf( buffer, size, "Unimplemented function %s.%s called", (char *)rec->ExceptionInformation[0], (char *)rec->ExceptionInformation[1] ); else diff --git a/dlls/kernel32/module.c b/dlls/kernel32/module.c index 96a240e..2b8f11e 100644 --- a/dlls/kernel32/module.c +++ b/dlls/kernel32/module.c @@ -1032,7 +1032,7 @@ FARPROC WINAPI GetProcAddress( HMODULE hModule, LPCSTR function ) if (!hModule) hModule = NtCurrentTeb()->Peb->ImageBaseAddress; - if (HIWORD(function)) + if ((ULONG_PTR)function >> 16) { ANSI_STRING str;

1 0