[Bug 36941] New: Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Bug ID: 36941
           Summary: Provide separate debug/symbol info files (.pdb) for
                    32-bit and 64-bit Wine-Gecko package (unstripped
                    32-bit 'xul.dll' too large to be mapped)
           Product: Wine-gecko
           Version: unspecified
          Hardware: x86
                OS: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: wine-gecko-unknown
          Assignee: [e-mail]
          Reporter: [e-mail]

Hello folks,

unstripped 32-bit Wine-Gecko binaries in "C:\windows\system32\gecko\2.24"
(32-bit WINEPREFIX) or "C:\windows\syswow64\gecko\2.24" (64-bit WINEPREFIX)
get overwritten each time Wine-Gecko is loaded.

Download:
http://sourceforge.net/projects/wine/files/Wine%20Gecko/2.24/wine_gecko-2.24...

--- snip ---
...
0025:Call advapi32.RegOpenKeyW(80000002,7ccb1540
L"Software\Wine\MSHTML",0113be14) ret=7cc50ab7
0025:Ret  advapi32.RegOpenKeyW() retval=00000000 ret=7cc50ab7
0025:Call advapi32.RegOpenKeyA(000000d8,7ccb0738 "2.24",0113bde0) ret=7cc509e2
0025:Ret  advapi32.RegOpenKeyA() retval=00000000 ret=7cc509e2
0025:Call advapi32.RegQueryValueExW(000000e0,7ccb151e
L"GeckoPath",00000000,0113bde8,0113be44,0113bde4) ret=7cc50a2a
0025:Ret  advapi32.RegQueryValueExW() retval=00000000 ret=7cc50a2a
0025:Call advapi32.RegCloseKey(000000e0) ret=7cc50a43
0025:Ret  advapi32.RegCloseKey() retval=00000000 ret=7cc50a43
0025:Call KERNEL32.CreateFileW(0113bb8c
L"C:\windows\system32\gecko\2.24\wine_gecko\\VERSION",80000000,00000001,00000000,00000003,00000080,00000000)
ret=7cc50804
0025:Ret  KERNEL32.CreateFileW() retval=000000e0 ret=7cc50804
0025:Call KERNEL32.ReadFile(000000e0,0113bb0c,00000080,0113bb08,00000000)
ret=7cc50892
0025:Ret  KERNEL32.ReadFile() retval=00000001 ret=7cc50892
0025:Call KERNEL32.CloseHandle(000000e0) ret=7cc508ae
0025:Ret  KERNEL32.CloseHandle() retval=00000001 ret=7cc508ae
0025:Call KERNEL32.GetEnvironmentVariableW(7ccb1468 L"PATH",0113b8b0,00000104)
ret=7cc4fbe0
0025:Ret  KERNEL32.GetEnvironmentVariableW() retval=00000037 ret=7cc4fbe0
0025:Call KERNEL32.SetEnvironmentVariableW(7ccb1468 L"PATH",0113b8b0
L"C:\windows\system32;C:\windows;C:\windows\system32\wbem;C:\windows\system32\gecko\2.24\wine_gecko\")
ret=7cc4fc3c
0025:Ret  KERNEL32.SetEnvironmentVariableW() retval=00000001 ret=7cc4fc3c
0025:Call KERNEL32.SetEnvironmentVariableW(7ccb1480
L"XPCOM_DEBUG_BREAK",7ccb14a4 L"warn") ret=7cc4fc57
0025:Ret  KERNEL32.SetEnvironmentVariableW() retval=00000001 ret=7cc4fc57
0025:Call KERNEL32.SetEnvironmentVariableW(7ccb14c0
L"NSPR_LOG_MODULES",0113b888 L"all:5") ret=7cc4fd25
0025:Ret  KERNEL32.SetEnvironmentVariableW() retval=00000001 ret=7cc4fd25
0025:Call KERNEL32.LoadLibraryExW(0113bafc
L"C:\windows\system32\gecko\2.24\wine_gecko\\xul.dll",00000000,00000008)
ret=7cc4fdf3
0025:Ret  KERNEL32.LoadLibraryExW() retval=00000000 ret=7cc4fdf3
0025:Call advapi32.RegCloseKey(000000d8) ret=7cc50afe
0025:Ret  advapi32.RegCloseKey() retval=00000000 ret=7cc50afe
0025:Call KERNEL32.GetSystemDirectoryW(0113bbb0,000000f7) ret=7cc4f9ec
0025:Ret  KERNEL32.GetSystemDirectoryW() retval=00000013 ret=7cc4f9ec
0025:Call ntdll.RtlAllocateHeap(00110000,00000000,00000074) ret=7cc4e900
0025:Ret  ntdll.RtlAllocateHeap() retval=001e0fb8 ret=7cc4e900
0025:Call KERNEL32.CreateProcessW(0113bbb0
L"C:\windows\system32\control.exe",001e0fb8
L"C:\windows\system32\control.exe appwiz.cpl
install_gecko",00000000,00000000,00000000,00000000,00000000,00000000,0113bdb8,0113bdfc)
ret=7cc4fb53
0025:trace:process:create_process_impl app
L"C:\windows\system32\control.exe" cmdline
L"C:\windows\system32\control.exe appwiz.cpl install_gecko"
0025:trace:process:open_exe_file looking for
L"C:\windows\system32\control.exe"
0025:trace:process:create_process_impl starting
L"C:\windows\system32\control.exe" as Win32 binary (0x10000000-0x10003000,
arch 014c)
0025:trace:process:create_process_impl started process pid 0035 tid 0036
0025:Ret  KERNEL32.CreateProcessW() retval=00000001 ret=7cc4fb53 
--- snip ---

The huge size of unstripped 32-bit 'xul.dll' (1.7 GB) causes the failure to map
the dll into 32-bit address space (load base: 0x69C40000).

This causes the (re)installation which overwrites the debug version with the
stripped release build.

The Wine Gecko "Debug Info" section in Wiki seems also out of date:
http://wiki.winehq.org/Gecko (paths are wrong and mentioned 'winetricks' recipe
no longer valid)

Regards

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

--- Comment #1 from Anastasius Focht  ---
Hello folks,

still present as of Wine Gecko 2.34 release.

http://wiki.winehq.org/Gecko

http://sourceforge.net/projects/wine/files/Wine%20Gecko/2.34/

$ wine --version
wine-1.7.30-23-gf5e3281

Regards

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
                 CC|                            |[e-mail]

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

--- Comment #2 from Austin English  ---
Created attachment 51407
  --> https://bugs.winehq.org/attachment.cgi?id=51407
patches

Here's a patchset against gecko's make_package to support building with MSVC.

Note that to do so, you'll need the build deps for Firefox, see
https://developer.mozilla.org/en-US/docs/Mozilla/Developer_guide/Build_Instr....
After that, with this patchset, doing ./wine/make_package -dbg

will build a debug tarball with pdb's. Replace wine_gecko, and put
wine_gecko-2.36-dbg in drive_c.

Prebuilt binaries are available at:
http://austinenglish.com/for_focht/wine_gecko-2.36-x86-msvc.tar.bz2

(once the patchset is merged, I'll rebuild and put on SF).

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

--- Comment #3 from Austin English  ---
(In reply to Austin English from comment #2)
> Created attachment 51407 [details]
> patches
>
> Here's a patchset against gecko's make_package to support building with MSVC.

Some of that's been committed:
http://sourceforge.net/p/wine/wine-gecko/ci/6932f792690bce197abf536c1d8c9057...
http://sourceforge.net/p/wine/wine-gecko/ci/17f208deadfa9785fb12621d44023816...
http://sourceforge.net/p/wine/wine-gecko/ci/b050f88e4f68d881865d5777aa8e4608...

So now wine-gecko will compile out of the box with MSVC. Still need the
packaging fixed though (wip).

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #51407|0                           |1
        is obsolete|                            |

--- Comment #4 from Austin English  ---
Created attachment 51462
  --> https://bugs.winehq.org/attachment.cgi?id=51462
packaging patch

This patch adds support for tar'ing up the .pdb's. Waiting on Jacek to commit
to the wine-gecko repo.

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
  Attachment #51462|0                           |1
        is obsolete|                            |

--- Comment #5 from Austin English  ---
Created attachment 51471
  --> https://bugs.winehq.org/attachment.cgi?id=51471
try 2

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #6 from Austin English  ---
(In reply to Austin English from comment #5)
> Created attachment 51471 [details]
> try 2

http://sourceforge.net/p/wine/wine-gecko/ci/08e9b5134eee6890d18439e81c6db3da...

and 32-bit msvc dlls/pdb's:
https://sourceforge.net/projects/wine/files/Wine%20Gecko/2.36/wine_gecko-2.3...
https://sourceforge.net/projects/wine/files/Wine%20Gecko/2.36/wine_gecko-2.3...

to use:
cd ~/.wine/drive_c/windows/system32/gecko
rm -rf wine_gecko
tar xvjf wine_gecko-2.36-x86-dbg-msvc.tar.bz2

cd ~/.wine/drive_c
tar xjvf wine_gecko-2.36-x86-dbg-msvc-pdb.tar.bz2

I'm going to mark this fixed. Please file a separate bug for 64-bit and CC me,
I'll take a look as time permits.

[Bug 36941] Provide separate debug/symbol info files (.pdb) for 32-bit and 64-bit Wine-Gecko package (unstripped 32-bit 'xul.dll' too large to be mapped)

https://bugs.winehq.org/show_bug.cgi?id=36941

Austin English  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
             Status|RESOLVED                    |CLOSED

--- Comment #7 from Austin English  ---
Closing.