[cmake-developers] [CMake 0015005]: Take in account CPACK_INSTALL_SCRIPT when running `cpack`

[Mantis Bug Tracker]

The following issue has been SUBMITTED. 
====================================================================== 
http://public.kitware.com/Bug/view.php?id=15005 
====================================================================== 
Reported By:                Alex Turbov
Assigned To:                
====================================================================== 
Project:                    CMake
Issue ID:                   15005
Category:                   CPack
Reproducibility:            always
Severity:                   minor
Priority:                   normal
Status:                     new
====================================================================== 
Date Submitted:             2014-07-03 05:17 EDT
Last Modified:              2014-07-03 05:17 EDT
====================================================================== 
Summary:                    Take in account CPACK_INSTALL_SCRIPT when running
`cpack`
Description: 
According lines 226-258 of the Source/CPack/cmCPackGenerator.cxx file, there are
few ways to preinstall files to be packed: 
1) via CPACK_INSTALL_COMMANDS variable [InstallProjectViaInstallCommands()
function]
2) CPACK_INSTALL_SCRIPT [InstallProjectViaInstallScript()]
3) CPACK_INSTALLED_DIRECTORIES [InstallProjectViaInstalledDirectories()]
4) and finally CPACK_INSTALL_CMAKE_PROJECTS
[InstallProjectViaInstallCMakeProjects()]

but Source/CPack/cpack.cxx at line 470 checks only 1,3,4 options! so if my
configuration file has only CPACK_INSTALL_SCRIPT variable defined, it doesn't
want to run w/ the error:

.../Source/CPack/cpack.cxx:479 Please specify build tree of the project that
uses CMake using CPACK_INSTALL_CMAKE_PROJECTS, specify CPACK_INSTALL_COMMANDS,
or specify CPACK_INSTALLED_DIRECTORIES.


====================================================================== 

Issue History 
Date Modified    Username       Field                    Change               
====================================================================== 
2014-07-03 05:17 Alex Turbov    New Issue                                    
======================================================================