This is an automated email from the git hooks/post-receive script. It was
generated because a ref change was pushed to the repository containing
the project "CMake".

The branch, next has been updated
       via  75f46e9143a5022405dffb5c1abd10355b4fee09 (commit)
       via  4f3b97063f724f5e0a2902e8d2c104012b52b7f5 (commit)
      from  3565b58ea8f6f54c9c8d3f6187361def72944672 (commit)

Those revisions listed above that are new to this repository have
not appeared on any other notification email; so we list those
revisions in full, below.

- Log -----------------------------------------------------------------
https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=75f46e9143a5022405dffb5c1abd10355b4fee09
commit 75f46e9143a5022405dffb5c1abd10355b4fee09
Merge: 3565b58 4f3b970
Author:     Domen Vrankar <domen.vran...@gmail.com>
AuthorDate: Mon May 23 14:24:15 2016 -0400
Commit:     CMake Topic Stage <kwro...@kitware.com>
CommitDate: Mon May 23 14:24:15 2016 -0400

    Merge topic 'cpack-deb-component-depends' into next
    
    4f3b9706 CPack/Deb prevent accidental component dependencies

diff --cc Help/release/dev/cpack-deb-imporvements.rst
index d3cb52c,9e45aa5..63312b5
--- a/Help/release/dev/cpack-deb-imporvements.rst
+++ b/Help/release/dev/cpack-deb-imporvements.rst
@@@ -11,10 -11,14 +11,11 @@@ cpack-deb-imporvement
  * The :module:`CPackDeb` module learned how to generate dependencies between
    Debian packages if multi-component setup is used and
    :variable:`CPACK_COMPONENT_<compName>_DEPENDS` variables are set.
-   This breaks compatibility with previous versions.
+   For backward compatibility this feature is disabled by default. See
+   :variable:`CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS`.
  
 -* The :module:`CPackDeb` module learned how to set the package release number
 -  (``DebianRevisionNumber`` in package file name).  See
 -  :variable:`CPACK_DEBIAN_PACKAGE_RELEASE`.
 -
 -* The :module:`CPackDeb` module now generates properly-named Debian packages::
 +* The :module:`CPackDeb` module learned how to set custom package file names
 +  including how to generate properly-named Debian packages::
  
      
<PackageName>_<VersionNumber>-<DebianRevisionNumber>_<DebianArchitecture>.deb
  

https://cmake.org/gitweb?p=cmake.git;a=commitdiff;h=4f3b97063f724f5e0a2902e8d2c104012b52b7f5
commit 4f3b97063f724f5e0a2902e8d2c104012b52b7f5
Author:     Domen Vrankar <domen.vran...@gmail.com>
AuthorDate: Mon May 23 20:21:32 2016 +0200
Commit:     Domen Vrankar <domen.vran...@gmail.com>
CommitDate: Mon May 23 20:21:32 2016 +0200

    CPack/Deb prevent accidental component dependencies
    
    Prevent accidental inter component dependency setting
    since this is a breaking feature in situations where
    another CPack module is already using the feature.
    It should be enabled if desired since it can cause
    issues when upgrading only one of the components
    for e.g. configuration instead of all the components
    at once.

diff --git a/Help/release/dev/cpack-deb-imporvements.rst 
b/Help/release/dev/cpack-deb-imporvements.rst
index 4fd35b8..9e45aa5 100644
--- a/Help/release/dev/cpack-deb-imporvements.rst
+++ b/Help/release/dev/cpack-deb-imporvements.rst
@@ -11,7 +11,8 @@ cpack-deb-imporvements
 * The :module:`CPackDeb` module learned how to generate dependencies between
   Debian packages if multi-component setup is used and
   :variable:`CPACK_COMPONENT_<compName>_DEPENDS` variables are set.
-  This breaks compatibility with previous versions.
+  For backward compatibility this feature is disabled by default. See
+  :variable:`CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS`.
 
 * The :module:`CPackDeb` module learned how to set the package release number
   (``DebianRevisionNumber`` in package file name).  See
diff --git a/Modules/CPackDeb.cmake b/Modules/CPackDeb.cmake
index d93d924..70bf81d 100644
--- a/Modules/CPackDeb.cmake
+++ b/Modules/CPackDeb.cmake
@@ -100,6 +100,13 @@
 #
 #    set(CPACK_DEBIAN_PACKAGE_DEPENDS "libc6 (>= 2.3.1-6), libc6 (< 2.4)")
 #
+# .. variable:: CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS
+#
+#  Sets inter component dependencies if listed with
+#  :variable:`CPACK_COMPONENT_<compName>_DEPENDS` variables.
+#
+#  * Mandatory : NO
+#  * Default   : -
 #
 # .. variable:: CPACK_DEBIAN_PACKAGE_MAINTAINER
 #
@@ -713,20 +720,23 @@ function(cpack_deb_prepare_package_vars)
         endif()
       endif()
     endforeach()
-    set(COMPONENT_DEPENDS "")
-    foreach (_PACK ${CPACK_COMPONENT_${_local_component_name}_DEPENDS})
-      get_component_package_name(_PACK_NAME "${_PACK}")
+
+    if(CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS)
+      set(COMPONENT_DEPENDS "")
+      foreach (_PACK ${CPACK_COMPONENT_${_local_component_name}_DEPENDS})
+        get_component_package_name(_PACK_NAME "${_PACK}")
+        if(COMPONENT_DEPENDS)
+          set(COMPONENT_DEPENDS "${_PACK_NAME} (= 
${CPACK_DEBIAN_PACKAGE_VERSION}), ${COMPONENT_DEPENDS}")
+        else()
+          set(COMPONENT_DEPENDS "${_PACK_NAME} (= 
${CPACK_DEBIAN_PACKAGE_VERSION})")
+        endif()
+      endforeach()
       if(COMPONENT_DEPENDS)
-        set(COMPONENT_DEPENDS "${_PACK_NAME} (= 
${CPACK_DEBIAN_PACKAGE_VERSION}), ${COMPONENT_DEPENDS}")
-      else()
-        set(COMPONENT_DEPENDS "${_PACK_NAME} (= 
${CPACK_DEBIAN_PACKAGE_VERSION})")
-      endif()
-    endforeach()
-    if(COMPONENT_DEPENDS)
-      if(CPACK_DEBIAN_PACKAGE_DEPENDS)
-        set(CPACK_DEBIAN_PACKAGE_DEPENDS "${COMPONENT_DEPENDS}, 
${CPACK_DEBIAN_PACKAGE_DEPENDS}")
-      else()
-        set(CPACK_DEBIAN_PACKAGE_DEPENDS "${COMPONENT_DEPENDS}")
+        if(CPACK_DEBIAN_PACKAGE_DEPENDS)
+          set(CPACK_DEBIAN_PACKAGE_DEPENDS "${COMPONENT_DEPENDS}, 
${CPACK_DEBIAN_PACKAGE_DEPENDS}")
+        else()
+          set(CPACK_DEBIAN_PACKAGE_DEPENDS "${COMPONENT_DEPENDS}")
+        endif()
       endif()
     endif()
   endif()
diff --git 
a/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend1.cmake.in 
b/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend1.cmake.in
index d207bcc..dc038fa 100644
--- a/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend1.cmake.in
+++ b/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend1.cmake.in
@@ -4,6 +4,7 @@
 
 if(CPACK_GENERATOR MATCHES "DEB")
    set(CPACK_DEB_COMPONENT_INSTALL "ON")
+   set(CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS "ON")
 endif()
 
 #
diff --git 
a/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend2.cmake.in 
b/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend2.cmake.in
index 803720a..1aa7d01 100644
--- a/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend2.cmake.in
+++ b/Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend2.cmake.in
@@ -4,6 +4,7 @@
 
 if(CPACK_GENERATOR MATCHES "DEB")
    set(CPACK_DEB_COMPONENT_INSTALL "ON")
+   set(CPACK_DEBIAN_ENABLE_COMPONENT_DEPENDS "ON")
 endif()
 
 #

-----------------------------------------------------------------------

Summary of changes:
 Help/release/dev/cpack-deb-imporvements.rst        |    3 +-
 Modules/CPackDeb.cmake                             |   36 +++++++++++++-------
 .../MyLibCPackConfig-components-depend1.cmake.in   |    1 +
 .../MyLibCPackConfig-components-depend2.cmake.in   |    1 +
 4 files changed, 27 insertions(+), 14 deletions(-)


hooks/post-receive
-- 
CMake
_______________________________________________
Cmake-commits mailing list
Cmake-commits@cmake.org
http://public.kitware.com/mailman/listinfo/cmake-commits

Reply via email to