summaryrefslogtreecommitdiffstats
path: root/Source/CPack
Commit message (Collapse)AuthorAgeFilesLines
* Fix left-over occurrences of else-after-returnMatthias Maennich2017-09-282-8/+8
| | | | | | | | Fix issues diagnosed by clang-tidy [readability-else-after-return] These were mostly only showing up on OSX. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Fix minor clang-tidy findingsMatthias Maennich2017-09-281-1/+1
| | | | | | | | | | | Fix issues diagnosed by clang-tidy - modernize-use-bool-literals - misc-string-integer-assignment - performance-faster-string-find - readability-redundant-string-cstr - readability-delete-null-pointer Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Clean up some C-Style castsMatthias Maennich2017-09-281-1/+1
| | | | | | Fix issues diagnosed by clang-tidy [google-readability-casting] Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Fix some occurrences using string by value rather than by const&Matthias Maennich2017-09-282-3/+4
| | | | | | | | Fix issues diagnosed by clang-tidy - performance-unnecessary-value-param - performance-unnecessary-copy-initialization Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Retire std::auto_ptr and its macro CM_AUTO_PTRMatthias Maennich2017-09-252-53/+49
| | | | Signed-off-by: Matthias Maennich <matthias@maennich.net>
* Merge topic 'cpackifw-options'Brad King2017-09-252-0/+43
|\ | | | | | | | | | | | | 9a24ab6b CPackIFW: Add some options Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1298
| * CPackIFW: Add some optionsKonstantin Podsvirov2017-09-212-0/+43
| | | | | | | | | | | | The `CPackIFW` module `cpack_ifw_configure_component` and `cpack_ifw_configure_component_group` commands gained a new `REPLACES` and `CHECKABLE` options.
* | Merge topic 'string-empty'Brad King2017-09-221-1/+1
|\ \ | |/ |/| | | | | | | | | 37d9387b Replace empty-string comparisons with checking against `empty()`. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1275
| * Replace empty-string comparisons with checking against `empty()`.Pavel Solodovnikov2017-09-211-1/+1
| |
* | Merge topic 'cpack-different-checksum-file-per-generator'Brad King2017-09-211-2/+1
|\ \ | | | | | | | | | | | | | | | | | | b06870e5 CPack: use a distinct checksum file for each generator Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1291
| * | CPack: use a distinct checksum file for each generatorDomen Vrankar2017-09-201-2/+1
| | | | | | | | | | | | | | | | | | | | | | | | | | | Different CPack generators could produce checksum files with the same name which were overwritten by each other since only package name without extensions was used for checksum file name generation. This patch adds package extension to checksum files to prevent collisions. Fixes: #16840
* | | Merge topic 'cpack-check-install-script'Brad King2017-09-211-1/+2
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | 09166339 CPack: Add missing check for CPACK_INSTALL_SCRIPT variable Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1090
| * | CPack: Add missing check for CPACK_INSTALL_SCRIPT variableAlex Turbov2017-09-201-1/+2
| |/ | | | | | | | | | | | | Also add a test case that uses CPACK_INSTALL_SCRIPT. Co-Author: Domen Vrankar <domen.vrankar@gmail.com> Fixes: #15005
* | Merge topic 'cxx11-nullptr'Brad King2017-09-207-23/+23
|\ \ | |/ |/| | | | | | | | | a5279ae5 Use C++11 nullptr (cont.) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1283
| * Use C++11 nullptr (cont.)Matthias Maennich2017-09-197-23/+23
| | | | | | | | | | | | | | Fix remaining occurrences of the issue addressed in commit 5962db4389 (Use C++11 nullptr, 2017-08-22) that are only showing up on macOS. Signed-off-by: Matthias Maennich <matthias@maennich.net>
* | Merge topic 'ranged-for'Brad King2017-09-1918-389/+246
|\ \ | | | | | | | | | | | | | | | | | | f43baf69 Meta: modernize old-fashioned loops to range-based `for` (CPack). Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1274
| * | Meta: modernize old-fashioned loops to range-based `for` (CPack).Pavel Solodovnikov2017-09-1918-389/+246
| | | | | | | | | | | | | | | | | | Changes done via `clang-tidy` with some manual fine-tuning for the variable naming and `auto` type deduction where appropriate.
* | | Merge topic 'string-clear'Brad King2017-09-195-21/+21
|\ \ \ | |_|/ |/| | | | | | | | | | | | | | 5db3aac1 Meta: replace empty-string assignments with `clear()`. Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1276
| * | Meta: replace empty-string assignments with `clear()`.Pavel Solodovnikov2017-09-155-21/+21
| |/
* | Use C++11 override instead of CM_OVERRIDEBrad King2017-09-1525-100/+100
|/ | | | | | | | We now require C++11 support including `override`. Drop use of the old compatibility macro. Convert references as follows: git grep -l CM_OVERRIDE -- '*.h' '*.hxx' '*.cxx' | xargs sed -i 's/CM_OVERRIDE/override/g'
* clang-format: format all code as Cpp11Daniel Pfeifer2017-08-301-3/+2
|
* Merge topic 'cpack-dmg-iwyu'Brad King2017-08-301-0/+1
|\ | | | | | | | | | | | | 8c51dbdd cmCPackDragNDropGenerator: Add missing include Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1206
| * cmCPackDragNDropGenerator: Add missing includeBrad King2017-08-291-0/+1
| | | | | | | | | | | | We use `std::sort` and so must include `<algorithm>`. Issue: #17233
* | Merge topic 'cpackifw-repositories-directories'Brad King2017-08-282-0/+40
|\ \ | | | | | | | | | | | | | | | | | | 5fe64511 CPackIFW: Add CPACK_IFW_REPOSITORIES_DIRECTORIES variable Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1150
| * | CPackIFW: Add CPACK_IFW_REPOSITORIES_DIRECTORIES variableKonstantin Podsvirov2017-08-172-0/+40
| | | | | | | | | | | | | | | | | | Add CPACK_IFW_REPOSITORIES_DIRECTORIES variable to specify additional repositories dirs that will be used to resolve and repack dependent components
* | | Merge topic 'iwyu-keep-cmConfigure'Daniel Pfeifer2017-08-2723-23/+20
|\ \ \ | | | | | | | | | | | | | | | | | | | | | | | | ca2233e3 IWYU: Mark cmConfigure.h with pragma: keep Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !1187
| * | | IWYU: Mark cmConfigure.h with pragma: keepDaniel Pfeifer2017-08-2623-23/+20
| | | | | | | | | | | | | | | | Also remove `#include "cmConfigure.h"` from most source files.
* | | | cmCPackDebGenerator: Use libarchive to create .deb fileDaniel Pfeifer2017-08-251-271/+19
|/ / / | | | | | | | | | | | | Remove the copy of OpenBSD's 'ar' implementation and instead use cmArchiveWrite with the "arbsd" format.
* | | Use C++11 nullptrDaniel Pfeifer2017-08-2414-52/+51
|/ /
* | CPackIFW: Add CPACK_IFW_PACKAGE_FILE_EXTENSION variableKonstantin Podsvirov2017-08-092-12/+27
| | | | | | | | | | Add CPACK_IFW_PACKAGE_FILE_EXTENSION variable to customize target binary format.
* | CPack: Pass volume mount name to AppleScript instead of volume name.Gusts Kaksis2017-07-201-1/+3
| | | | | | | | | | | | Get the mount name from mount point path returned by hdiutil instead of assuming it is the volume name. They can be different in case of conflict with an already-mounted volume.
* | Change ComputeFileMD5 to ComputeFileHashAndré Klitzing2017-07-141-5/+4
| | | | | | | | | | | | | | * Use a parameter to select hash algorithm * Return a std::string as result or an empty string if it fails * Avoids unnecessary copy of hash value
* | CPackDeb: Enable the DEB generator on WindowsNils Gladitz2017-06-282-10/+28
| | | | | | | | | | | | | | | | While some features require external Unix tools the generator is mostly portable. By enabling it on Windows it can be used for cross platform packaging.
* | cmCPackDragNDropGenerator: Drop unused memberBrad King2017-06-141-2/+0
| | | | | | | | | | Drop the unused `InstallPrefix` member that shadows a member in a subclass.
* | Merge topic 'cpack-freebsd-pkg'Brad King2017-06-123-0/+405
|\ \ | | | | | | | | | | | | | | | | | | 2042cae9 CPack-FreeBSD: add a generator for FreeBSD pkg(8) Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !693
| * | CPack-FreeBSD: add a generator for FreeBSD pkg(8)Adriaan de Groot2017-06-103-0/+405
| |/ | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | | Adds an option CPACK_ENABLE_FREEBSD_PKG to allow CPack to look for FreeBSD's libpkg / pkg(8). If this is set and the libpkg headers and library are found (which they will be, by default, on any FreeBSD system), then add a FreeBSD pkg(8) generator. The FreeBSD package tool pkg(8) uses tar.xz files (.txz) with two metadata files embedded (+MANIFEST and +COMPACT_MANIFEST). This introduces a bunch of FreeBSD-specific CPACK_FREEBSD_PACKAGE_* variables for filling in the metadata; the Debian generator does something similar. Documentation for the CPack CMake-script is styled after the Debian generator. Implementation notes: - Checks for libpkg -- the underlying implementation for pkg(8) -- and includes FreeBSD package-generation if building CMake on a UNIX host. Since libpkg can be used on BSDs, Linux and OSX, this potentially adds one more packaging format. In practice, this will only happen on FreeBSD and DragonflyBSD. - Copy-paste from cmCPackArchiveGenerator to special-case the metadata generation and to run around the internal archive generation: use libpkg instead. - Generating the metadata files is a little contrived. - Most of the validation logic for package settings is in CPackFreeBSD.cmake, as well as the code that tries to re-use packaging settings that may already be set up for Debian. - libpkg has its own notion of output filename, so we have another contrived bit of code that munges the output file list so that CPack can find the output. - Stick with C++98.
* | Add CM_OVERRIDE to CPack generator destructorsBrad King2017-06-088-8/+8
|/
* Remove second arg: npos in substr usagesPavel Solodovnikov2017-06-012-4/+4
|
* cmCPackLog: pass std::string by const&Daniel Pfeifer2017-05-251-6/+6
|
* CPack/Archive: per component filenames supportDomen Vrankar2017-05-162-16/+46
| | | | | | Support for setting archive packager specific per component filenames and monolithic package filenames.
* Merge topic 'cpackifw-i18n'Brad King2017-05-164-16/+101
|\ | | | | | | | | | | | | 72ac7ad9 CPackIFW: Internationalization Support Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !748
| * CPackIFW: Internationalization SupportKonstantin Podsvirov2017-05-124-16/+101
| | | | | | | | | | | | | | | | | | Changes: - DISPLAY_NAME and DESCRIPTION in CPackIFW module now is MULTI_ARGS; - Added internationalization support for DisplayName and Description properties in cmCPackIFWPackage class; - Added documentation to CPackIFW module; - Added release note.
* | CPack unify component/monolithic package symlink handlingDomen Vrankar2017-05-141-0/+1
|/ | | | | | | | | | | Non component package generation was recursing through symlinks while monolithic package generation did not. This was causing component archive packages to contain both symlink and a directory with the same name on the same level if symlink contained files which caused errors during extraction of such packages. Closes #16051
* Merge topic 'cpackifw-access-refactoring'Brad King2017-05-115-558/+583
|\ | | | | | | | | | | | | 543cf134 CPackIFW: Access refactoring Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !820
| * CPackIFW: Access refactoringKonstantin Podsvirov2017-05-095-558/+583
| | | | | | | | | | | | Changes: - Access members with this->; - Access nested enum values with class name.
* | cmState: remove RemoveUnscriptableCommandsDaniel Pfeifer2017-05-081-2/+0
| |
* | cmake: initialize with Role that controls which commands to registerDaniel Pfeifer2017-05-082-2/+2
|/
* Merge topic 'vector-over-list'Brad King2017-05-052-6/+6
|\ | | | | | | | | | | | | 3e027d9d c++: prefer vectors over lists Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !799
| * c++: prefer vectors over listsBen Boeckel2017-05-042-6/+6
| | | | | | | | | | | | | | | | | | | | | | None of these usages of `std::list` were inserting or removing elements in the middle of the structure, so there were no benefits to using it. Other uses were related to C pointers being stable in a list of strings whereas in a vector of strings, small pointer optimizations could be moved and become invalid after a modification to the hosting vector. None of these uses modified the vector after handing out a C string to an external store.
* | CPackIFW: Refactor out some commonly used codeKonstantin Podsvirov2017-05-0310-322/+238
|/ | | | Move common used code to new cmCPackIFWCommon base class.