Excellent. I will revert my hack and set DCMTK_FORCE_FPIC_ON_UNIX to ON instead.<br><br>Will also fix the SHA1.<br><br>Thanks<br>Jc<br><br><div class="gmail_quote">On Mon, Aug 1, 2011 at 5:41 PM, OFFIS DICOM Team <span dir="ltr"><<a href="mailto:dicom@offis.de">dicom@offis.de</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">Hi,<br>
<br>
Am 01.08.2011 20:41, schrieb Jean-Christophe Fillion-Robin:<div class="im"><br>
<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">
We already have a WITH_THREADS flag in DCMTK which adds libpthread if<br>
existing/necessary. Usually, with threads is turned on.<br>
<br>
Does it mean that turning WITH_THREAD OFF won't append the /MT or /MTd<br>
flags ?<br>
</blockquote>
<br></div>
No, on Windows we usually use /MT and /MTd in DCMTK. For CTK, we now (since<br>
today) use the flags automatically determined by CMake, i.e. /MD and /MDd<br>
for compilation. In both cases we do not care about whether<br>
DCMTK_WITH_THREADS is on or off. Both, /MT(d) and /MD(d) flags, select a<br>
microsoft runtime library including threaded versions, as far as I (and you<br>
probably) know:<br>
<br>
<a href="http://msdn.microsoft.com/en-us/library/2kzt1wy3%28v=vs.71%29.aspx" target="_blank">http://msdn.microsoft.com/en-<u></u>us/library/2kzt1wy3(v=vs.71).<u></u>aspx</a><br>
<br>
If WITH_THREADS is off, DCMTK's thread classes like OFThread are not<br>
compiled, and pthread and the like are not added to the linker.<br>
<br>
<blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;"><div class="im">
I could commit it any time/immediately in DCMTK. My colleague noted that<br>
we should take care that we only put -fPIC into effect if it is<br>
understood by the compiler and needed, so I wonder whether "IF(UNIX" is<br>
a sufficient guard. Any hints or recommendations?<br>
<br>
<br></div>
You could for example do: [...]<br>
</blockquote>
<br>
I inserted a flag DCMTK_FORCE_FPIC_ON_UNIX in DCMTK [1] which can be set<br>
from CTK's DCMTK.cmake via CMAKE_ARGS. If this makes sense, I can change the<br>
DCMTK.cmake accordingly. Also, I removed mktemp from the tparser.cc [2] so<br>
the warning an unix should be gone now.<br>
<br>
Thanks for helping!<br>
<br>
Michael<br>
<br>
[1]<br>
<a href="http://git.dcmtk.org/web?p=dcmtk.git;a=commitdiff;h=09db15ff595da6c35330fd7f63669aeb9952e015" target="_blank">http://git.dcmtk.org/web?p=<u></u>dcmtk.git;a=commitdiff;h=<u></u>09db15ff595da6c35330fd7f63669a<u></u>eb9952e015</a><br>


<br>
[2] <a href="http://git.dcmtk.org/web?p=dcmtk.git;a=commitdiff;h=96ce7c46b48245512385170bfb72ec3cdc32a3a6" target="_blank">http://git.dcmtk.org/web?p=<u></u>dcmtk.git;a=commitdiff;h=<u></u>96ce7c46b48245512385170bfb72ec<u></u>3cdc32a3a6</a><div>

<div></div><div class="h5"><br>
<br>
-- <br>
OFFIS DICOM Team, Escherweg 2, 26121 Oldenburg, Germany<br>
E-Mail: <a href="mailto:dicom@offis.de" target="_blank">dicom@offis.de</a>, URL: <a href="http://dicom.offis.de" target="_blank">http://dicom.offis.de</a><br>
</div></div></blockquote></div><br><br clear="all"><br>-- <br>+1 919 869 8849<br><br>