<html>
  <head>
    <meta content="text/html; charset=ISO-8859-1"
      http-equiv="Content-Type">
  </head>
  <body bgcolor="#FFFFFF" text="#000000">
    <div class="moz-cite-prefix">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<br>
      <br>
      On 04/19/2013 02:40 PM, Jean-Christophe Fillion-Robin wrote:<br>
    </div>
    <blockquote
cite="mid:CANLz8R5VGVLT0nhNLpY_hxKQUCGLdwyi-+wC=cn8qsm51nXYXw@mail.gmail.com"
      type="cite">
      <div dir="ltr">Hi Matt, 
        <div><br>
        </div>
        <div style="">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 style=""><br>
        </div>
        <div style="">Thanks</div>
        <div style="">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 moz-do-not-send="true"
              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 moz-do-not-send="true"
                            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 moz-do-not-send="true"
                            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 moz-do-not-send="true"
                            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 moz-do-not-send="true"
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 moz-do-not-send="true"
                            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 moz-do-not-send="true"
                            href="mailto:mitk-users@lists.sourceforge.net"
                            target="_blank">mitk-users@lists.sourceforge.net</a><br>
                          <a moz-do-not-send="true"
                            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 moz-do-not-send="true"
              href="mailto:Ctk-developers@commontk.org">Ctk-developers@commontk.org</a><br>
            <a moz-do-not-send="true"
              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>
        +1 919 869 8849<br>
      </div>
    </blockquote>
    <br>
  </body>
</html>