Sounds good. Rather than my originally proposed fix...<div><span class="Apple-style-span" style="border-collapse: collapse; "><div><br></div><div>  SET( VERDICT_LIBRARY_PROPERTIES</div><div><div>    VERSION "${VTK_VERSION}"</div>
<div>    SOVERSION "${VTK_MAJOR_VERSION}.${VTK_MINOR_VERSION}"</div><div>    )</div><div><br></div><div><br></div><div>...I am leaning toward this one instead:</div><div><div><br></div><div>  SET(VERDICT_LIBRARY_PROPERTIES ${VTK_LIBRARY_PROPERTIES})</div>
<div><div><br></div><div>(that way, it will remain consistent with VTK_LIBRARY_PROPERTIES in the event that somebody makes a change to them...)</div><div><br></div><div>I will commit this later today or tomorrow morning unless somebody objects.</div>
<div><br></div><div><br></div><div>Thanks to all for the discussion,</div><div>David Cole</div><div><br></div></div></div></div></span><br><div class="gmail_quote">On Wed, Apr 8, 2009 at 7:42 PM, Philippe P. Pebay <span dir="ltr"><<a href="mailto:pppebay@sandia.gov">pppebay@sandia.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex;">Thanks Clint.<br>
<br>
David, I think we can proceed with making libverdict consistent with the rest of VTK.<br><font color="#888888">
<br>
P</font><div><div></div><div class="h5"><br>
<br>
<br>
Clinton Stimpson wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
David, your suggested fix looks fine to me, and should work fine with our use of standalone verdict.<br>
The whole VERDICT_LIBRARY_PROPERTIES was part of my request a while back.<br>
<br>
Clint<br>
<br>
Philippe P. Pebay wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hello David<br>
<br>
I don't remember why we did that when we integrated Verdict into VTK. There might have been a reason, which I forgot. Some folks are using the standalone version of Verdict. I am forwarding your question to the verdict-dev list.<br>

<br>
Thanks<br>
P<br>
<br>
David Cole wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hey all,<br>
<br>
<br>
The VTK libraries (with the exception of verdict) all use the following chunk of code from the main VTK CMakeLists.txt file to specify thier VERSION and SOVERSION properties:<br>
<br>
  # Append the library version information to the library target<br>
  # properties.  A parent project may set its own properties and/or may<br>
  # block this.<br>
  IF(NOT VTK_NO_LIBRARY_VERSION)<br>
    SET(VTK_LIBRARY_PROPERTIES ${VTK_LIBRARY_PROPERTIES}<br>
      VERSION "${VTK_VERSION}"<br>
      SOVERSION "${VTK_MAJOR_VERSION}.${VTK_MINOR_VERSION}"<br>
      )<br>
  ENDIF(NOT VTK_NO_LIBRARY_VERSION)<br>
<br>
<br>
The verdict library instead uses this chunk of code from VTK/Utilities/CMakeLists.txt:<br>
<br>
  SET( VERDICT_LIBRARY_PROPERTIES VERSION "vtk${VTK_VERSION}" )<br>
<br>
<br>
I (think I) understand the "vtk" is intended to distinguish it from when it is built as a standalone library and not as part of VTK...<br>
But the net effect of this discrepancy is that the install tree contains many libraries with three files each like this...:<br>
  libvtktiff.so -> libvtktiff.so.5.4<br>
  libvtktiff.so.5.4 -> libvtktiff.so.5.4.0<br>
  libvtktiff.so.5.4.0<br>
<br>
...and then the verdict library with only two files that do not follow the same convention:<br>
  libvtkverdict.so -> libvtkverdict.so.vtk5.4.0<br>
  libvtkverdict.so.vtk5.4.0<br>
<br>
<br>
So here's the question:<br>
Isn't the name "libvtkverdict" sufficient to distinguish the library from its standalone non-VTK library...? Can I make libvtkverdict consistent with the other VTK libraries by changing the code to this: ??<br>

<br>
  SET( VERDICT_LIBRARY_PROPERTIES<br>
    VERSION "${VTK_VERSION}"<br>
    SOVERSION "${VTK_MAJOR_VERSION}.${VTK_MINOR_VERSION}"<br>
    )<br>
<br>
Or is there some other reason that I am missing that it needs to be the way it is?<br>
<br>
<br>
Thanks for any insight / feedback......<br>
<br>
David Cole<br>
<br>
</blockquote>
<br>
</blockquote>
<br>
<br>
</blockquote>
<br>
<br></div></div><div><div></div><div class="h5">
-- <br>
Philippe Pébay<br>
Sandia National Laboratories<br>
<br>
<br>
</div></div></blockquote></div><br></div>