[Bug 27273] Microsoft Office Communicator 2007 R2 does not install, doesn't recognize MSXML 6.0 SP1 as installed (ACTION_FileVersionMatches needs to check version for languages)

http://bugs.winehq.org/show_bug.cgi?id=27273

Anastasius Focht  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |UNCONFIRMED
          Component|-unknown                    |msi
         Resolution|WONTFIX                     |
            Summary|Microsoft Office            |Microsoft Office
                   |Communicator 2007 R2 does   |Communicator 2007 R2 does
                   |not install, doesn't        |not install, doesn't
                   |recognize MSXML 6.0 SP1 as  |recognize MSXML 6.0 SP1 as
                   |installed                   |installed
                   |                            |(ACTION_FileVersionMatches
                   |                            |needs to check version for
                   |                            |languages)

--- Comment #10 from Anastasius Focht  2012-04-23 02:12:34 CDT ---
Hello again,

well monday morning :|

Actually should detect Wine builtin as MSXML6 SP1 using file signature match
(registry will fail, see bug 30078) so this is a legitimate bug.
As previously written, the file version should match as Wine builtin has the
correct version resource for SP1.

--- snip ---
0009:trace:msi:MSI_DatabaseOpenViewW L"select * from DrLocator where Signature_
= 'DirectorySearchMSXML60'" 0x32eed8 
...
0009:trace:msi:ACTION_SearchDirectory 0x158e98, 0x32f8ec,
L"C:\windows\system32\", 0, 0x32f920
0009:trace:msi:ACTION_RecurseSearchDirectory Searching directory
L"C:\windows\system32\" for file L"msxml6.dll", depth 0
0009:Call ntdll.RtlAllocateHeap(00110000,00000000,00000040) ret=7ecb2df7
0009:Ret  ntdll.RtlAllocateHeap() retval=00164fc8 ret=7ecb2df7
0009:Call shlwapi.PathAddBackslashW(00164fc8 L"C:\windows\system32\")
ret=7ecb4ca3
0009:Ret  shlwapi.PathAddBackslashW() retval=00164ff0 ret=7ecb4ca3
0009:Call KERNEL32.FindFirstFileW(00164fc8
L"C:\windows\system32\msxml6.dll",0032eed8) ret=7ecb4cd3
0009:Ret  KERNEL32.FindFirstFileW() retval=00166700 ret=7ecb4cd3
0009:fixme:msi:ACTION_FileVersionMatches : need to check version for languages
L"0" 
--- snip ---

Source:
http://source.winehq.org/git/wine.git/blob/0f3eb5f689542f04240c8763f724203dc...

--- snip ---
 616 static UINT ACTION_FileVersionMatches(const MSISIGNATURE *sig, LPCWSTR
filePath,
 617  BOOL *matches)
 618 {
 619     UINT rc = ERROR_SUCCESS;
 620 
 621     *matches = FALSE;
 622     if (sig->Languages)
 623     {
 624         FIXME(": need to check version for languages %s\n",
 625          debugstr_w(sig->Languages));
 626     }
 627     else
 628     {
...
 680     }
 681     return rc;
 682 }
--- snip ---

$ wine --version
wine-1.5.2-191-gd080774

Regards