Re: [cmake-developers] [CMake 0011944]: CPackDeb: Support dependencies between components/Debian packages

2015-05-08 Thread Raffi Enficiaud

Le 08/05/15 07:54, Domen Vrankar a écrit :

Would you please add
set( CPACK_DEBIAN_PACKAGE_DEBUG ON)

to the file
MyLibCPackConfig-splitted-components-depend2.cmake.in

so that we also have the debug logs?


I currently don't have access to my computer so I'll send you that in
about a week.


Sorry I forgot about this...

Attached is verbose output and patch that I was using (slightly
modified patch that you provided rebased to current master).



Hi,

Thanks, I will have a look this afternoon (sorry for the delay),

Raffi


--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


[cmake-developers] [CMake 0015560]: EXTERNALPROJECT_ADD does not longer build from SOURCE_DIR

2015-05-08 Thread Mantis Bug Tracker

The following issue has been SUBMITTED. 
== 
https://public.kitware.com/Bug/view.php?id=15560 
== 
Reported By:LCID Fire
Assigned To:
== 
Project:CMake
Issue ID:   15560
Category:   CMake
Reproducibility:always
Severity:   feature
Priority:   normal
Status: new
== 
Date Submitted: 2015-05-08 06:05 EDT
Last Modified:  2015-05-08 06:05 EDT
== 
Summary:EXTERNALPROJECT_ADD does not longer build from
SOURCE_DIR
Description: 
When specifying SOURCE_DIR, I get the same as bug
https://public.kitware.com/Bug/view.php?id=10346:

error: no download info for 'libtorrent' -- please specify
  existing/non-empty SOURCE_DIR or one of URL, CVS_REPOSITORY and CVS_MODULE,
  SVN_REPOSITORY, GIT_REPOSITORY, HG_REPOSITORY or DOWNLOAD_COMMAND

Has this regressed?

Steps to Reproduce: 
ExternalProject_Add( libtorrent
SOURCE_DIR lib/libtorrent
)
== 

Issue History 
Date ModifiedUsername   FieldChange   
== 
2015-05-08 06:05 LCID Fire  New Issue
==

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


Re: [cmake-developers] [CMake 0011944]: CPackDeb: Support dependencies between components/Debian packages

2015-05-08 Thread Raffi Enficiaud

Le 08/05/15 07:54, Domen Vrankar a écrit :

Would you please add
set( CPACK_DEBIAN_PACKAGE_DEBUG ON)

to the file
MyLibCPackConfig-splitted-components-depend2.cmake.in

so that we also have the debug logs?


I currently don't have access to my computer so I'll send you that in
about a week.


Sorry I forgot about this...

Attached is verbose output and patch that I was using (slightly
modified patch that you provided rebased to current master).



Hi,

In fact, CPACK_DEBIAN_PACKAGE_DEBUG ON does not produce the desired 
effect. The messages executed inside the CPackDeb.cmake are redirected 
for some reason to the error stream.


Would you please add the following to line 61, file 
RunCPackVerifyResult.cmake


message(STATUS CPack_error=${CPack_error})

It also looks like the definition of the 
CPACK_DEBIAN_APPLICATIONS_PACKAGE_DEPENDS (equals to an empty string in 
the test) has precedence over the shlibdeps output.


Thanks,
Raffi

--

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


[cmake-developers] [PATCH] FindHDF5.cmake HDF5_VERSION Support

2015-05-08 Thread Huebl, Axel
This commit adds VERSION support for HDF5 from
the same sources as it adds the HDF5_IS_PARALLEL
flag.

Previously posted on
  https://github.com/Kitware/CMake/pull/153
and improved with feedback from Brad King (thx!).
---
 Modules/FindHDF5.cmake | 17 ++---
 1 file changed, 14 insertions(+), 3 deletions(-)

diff --git a/Modules/FindHDF5.cmake b/Modules/FindHDF5.cmake
index 0d58e13..69f31ed 100644
--- a/Modules/FindHDF5.cmake
+++ b/Modules/FindHDF5.cmake
@@ -51,6 +51,7 @@
 #   bindings.
 #   HDF5_LIBRARIES - Required libraries for all requested bindings
 #   HDF5_FOUND - true if HDF5 was found on the system
+#   HDF5_VERSION - HDF5 Version in format Major.Minor.Release
 #   HDF5_LIBRARY_DIRS - the full set of library directories
 #   HDF5_IS_PARALLEL - Whether or not HDF5 was found with parallel IO
support
 #   HDF5_C_COMPILER_EXECUTABLE - the path to the HDF5 C wrapper compiler
@@ -60,6 +61,7 @@

 #=
 # Copyright 2009 Kitware, Inc.
+#   2015 Axel Huebl, Helmholtz-Zentrum Dresden - Rossendorf
 #
 # Distributed under the OSI-approved BSD License (the License);
 # see accompanying file Copyright.txt for details.
@@ -335,6 +337,7 @@ if( NOT HDF5_FOUND )
 # If the HDF5 include directory was found, open H5pubconf.h to
determine if
 # HDF5 was compiled with parallel IO support
 set( HDF5_IS_PARALLEL FALSE )
+set( HDF5_VERSION  )
 foreach( _dir IN LISTS HDF5_INCLUDE_DIRS )
 if( EXISTS ${_dir}/H5pubconf.h )
 file( STRINGS ${_dir}/H5pubconf.h
@@ -343,6 +346,14 @@ if( NOT HDF5_FOUND )
 if( HDF5_HAVE_PARALLEL_DEFINE )
 set( HDF5_IS_PARALLEL TRUE )
 endif()
+
+file( STRINGS ${_dir}/H5pubconf.h
+HDF5_VERSION_DEFINE
+REGEX #define H5_VERSION )
+if( ${HDF5_VERSION_DEFINE} MATCHES
+H5_VERSION[ \t]+\([0-9]+\\.[0-9]+\\.[0-9]+)\ )
+set( HDF5_VERSION ${CMAKE_MATCH_1} )
+endif()
 endif()
 endforeach()
 set( HDF5_IS_PARALLEL ${HDF5_IS_PARALLEL} CACHE BOOL
@@ -357,8 +368,8 @@ if( NOT HDF5_FOUND )

 endif()

-find_package_handle_standard_args( HDF5 DEFAULT_MSG
-HDF5_LIBRARIES
-HDF5_INCLUDE_DIRS
+find_package_handle_standard_args( HDF5
+REQUIRED_VARS HDF5_LIBRARIES HDF5_INCLUDE_DIRS
+VERSION_VAR   HDF5_VERSION
 )

-- 
2.1.4



smime.p7s
Description: S/MIME Cryptographic Signature
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Re: [cmake-developers] [PATCH v2] FindHDF5.cmake HDF5_VERSION Support

2015-05-08 Thread Huebl, Axel
This commit adds VERSION support for HDF5 from
the same sources as it adds the HDF5_IS_PARALLEL
flag.

Previously posted on
  https://github.com/Kitware/CMake/pull/153

and improved with feedback from

 - Brad King
 - Rolf Eike Beer
---
 Modules/FindHDF5.cmake | 19 ---
 1 file changed, 16 insertions(+), 3 deletions(-)

diff --git a/Modules/FindHDF5.cmake b/Modules/FindHDF5.cmake
index 0d58e13..5f41b0f 100644
--- a/Modules/FindHDF5.cmake
+++ b/Modules/FindHDF5.cmake
@@ -51,6 +51,7 @@
 #   bindings.
 #   HDF5_LIBRARIES - Required libraries for all requested bindings
 #   HDF5_FOUND - true if HDF5 was found on the system
+#   HDF5_VERSION - HDF5 version in format Major.Minor.Release
 #   HDF5_LIBRARY_DIRS - the full set of library directories
 #   HDF5_IS_PARALLEL - Whether or not HDF5 was found with parallel IO
support
 #   HDF5_C_COMPILER_EXECUTABLE - the path to the HDF5 C wrapper compiler
@@ -60,6 +61,7 @@

 #=
 # Copyright 2009 Kitware, Inc.
+#   2015 Axel Huebl, Helmholtz-Zentrum Dresden - Rossendorf
 #
 # Distributed under the OSI-approved BSD License (the License);
 # see accompanying file Copyright.txt for details.
@@ -335,6 +337,7 @@ if( NOT HDF5_FOUND )
 # If the HDF5 include directory was found, open H5pubconf.h to
determine if
 # HDF5 was compiled with parallel IO support
 set( HDF5_IS_PARALLEL FALSE )
+set( HDF5_VERSION  )
 foreach( _dir IN LISTS HDF5_INCLUDE_DIRS )
 if( EXISTS ${_dir}/H5pubconf.h )
 file( STRINGS ${_dir}/H5pubconf.h
@@ -343,6 +346,16 @@ if( NOT HDF5_FOUND )
 if( HDF5_HAVE_PARALLEL_DEFINE )
 set( HDF5_IS_PARALLEL TRUE )
 endif()
+unset(HDF5_HAVE_PARALLEL_DEFINE)
+
+file( STRINGS ${_dir}/H5pubconf.h
+HDF5_VERSION_DEFINE
+REGEX ^[ \t]*#[ \t]*define[ \t]+H5_VERSION[ \t]+ )
+if( ${HDF5_VERSION_DEFINE} MATCHES
+H5_VERSION[ \t]+\([0-9]+\\.[0-9]+\\.[0-9]+)\ )
+set( HDF5_VERSION ${CMAKE_MATCH_1} )
+endif()
+unset(HDF5_VERSION_DEFINE)
 endif()
 endforeach()
 set( HDF5_IS_PARALLEL ${HDF5_IS_PARALLEL} CACHE BOOL
@@ -357,8 +370,8 @@ if( NOT HDF5_FOUND )

 endif()

-find_package_handle_standard_args( HDF5 DEFAULT_MSG
-HDF5_LIBRARIES
-HDF5_INCLUDE_DIRS
+find_package_handle_standard_args( HDF5
+REQUIRED_VARS HDF5_LIBRARIES HDF5_INCLUDE_DIRS
+VERSION_VAR   HDF5_VERSION
 )

-- 
2.1.4



smime.p7s
Description: S/MIME Cryptographic Signature
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Re: [cmake-developers] [PATCH] FindHDF5.cmake HDF5_VERSION Support

2015-05-08 Thread Rolf Eike Beer
Huebl, Axel wrote:
 This commit adds VERSION support for HDF5 from
 the same sources as it adds the HDF5_IS_PARALLEL
 flag.

 @@ -343,6 +346,14 @@ if( NOT HDF5_FOUND )
  if( HDF5_HAVE_PARALLEL_DEFINE )
  set( HDF5_IS_PARALLEL TRUE )
  endif()
 +
 +file( STRINGS ${_dir}/H5pubconf.h
 +HDF5_VERSION_DEFINE
 +REGEX #define H5_VERSION )

I suggest using #define[ \t]+H5_VERSION[ \t]+: this works also on changed 
indendation and does not accidentially match H5_VERSION_FOO.

 +if( ${HDF5_VERSION_DEFINE} MATCHES
 +H5_VERSION[ \t]+\([0-9]+\\.[0-9]+\\.[0-9]+)\ )
 +set( HDF5_VERSION ${CMAKE_MATCH_1} )
 +endif()

please unset HDF5_VERSION_DEFINE here.

Greetings,

Eike

signature.asc
Description: This is a digitally signed message part.
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Re: [cmake-developers] [PATCH] FindHDF5.cmake HDF5_VERSION Support

2015-05-08 Thread Huebl, Axel
On 08.05.2015 17:52, Rolf Eike Beer wrote:
 Huebl, Axel wrote:
 This commit adds VERSION support for HDF5 from
 the same sources as it adds the HDF5_IS_PARALLEL
 flag.
 
 @@ -343,6 +346,14 @@ if( NOT HDF5_FOUND )
  if( HDF5_HAVE_PARALLEL_DEFINE )
  set( HDF5_IS_PARALLEL TRUE )
  endif()
 +
 +file( STRINGS ${_dir}/H5pubconf.h
 +HDF5_VERSION_DEFINE
 +REGEX #define H5_VERSION )
 
 I suggest using #define[ \t]+H5_VERSION[ \t]+: this works also on changed 
 indendation and does not accidentially match H5_VERSION_FOO.
 
 +if( ${HDF5_VERSION_DEFINE} MATCHES
 +H5_VERSION[ \t]+\([0-9]+\\.[0-9]+\\.[0-9]+)\ )
 +set( HDF5_VERSION ${CMAKE_MATCH_1} )
 +endif()
 
 please unset HDF5_VERSION_DEFINE here.
 
 Greetings,
 
 Eike
 

Thank you for the feedback.

Good points, I agree!
Shall I unset HDF5_HAVE_PARALLEL_DEFINE for sake of consistency, too?

The second regex should avoid matching H5_VERSION_FOO anyway, even if
it would include several lines in the beginning.

More general would be
  ^[ \t]*#[ \t]*define[ \t]+H5_VERSION[ \t]+

but the question will be how far we should obfuscate it, I guess :)


Greetings,
Axel

-- 

Axel Huebl
Phone +49 351 260 3582
https://www.hzdr.de/crp
Computational Radiation Physics
Laser Particle Acceleration Division
Helmholtz-Zentrum Dresden - Rossendorf e.V.

Bautzner Landstrasse 400, 01328 Dresden
POB 510119, D-01314 Dresden
Vorstand: Prof. Dr.Dr.h.c. R. Sauerbrey
  Prof. Dr.Dr.h.c. P. Joehnk
VR 1693 beim Amtsgericht Dresden



smime.p7s
Description: S/MIME Cryptographic Signature
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers

Re: [cmake-developers] [CMake 0011944]: CPackDeb: Support dependencies between components/Debian packages

2015-05-08 Thread Domen Vrankar
 Please find attached the reworked patch and the fix (separate file) for the
 problem you are facing.
 In separate patches, I also enabled the full debug messages output in a
 separate patch, + some additional logs.

Bitten by my own change... Nice catch. Thanks for finding this.

Your changes did not fix my problem however the changes to logging
made it simple to find what was wrong - missing dpkg-shlibdeps
executable. I'm surprised that it doesn't come along with the vanilla
Debina 7.8.0 installation. Fortunately that revealed this issue.

Applied patches and fixes:
http://www.cmake.org/gitweb?p=cmake.git;a=commit;h=e3f522f

Next I'll take a look at you documentation patch and the comments that
came along it that mail.

Thanks,
Domen
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


Re: [cmake-developers] [CMake 0011944]: CPackDeb: Support dependencies between components/Debian packages

2015-05-08 Thread Raffi Enficiaud

Le 08/05/15 07:54, Domen Vrankar a écrit :

Would you please add
set( CPACK_DEBIAN_PACKAGE_DEBUG ON)

to the file
MyLibCPackConfig-splitted-components-depend2.cmake.in

so that we also have the debug logs?


I currently don't have access to my computer so I'll send you that in
about a week.


Sorry I forgot about this...

Attached is verbose output and patch that I was using (slightly
modified patch that you provided rebased to current master).



Dear Domen,

Please find attached the reworked patch and the fix (separate file) for 
the problem you are facing.
In separate patches, I also enabled the full debug messages output in a 
separate patch, + some additional logs.


Thanks,
Raffi


From 0d7a383618364a4f33e9f00b94d730f9b77c53ac Mon Sep 17 00:00:00 2001
From: Domen Vrankar domen.vran...@gmail.com
Date: Fri, 8 May 2015 07:51:30 +0200
Subject: [PATCH 1/4] CPackDEB: Enabling the settings of the dependencies per
 component.

---
 Modules/CPackDeb.cmake | 48 +--
 Tests/CMakeLists.txt   |  4 +-
 .../MyLibCPackConfig-components-depend1.cmake.in   | 20 +
 .../MyLibCPackConfig-components-depend2.cmake.in   | 31 
 .../RunCPackVerifyResult-components-depend1.cmake  | 85 
 .../RunCPackVerifyResult-components-depend2.cmake  | 92 ++
 6 files changed, 272 insertions(+), 8 deletions(-)
 create mode 100644 
Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend1.cmake.in
 create mode 100644 
Tests/CPackComponentsDEB/MyLibCPackConfig-components-depend2.cmake.in
 create mode 100644 
Tests/CPackComponentsDEB/RunCPackVerifyResult-components-depend1.cmake
 create mode 100644 
Tests/CPackComponentsDEB/RunCPackVerifyResult-components-depend2.cmake

diff --git a/Modules/CPackDeb.cmake b/Modules/CPackDeb.cmake
index d1d5d09..a2e37c0 100644
--- a/Modules/CPackDeb.cmake
+++ b/Modules/CPackDeb.cmake
@@ -54,6 +54,20 @@
 #
 #  May be used to set deb dependencies.
 #
+# .. variable:: CPACK_DEBIAN_COMP_PACKAGE_DEPENDS
+#
+#  * Mandatory : NO
+#  * Default   : :variable:`CPACK_DEBIAN_PACKAGE_DEPENDS`
+#
+#  Indicates the debian package dependencies for a specific component 'COMP'.
+#  This value has priority over :variable:`CPACK_DEBIAN_PACKAGE_DEPENDS`. If
+#  :variable:`CPACK_DEBIAN_PACKAGE_SHLIBDEPS` is set, then the discovered
+#  dependencies will be appended to `CPACK_DEBIAN_COMP_PACKAGE_DEPENDS` if 
set
+#  (intead of `CPACK_DEBIAN_PACKAGE_DEPENDS`).
+#  The value of `CPACK_DEBIAN_COMP_PACKAGE_DEPENDS` can be set to an empty 
string
+#  to enable the automatic discovery of dependencies without inheriting from
+#  the default value of :variable:`CPACK_DEBIAN_PACKAGE_DEPENDS`.
+#
 # .. variable:: CPACK_DEBIAN_PACKAGE_MAINTAINER
 #
 #  * Mandatory : YES
@@ -362,13 +376,6 @@ function(cpack_deb_prepare_package_vars)
 # Remove blank control file
 # Might not be safe if package actual contain file or directory named 
debian
 file(REMOVE_RECURSE ${CPACK_TEMPORARY_DIRECTORY}/debian)
-
-# Append user depend if set
-if(CPACK_DEBIAN_PACKAGE_DEPENDS)
-  set(CPACK_DEBIAN_PACKAGE_DEPENDS 
${CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS}, ${CPACK_DEBIAN_PACKAGE_DEPENDS})
-else()
-  set(CPACK_DEBIAN_PACKAGE_DEPENDS 
${CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS})
-endif()
   else()
 if(CPACK_DEBIAN_PACKAGE_DEBUG)
   message( CPackDeb Debug: Using only user-provided depends because 
package does not contain executable files that contain dynamically linked 
libraries.)
@@ -426,6 +433,33 @@ function(cpack_deb_prepare_package_vars)
   # Depends:
   # You should set: DEBIAN_PACKAGE_DEPENDS
   # TODO: automate 'objdump -p | grep NEEDED'
+
+  # if per-component dependency, overrides the global 
CPACK_DEBIAN_PACKAGE_DEPENDS
+  # automatic dependency discovery will be performed afterwards.
+  if(CPACK_DEB_PACKAGE_COMPONENT)
+string(TOUPPER ${CPACK_DEB_PACKAGE_COMPONENT} _local_component_name)
+set(_component_depends_var 
CPACK_DEBIAN_${_local_component_name}_PACKAGE_DEPENDS)
+
+# if set, overrides the global dependency
+if(DEFINED ${_component_depends_var})
+  set(CPACK_DEBIAN_PACKAGE_DEPENDS ${${_component_depends_var}})
+  if(CPACK_DEBIAN_PACKAGE_DEBUG)
+message(CPackDeb Debug: component '${_local_component_name}' 
dependencies set to '${CPACK_DEBIAN_PACKAGE_DEPENDS}')
+  endif()
+endif()
+  endif()
+
+  # at this point, the CPACK_DEBIAN_PACKAGE_DEPENDS is properly set
+  # to the minimal dependency of the package
+  # Append automatic dependance discovery.
+  if(CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS AND NOT 
CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS STREQUAL )
+if (CPACK_DEBIAN_PACKAGE_DEPENDS)
+  set (CPACK_DEBIAN_PACKAGE_DEPENDS ${CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS}, 
${CPACK_DEBIAN_PACKAGE_DEPENDS})
+else ()
+  set (CPACK_DEBIAN_PACKAGE_DEPENDS ${CPACK_DEBIAN_PACKAGE_AUTO_DEPENDS})
+endif ()
+  endif()
+
   

[cmake-developers] [CMake 0015561]: file COPY FILES_MATCHING PATTERN copies folders' structure

2015-05-08 Thread Mantis Bug Tracker

The following issue has been SUBMITTED. 
== 
http://www.cmake.org/Bug/view.php?id=15561 
== 
Reported By:Wojciech Mamrak
Assigned To:
== 
Project:CMake
Issue ID:   15561
Category:   CMake
Reproducibility:always
Severity:   minor
Priority:   high
Status: new
== 
Date Submitted: 2015-05-08 09:00 EDT
Last Modified:  2015-05-08 09:00 EDT
== 
Summary:file COPY FILES_MATCHING PATTERN copies folders'
structure
Description: 
This command:

file(COPY src/ DESTINATION dst FILES_MATCHING PATTERN *.bat)

copies not only all *.bat files from src to dst, but also folders' structure.
For example, if src is:
src:
 - foo.bat
 - bar.bat
 - images [folder]
   - foo.png
   - bar [folder]

then after calling the command, dst is:
dst:
 - foo.bat
 - bar.bat
 - images [folder]
   - bar [folder]
== 

Issue History 
Date ModifiedUsername   FieldChange   
== 
2015-05-08 09:00 Wojciech MamrakNew Issue
==

-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


Re: [cmake-developers] Missing CPACK_PACKAGE_CONTACT in test for CPackComponentsPrefix

2015-05-08 Thread Domen Vrankar
2015-05-07 15:12 GMT+02:00 Kornel Benko kor...@lyx.org:
 Hi,
 testing new cmake, I get ctest error for the call
 # ctest -R CPackComponentsPrefix
[snip]

 Adding line
 set(CPACK_PACKAGE_CONTACT someContact)
 in Tests/CPackComponentsPrefix/CMakeLists.txt cures the testcase for me.

Thanks for reporting this.

It would seem that a few CPack tests weren't testing RPM and DEB
packages as variables CPACK_BINARY_DEB and CPACK_BINARY_RPM were
not set.

I'll fix this so that tests will run in nightly testing.

Thanks,
Domen
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers


Re: [cmake-developers] Missing CPACK_PACKAGE_CONTACT in test for CPackComponentsPrefix

2015-05-08 Thread Domen Vrankar
 I'll fix this so that tests will run in nightly testing.

Applied patch:
http://www.cmake.org/gitweb?p=cmake.git;a=commit;h=3f300b

Thanks,
Domen
-- 

Powered by www.kitware.com

Please keep messages on-topic and check the CMake FAQ at: 
http://www.cmake.org/Wiki/CMake_FAQ

Kitware offers various services to support the CMake community. For more 
information on each offering, please visit:

CMake Support: http://cmake.org/cmake/help/support.html
CMake Consulting: http://cmake.org/cmake/help/consulting.html
CMake Training Courses: http://cmake.org/cmake/help/training.html

Visit other Kitware open-source projects at 
http://www.kitware.com/opensource/opensource.html

Follow this link to subscribe/unsubscribe:
http://public.kitware.com/mailman/listinfo/cmake-developers