[Ctk-developers] CTK Update to latest DCMTK

Julien Finet julien.finet at kitware.com
Thu Aug 4 14:36:23 EDT 2011


Hi Michael,
Using 085525e643cab5ac826823c7a31694**37761fe0d3 seems to fix the warning:
http://my.cdash.org/index.php?project=CTK&display=project&filtercount=3&showfilters=1&filtercombine=and&field1=buildname/string&compare1=61&value1=WindowsXP-VS2008Express-QT4.6.3-Debug&field2=site/string&compare2=61&value2=erna.kitware&field3=buildstamp/string&compare3=61&value3=20110804-1757-Experimental&collapse=0

I think you can go ahead and update the default git SHA1 tag for DCMTK.

Thanks,
Julien.

On Thu, Aug 4, 2011 at 12:39 PM, OFFIS DICOM Team <dicom at offis.de> wrote:

> Hi Julien,
>
> Am 04.08.2011 17:00, schrieb Julien Finet:
>
>  Hi Michael,
>>
>> There is still a mktemp warning on Windows: *
>>
>>
>> 3>51>..\..\..\DCMTK\dcmnet\**apps\storescp.cc(87) : warning
>> C4273:'mktemp'
>> : inconsistent dll linkage
>>
>
> thanks for pointing me to that issue. It just showed up since CTK uses the
> CMake-determined compiler flags by disabling the DCMTK pre-defined flags
> (which caused a problem due to /MT flag). So we did not notice it before,
> sorry.
>
> It was fixed yesterday. So if you switch to a newer revision of DCMTK in
> DCMTK.cmake the warning will be gone.
>
> The minimum commit is 901ba8f1e50ef91627b0cdbb623835**d64e3f6fe4.
> The latest  commit is 085525e643cab5ac826823c7a31694**37761fe0d3 (two
> later).
>
> Works for me in CTK under Win XP and Win 7 (32 bit), as well as on Kubuntu
> 64 bit. If someone from the CTK core team can confirm that, he might
> upgrade
> the DCMTK commit hash or I do if I have an OK. Getting more careful now :-)
>
> HTH,
> Michael
>
>
> --
> OFFIS DICOM Team, Escherweg 2, 26121 Oldenburg, Germany
> E-Mail: dicom at offis.de, URL: http://dicom.offis.de
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://public.kitware.com/pipermail/ctk-developers/attachments/20110804/d64935c7/attachment.html>


More information about the Ctk-developers mailing list