[opensource-dev] Review Request: OPEN-112 Windows debug build fails at run time because of an absent openjpeg.dll.

[Vadim ProductEngine]

-----------------------------------------------------------
This is an automatically generated e-mail. To reply, visit:
http://codereview.secondlife.com/r/445/
-----------------------------------------------------------

(Updated Sept. 13, 2011, 5:10 a.m.)


Review request for Viewer.


Summary (updated)
-------

Note: In code comments are for code review only and will be removed.
Debug builds have openjpegd.dll and it is a dependency for some of the unit tests. However, the viewer (secondlife-bin.exe) has openjpeg.dll dependency. It is the same debug library but, without the 'd' in its name. So, there are two or more dependencies of the same library with different names. This can be tested by building a debug configuration and invoking the viewer and then, the error appears about the missing openjpeg.dll. Then, change the name of openjpegd.dll to openjpeg.dll and invoke the viewer again. This time it should run correctly and present the log in page. Modifications are need to add a copy instruction so that both named versions of the debug library are present in the openjpeg archive. Additional modifications to the viewer are needed to place openjpeg.dll properly; while leaving openjpegd.dll in place for unit tests.


This addresses bug OPEN-112.
    http://jira.secondlife.com/browse/OPEN-112


Diffs
-----

  doc/contributions.txt a95b822cf2c2 
  indra/cmake/Copy3rdPartyLibs.cmake a95b822cf2c2 
  indra/newview/CMakeLists.txt a95b822cf2c2 
  indra/newview/viewer_manifest.py a95b822cf2c2 

Diff: http://codereview.secondlife.com/r/445/diff


Testing
-------

Built Debug configuration with revised local library. Copy of revised library is here: http://bitbucket.org/NickyP/viewer-development-open-112/downloads/openjpeg-1.4-windows-20110819.tar.bz2 
MD5 hash is 271a8e9253efaef9884181cd58984dd9.
Logged in to aditi mesh city and walked around. No problems noted.


Thanks,

Nicky

-------------- next part --------------
An HTML attachment was scrubbed...
URL: http://lists.secondlife.com/pipermail/opensource-dev/attachments/20110913/96b565d0/attachment.htm