Browse Source

Merge topic 'cpack-status-callback'

4a3ae1710e CPack/External: Fix status messages of staging scripts

Acked-by: Kitware Robot <[email protected]>
Merge-request: !2696
Brad King 6 years ago
parent
commit
d99e1cca20

+ 10 - 0
Source/CPack/cpack.cxx

@@ -89,6 +89,15 @@ int cpackDefinitionArgument(const char* argument, const char* cValue,
   return 1;
 }
 
+static void cpackProgressCallback(const char* message, float progress,
+                                  void* clientdata)
+{
+  (void)progress;
+  (void)clientdata;
+
+  std::cout << "-- " << message << std::endl;
+}
+
 // this is CPack.
 int main(int argc, char const* const* argv)
 {
@@ -202,6 +211,7 @@ int main(int argc, char const* const* argv)
   cmake cminst(cmake::RoleScript);
   cminst.SetHomeDirectory("");
   cminst.SetHomeOutputDirectory("");
+  cminst.SetProgressCallback(cpackProgressCallback, nullptr);
   cminst.GetCurrentSnapshot().SetDefaultDefinitions();
   cmGlobalGenerator cmgg(&cminst);
   cmMakefile globalMF(&cmgg, cminst.GetCurrentSnapshot());

+ 2 - 0
Tests/RunCMake/CPack/tests/EXTERNAL/create_package.cmake

@@ -22,3 +22,5 @@ expect_file(${CPACK_TEMPORARY_DIRECTORY}/f1/share/cpack-test/f1.txt)
 expect_file(${CPACK_TEMPORARY_DIRECTORY}/f2/share/cpack-test/f2.txt)
 expect_file(${CPACK_TEMPORARY_DIRECTORY}/f3/share/cpack-test/f3.txt)
 expect_file(${CPACK_TEMPORARY_DIRECTORY}/f4/share/cpack-test/f4.txt)
+
+message(STATUS "This status message is expected to be visible")

+ 1 - 0
Tests/RunCMake/CPack/tests/EXTERNAL/stage_and_package-stdout.txt

@@ -0,0 +1 @@
+-- This status message is expected to be visible