[Bug 31640] New: hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Bug #: 31640
           Summary: hugin's enfuse.exe crashes without native vcomp100.dll
                    (purist)
           Product: Wine
           Version: 1.5.12
          Platform: x86
        OS/Version: Linux
            Status: NEW
          Severity: normal
          Priority: P2
         Component: vcomp
        AssignedTo: [e-mail]
        ReportedBy: [e-mail]
    Classification: Unclassified

Andre Hentschel suggested this test case.

Download and run the installer from
http://sourceforge.net/projects/hugin/files/hugin/hugin-2011.2/HuginSetup_20...
Then:
cd ~/".wine/drive_c/Program Files/Hugin/bin"
rm vcomp100.dll
wget http://www.realworldimagery.com/Samples/peopl911.tif
mv peopl911.tif a.tif
wget http://www.realworldimagery.com/Samples/VEHIC200.tif
mv VEHIC200.tif b.tif
wine enfuse.exe --output=c.tif a.tif b.tif

Result:
unimplemented function vcomp100.dll._vcomp_fork

FWIW, winedump -j import says it needs the following symbols from vcomp100:

omp_get_dynamic
omp_get_max_threads
omp_get_nested
omp_get_num_procs
omp_get_num_threads
omp_get_thread_num
omp_set_dynamic
omp_set_nested
_vcomp_fork
_vcomp_for_static_end
_vcomp_for_static_simple_init

Let's make this bug about the 32 bit version of enfuse.exe.
Once that's working, let's do another bug for the 64 bit version
and/or similar bugs for enblend.exe.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

--- Comment #1 from Dan Kegel  2012-09-07 23:21:35 CDT ---
BTW those two .tif files are just the first random .tif files I found
with alpha channels; hugin complains if you give it files without alpha
channels.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |download
                URL|                            |http://hugin.sourceforge.ne
                   |                            |t/

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |source

--- Comment #2 from Dan Kegel  2012-09-12 22:27:56 CDT ---
The source for the plugin that uses openmp is at
http://sourceforge.net/scm/?type=hg&group_id=123407

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Andrew Aladjev  changed:

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

--- Comment #3 from Andrew Aladjev  2012-09-29 06:38:42 CDT ---
wine 1.5.14 + cities xl 2012:

wine: Call from 0x7b83c999 to unimplemented function vcomp.dll._vcomp_fork,
aborting
fixme:dbghelp:elf_search_auxv can't find symbol in module
wine: Unimplemented function vcomp.dll._vcomp_fork called at address 0x7b83c999
(thread 0036), starting debugger...
Unhandled exception: unimplemented function vcomp.dll._vcomp_fork called in
32-bit code (0x7b83c999).
fixme:dbghelp:i386_stack_walk new PC=7dbbb6e4 different from Eip=7dbba641

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Dan Kegel  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
           Keywords|                            |patch

--- Comment #4 from Dan Kegel  2012-10-02 11:27:02 CDT ---
The patch series at http://kegel.com/wine/openmp/ seems to make enfuse.exe
happy.

I submitted the first few, will keep at it until they're committed.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Qian Hong  changed:

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

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Borislav Sabev  changed:

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

--- Comment #5 from Borislav Sabev  ---
Created attachment 48208
  --> http://bugs.winehq.org/attachment.cgi?id=48208
CitiesXLPlatinumDebugOutput

This bug is still affecting Wine 1.7.15 x86 and CitiesXL Platinum.

It is still unimplemented, I just wanted to add some more information to it.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Jarkko K  changed:

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

--- Comment #6 from Jarkko K  ---
Gothic 3 demo crashes on unimplemented function vcomp.dll._vcomp_fork (purist)
https://bugs.winehq.org/show_bug.cgi?id=31643

Risen crashes on unimplemented function vcomp.dll.omp_init_lock
https://bugs.winehq.org/show_bug.cgi?id=26688

Unhandled exception: page fault on write access in CitiesXL_Platinum.exe
https://bugs.winehq.org/show_bug.cgi?id=33662

What's the patch status?

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

--- Comment #7 from Dan Kegel  ---
I stopped working on it when Alexandre said he didn't want a stub
implementation.  I think I know how to do an implementation complete enough
for him, but haven't had time to do it.  If someone else wanted to help finish
it, I could work with them a bit on it.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

--- Comment #8 from Jarkko K  ---
I send email to someone who might have enough knowledge...

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Sebastian Lackner  changed:

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

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

--- Comment #9 from Sebastian Lackner  ---
I've just added a rebased version of Dan Kegels patches to Wine-Staging:
https://github.com/wine-compholio/wine-staging/tree/master/patches/vcomp-Stu...

There was still a mistake in the spec files which is also fixed in the version
above. I was planning to implement various kernel32/ntdll threadpool functions
during the next days/weeks, when this is done they might be much easier to make
vcomp multithreaded.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Sebastian Lackner  changed:

What    |Removed                     |Added
----------------------------------------------------------------------------
      Fixed by SHA1|                            |6f34d0fe1f67632cc24b304a536
                   |                            |a9790e9365ec0
             Status|NEW                         |RESOLVED
         Resolution|---                         |FIXED

--- Comment #10 from Sebastian Lackner  ---
As far as I can tell, the app works fine in purist mode now. The download links
for the *.tif files in the original description are dead, but I tested with a
couple of other images and it works fine.

The last required function _vcomp_for_static_simple_init was implemented in
http://source.winehq.org/git/wine.git/commit/6f34d0fe1f67632cc24b304a536a979...,
so I'll use that for the Fixed SHA1.

[Bug 31640] hugin's enfuse.exe crashes without native vcomp100.dll (purist)

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

Alexandre Julliard  changed:

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

--- Comment #11 from Alexandre Julliard  ---
Closing bugs fixed in 1.7.49.