<div dir="ltr">For reference, a solution has been proposed: <a href="https://github.com/commontk/CTK/issues/329">https://github.com/commontk/CTK/issues/329</a><div style>It should be merged shortly. </div></div><div class="gmail_extra">
<br><br><div class="gmail_quote">On Thu, May 9, 2013 at 6:22 AM, Sascha Zelzer <span dir="ltr"><<a href="mailto:s.zelzer@dkfz-heidelberg.de" target="_blank">s.zelzer@dkfz-heidelberg.de</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div bgcolor="#FFFFFF" text="#000000">
<div>Hi JC,<br>
<br>
Did you have time to have a look at the broken support for the
official DCMTK snapshot inside CTK? If I remember correctly, we
settled on the decision to support the latest official DCMTK
snapshot together with the patched versions of DCMTK contained in
the commontk Github repositories (eventually by "ifdef"s in the
CTK code).<br>
<br>
As Marco pointed out in his E-Mail, it looks like your merge
containing changes to the FindDCMTK.cmake script broke the support
for the DCMTK snapshot...<br>
<br>
It would be great to solve this issue before the Hackfest.<br>
<br>
Thanks,<br>
<br>
Sascha<div><div class="h5"><br>
<br>
On 04/19/2013 02:40 PM, Jean-Christophe Fillion-Robin wrote:<br>
</div></div></div><div><div class="h5">
<blockquote type="cite">
<div dir="ltr">Hi Matt,
<div><br>
</div>
<div>I don't know which change or who did these
changes. But if we can find out, I could provide guidance so
that we can add so configure check in order to make the
feature introspection of DCMTK robust. That way, we would be
able to include / exclude part of the CTKDCMTK library
appropriately. </div>
<div><br>
</div>
<div>Thanks</div>
<div>Jc</div>
</div>
<div class="gmail_extra"><br>
<br>
<div class="gmail_quote">On Fri, Apr 19, 2013 at 2:50 AM,
Clarkson, Matt <span dir="ltr"><<a href="mailto:m.clarkson@ucl.ac.uk" target="_blank">m.clarkson@ucl.ac.uk</a>></span>
wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<div style="word-wrap:break-word">
Hi there CTK-ers,
<div><br>
</div>
<div>please can anyone help me out with a CTK / DCMTK
problem? It looks to me like recent changes to CTK stop
CTK working with the official DCMTK snapshots which are
used in MITK.</div>
<div>Is that right?</div>
<div><br>
</div>
<div>If anyone can shed light on the way forward, Im happy
to try and help here.</div>
<div><br>
</div>
<div>Thanks</div>
<div><br>
</div>
<div>Matt</div>
<div><br>
</div>
<div>
<div>
<div><br>
<div>Begin forwarded message:</div>
<br>
<blockquote type="cite">
<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span style="font-family:'Helvetica';font-size:medium;color:rgba(0,0,0,1.0)"><b>From:
</b></span><span style="font-family:'Helvetica';font-size:medium">Marco
Nolden <<a href="mailto:m.nolden@dkfz-heidelberg.de" target="_blank">m.nolden@dkfz-heidelberg.de</a>><br>
</span></div>
<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span style="font-family:'Helvetica';font-size:medium;color:rgba(0,0,0,1.0)"><b>Subject:
</b></span><span style="font-family:'Helvetica';font-size:medium"><b>Re:
[mitk-users] MITK with Latest CTK - problem
with DCMTK</b><br>
</span></div>
<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span style="font-family:'Helvetica';font-size:medium;color:rgba(0,0,0,1.0)"><b>Date:
</b></span><span style="font-family:'Helvetica';font-size:medium">13
April 2013 22:38:37 BST<br>
</span></div>
<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span style="font-family:'Helvetica';font-size:medium;color:rgba(0,0,0,1.0)"><b>To:
</b></span><span style="font-family:'Helvetica';font-size:medium">"Clarkson,
Matt" <<a href="mailto:m.clarkson@ucl.ac.uk" target="_blank">m.clarkson@ucl.ac.uk</a>><br>
</span></div>
<div style="margin-top:0px;margin-right:0px;margin-bottom:0px;margin-left:0px"><span style="font-family:'Helvetica';font-size:medium;color:rgba(0,0,0,1.0)"><b>Cc:
</b></span><span style="font-family:'Helvetica';font-size:medium">mitk-users
<<a href="mailto:mitk-users@lists.sourceforge.net" target="_blank">mitk-users@lists.sourceforge.net</a>><br>
</span></div>
<br>
<div>Hi Matt,<br>
<br>
MITK's super build is using
dcmtk-3.6.1_20120222, but there is a newer
snapshot here [1] and from a quick look through
the Changes we could upgrade if it compiles with
MITK. If you like to try it you could replace
the URL in CMakeExternals/DCMTK.cmake (and
remove or adapt the URL_MD5 line).<br>
<br>
I'm not sure though that the CTK change you
mention will work with that snapshot. CTK by
default uses a forked version of DCMTK and there
has been some activity lately. IIRC we agreed
that CTK _should_ always build with the official
DCMTK snapshots as well, but I'm afraid that
doesn't get tested regularly. So if that is
broken we should contact JC if the changes can
be made in a compatible way.<br>
<br>
If you could confirm that MITK builds with the
newer snapshot we could upgrade this on our
side. Unfortunately I can't test it myself at
the moment.<br>
<br>
Upgrading the CTK version can be more tricky,
Sascha could maybe comment on that. If you are
just interested in the slider tick change, we
could also use a special version, did that
before.<br>
<br>
Best,<br>
<br>
Marco<br>
<br>
[1]: <a href="http://dicom.offis.de/download/dcmtk/snapshot/dcmtk-3.6.1_20121102.tar.gz" target="_blank">
http://dicom.offis.de/download/dcmtk/snapshot/dcmtk-3.6.1_20121102.tar.gz</a><br>
<br>
<br>
<br>
Am 04.04.2013 21:06, schrieb Clarkson, Matt:<br>
<blockquote type="cite">Hi there,<br>
<br>
Following the merging of CTK issue #313 to CTK
master (CTK commit 099a88bb24), I then tried
to update my MITK to compile with this version
of CTK, so I could pick up Miklos's new slider
tick interval.<br>
However, in the CTK master branch, commit
52d9539f5e, which is 3 before the latest,
causes CTK to not build using the DCMTK
version that MITK is using.<br>
<br>
So, it looks to me like MITK will need to
consider upgrading DCMTK before they can move
to the latest CTK, which is the reason I post
here, instead of the CTK mailing list.<br>
<br>
However, I don't really understand what the
reason for the CTK change is, or why CTK has a
different fork of DCMTK, or what exactly is
the version of DCMTK that MITK is using -
dcmtk-3.6.1_20120222.tar.gz, so Im a bit
stuck.<br>
<br>
Can anyone in MITK-land shed a light on a way
forward?<br>
<br>
Thanks<br>
<br>
Matt<br>
<br>
<br>
<br>
------------------------------------------------------------------------------<br>
Minimize network downtime and maximize team
effectiveness.<br>
Reduce network management and security
costs.Learn how to hire<br>
the most talented Cisco Certified
professionals. Visit the<br>
Employer Resources Portal<br>
<a href="http://www.cisco.com/web/learning/employer_resources/index.html" target="_blank">http://www.cisco.com/web/learning/employer_resources/index.html</a><br>
_______________________________________________<br>
mitk-users mailing list<br>
<a href="mailto:mitk-users@lists.sourceforge.net" target="_blank">mitk-users@lists.sourceforge.net</a><br>
<a href="https://lists.sourceforge.net/lists/listinfo/mitk-users" target="_blank">https://lists.sourceforge.net/lists/listinfo/mitk-users</a><br>
</blockquote>
<br>
<br>
</div>
</blockquote>
</div>
<br>
</div>
</div>
</div>
<br>
_______________________________________________<br>
Ctk-developers mailing list<br>
<a href="mailto:Ctk-developers@commontk.org" target="_blank">Ctk-developers@commontk.org</a><br>
<a href="http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers" target="_blank">http://public.kitware.com/cgi-bin/mailman/listinfo/ctk-developers</a><br>
<br>
</blockquote>
</div>
<br>
<br clear="all">
<div><br>
</div>
-- <br>
<a href="tel:%2B1%20919%20869%208849" value="+19198698849" target="_blank">+1 919 869 8849</a><br>
</div>
</blockquote>
<br>
</div></div></div>
</blockquote></div><br><br clear="all"><div><br></div>-- <br>+1 919 869 8849<br>
</div>