[Cmake-commits] [cmake-commits] king committed cmTarget.cxx 1.285 1.286

cmake-commits at cmake.org cmake-commits at cmake.org
Wed Jan 13 13:00:34 EST 2010


Update of /cvsroot/CMake/CMake/Source
In directory public:/mounts/ram/cvs-serv27951/Source

Modified Files:
	cmTarget.cxx 
Log Message:
Name Cygwin DLLs with SOVERSION, not VERSION

Cygwin versions .dll files by putting the version number in the file
name.  Our fix to issue #3571 taught CMake to do this, but it used the
VERSION target property.  It is better to use the SOVERSION property
since that is the interface (rather than implementation) version.

Change based on patch from issue #10122.


Index: cmTarget.cxx
===================================================================
RCS file: /cvsroot/CMake/CMake/Source/cmTarget.cxx,v
retrieving revision 1.285
retrieving revision 1.286
diff -C 2 -d -r1.285 -r1.286
*** cmTarget.cxx	14 Dec 2009 22:46:29 -0000	1.285
--- cmTarget.cxx	13 Jan 2010 18:00:29 -0000	1.286
***************
*** 3090,3094 ****
  
    // Name shared libraries with their version number on some platforms.
!   if(const char* version = this->GetProperty("VERSION"))
      {
      if(this->GetType() == cmTarget::SHARED_LIBRARY && !implib &&
--- 3090,3094 ----
  
    // Name shared libraries with their version number on some platforms.
!   if(const char* soversion = this->GetProperty("SOVERSION"))
      {
      if(this->GetType() == cmTarget::SHARED_LIBRARY && !implib &&
***************
*** 3096,3100 ****
        {
        outBase += "-";
!       outBase += version;
        }
      }
--- 3096,3100 ----
        {
        outBase += "-";
!       outBase += soversion;
        }
      }



More information about the Cmake-commits mailing list