summaryrefslogtreecommitdiffstats
path: root/Source/CPack
Commit message (Collapse)AuthorAgeFilesLines
* tidy: Fix modernize-raw-string-literal in FreeBSD CPack generatorBrad King2019-04-051-1/+1
|
* Modernize: Enable modernize-raw-string-literal in clang-tidyArtur Ryt2019-04-021-13/+13
|
* Modernize: Prefer .substr in place of .c_str() + intArtur Ryt2019-03-301-1/+1
| | | | | | A lot of temporary/local strings were created out of C-strings substr can utilize current string size, so in theory be a little more efficient.
* Replace use of CollapseCombinedPath with CollapseFullPathBrad King2019-03-192-3/+2
| | | | | | | | | | | `CollapseCombinedPath` was introduced by commit 551d3343cd (cmDependsC: Collapse relative include paths, 2013-06-19, v2.8.12~237^2) where the existing `CollapseFullPath` should have been used instead. Then its use proliferated slightly. Since `CollapseCombinedPath` is less widely used and less robust (see issue #19049), use `CollapseFullPath` everywhere instead. Issue: #19050
* cmSystemTools: More functions accept `std::string` paramsVitaly Stakhovsky2019-02-201-2/+1
|
* cmMakefile::ConfigureFile: Accept `std::string` parametersVitaly Stakhovsky2019-02-196-12/+13
|
* Delete some default constructors and assignment operatorsAlbert Astals Cid2019-02-152-0/+6
| | | | | | They are unused, but if someone used them they would lead to problems since they would copy the internal raw pointers and the destructor would cause double delete
* Merge topic 'systools-stdstring2'Brad King2019-02-131-2/+1
|\ | | | | | | | | | | | | 080a79ca4a cmSystemTools: More methods accept `std::string` params Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2952
| * cmSystemTools: More methods accept `std::string` paramsVitaly Stakhovsky2019-02-111-2/+1
| |
* | cmake: Progress functions use `std::string` paramVitaly Stakhovsky2019-02-113-4/+5
|/
* Merge topic 'modernize-for-loops'Brad King2019-02-112-37/+22
|\ | | | | | | | | | | | | | | | | 01b2d6ab74 Modernize: Use ranged for-loops when possible 15bdbec017 cmAlgorithms: Make cmRange advance/retreat safe for rvalues Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Daniel Pfeifer <daniel@pfeifer-mail.de> Merge-request: !2901
| * Modernize: Use ranged for-loops when possibleArtur Ryt2019-02-072-37/+22
| | | | | | | | | | | | | | Replaced most manual `const_iterator`-based loops and some reverse-iterator loops with range loops. Fixes: #18858
* | Merge topic 'runsinglecommand'Brad King2019-02-088-23/+22
|\ \ | |/ |/| | | | | | | | | 65baaa0e37 cmSystemTools::RunSingleCommand: Accept std::string argument Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2927
| * cmSystemTools::RunSingleCommand: Accept std::string argumentVitaly Stakhovsky2019-02-068-23/+22
| |
* | Prefer front/back/data over dereferencing begin/rbegin iterArtur Ryt2019-02-061-1/+1
|/ | | | Changed for sequenced containers: vector, list, string and array
* Merge topic 'functional-callbacks'Brad King2019-01-302-13/+5
|\ | | | | | | | | | | | | | | | | | | 8c92db829b MessageCallback: Remove unused bool& argument bcee24aecc Use `std::function` for callbacks Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: vvs31415 <vstakhovsky@fastmail.com> Acked-by: Daniel Pfeifer <daniel@pfeifer-mail.de> Merge-request: !2872
| * Use `std::function` for callbacksRegina Pfeifer2019-01-292-13/+5
| |
* | Merge topic 'copyfile-stdstring'Brad King2019-01-303-8/+4
|\ \ | | | | | | | | | | | | | | | | | | c31b6e616d cmSystemTools: copy file member functions accept std::string params Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2877
| * | cmSystemTools: copy file member functions accept std::string paramsVitaly Stakhovsky2019-01-293-8/+4
| |/ | | | | | | | | Cleaned up `c_str()`s. `cmSystemTools::CopyFileIfDifferent()` removed as redundant.
* | Merge topic 'tidy-use-equals-default'Brad King2019-01-2921-79/+27
|\ \ | |/ |/| | | | | | | | | | | | | 094f01d0f0 cleanup: Prefer compiler provided special member functions 55671b41d2 clang-tidy: Use `= default` Acked-by: Kitware Robot <kwrobot@kitware.com> Acked-by: Brad King <brad.king@kitware.com> Merge-request: !2841
| * clang-tidy: Use `= default`Regina Pfeifer2019-01-2521-79/+27
| | | | | | | | | | | | Suppress some cases in `Source/cmGeneratorExpressionNode.cxx` and `Source/cmUVHandlePtr.h` where a few older compilers require a user-defined default constructor (with `{}`).
* | Merge topic 'renamefile-string'Brad King2019-01-251-1/+1
|\ \ | |/ |/| | | | | | | | | 9e5c13738b cmSystemTools::RenameFile: Accepts std::string args Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2842
| * cmSystemTools::RenameFile: Accepts std::string argsVitaly Stakhovsky2019-01-231-1/+1
| |
* | clang-tidy: Pass by valueRegina Pfeifer2019-01-222-31/+28
|/
* Merge topic 'read-list-file'Brad King2019-01-213-7/+7
|\ | | | | | | | | | | | | 25caf7bafe cmMakefile::ReadListFile() accepts std::string argument Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2821
| * cmMakefile::ReadListFile() accepts std::string argumentVitaly Stakhovsky2019-01-173-7/+7
| | | | | | | | Same for cmMakefile::ReadDependentFile(); some cleanup
* | clang-tidy: Use emplaceRegina Pfeifer2019-01-174-14/+15
|/
* Properties: Add CMAKE_ROLE global propertyKyle Edwards2019-01-172-2/+4
| | | | | This property allows scripts to determine whether they're in project mode, script mode, find-package mode, CTest, or CPack.
* Fix most clang -Wextra-semi-stmt warnings in C++ filesSean McBride2019-01-151-7/+9
| | | | Suppress one in code generated by flex.
* clang-tidy: Simplify boolean expressionsRegina Pfeifer2019-01-152-9/+3
|
* IWYU: Update CMake code for IWYU built with Clang 6Brad King2019-01-153-0/+3
| | | | | IWYU now correctly requires `<utility>` for `std::move`. It also requires a container header when used via a range-based for loop.
* Add missing overrideVitaly Stakhovsky2018-12-301-6/+6
|
* Merge topic 'cpack-status-callback'Brad King2018-12-101-0/+10
|\ | | | | | | | | | | | | 4a3ae1710e CPack/External: Fix status messages of staging scripts Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2696
| * CPack/External: Fix status messages of staging scriptsNils Gladitz2018-12-071-0/+10
| | | | | | | | | | | | | | | | Set progress callback on cmake instance used by CPack. The progress callback is used to output STATUS messages which are otherwise missing when issued from e.g. CPack External staging scripts. Fixes: #18567
* | cmCPackFreeBSDGenerator: fix clang-tidy issuesRegina Pfeifer2018-12-061-26/+19
| | | | | | | | | | | | Install FreeBSD's libpkg on Linux and configure CMake with CPACK_ENABLE_FREEBSD_PKG and CMake_RUN_IWYU set to ON. Then, fix all warnings.
* | Merge topic 'productbuild-encode-pkg-url-ref'Brad King2018-12-061-1/+2
|\ \ | | | | | | | | | | | | | | | | | | | | | 7954ba9bc1 productbuild: escape pkg-ref urls 652210e901 cmSystemTools: Add EncodeURL helper Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2681
| * | productbuild: escape pkg-ref urlsBrad King2018-11-281-1/+2
| | | | | | | | | | | | | | | Inspired-by: James Goruk <james.goruk@gmail.com> Fixes: #18645
* | | CPack/Deb: Remove paxr as a supported tar format for a .debGuillem Jover2018-11-281-1/+1
|/ / | | | | | | | | | | | | | | | | The only format supported from the ones available within cpack is gnutar, which supports large numbers encoded in base-256, and long filenames and linknames. PAX is not and has never been a supported format by dpkg. Fixes: #18620
* | Merge topic 'readability-static-accessed-through-instance'Brad King2018-11-282-4/+6
|\ \ | | | | | | | | | | | | | | | | | | 3e60580784 clang-tidy: Fix readability-static-accessed-through-instance Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2647
| * | clang-tidy: Fix readability-static-accessed-through-instanceRegina Pfeifer2018-11-212-4/+6
| | | | | | | | | | | | Enable the check in .clang-tidy and fix all warnings.
* | | clang-tidy: fix warnings in macOS-only codeBrad King2018-11-274-5/+5
|/ /
* | cpack: When given an unknown generator print out all valid generatorsRobert Maynard2018-11-091-1/+14
| | | | | | | | This makes cpack behavior match cmake when passed an invalid generator
* | cpack: Better error message when generator doesn't exist.Robert Maynard2018-11-081-2/+3
|/ | | | | | Currently the CPack error message when passed an invalid generator name reads like the generator failed to work, rather than the name was invalid.
* CPack: Rename Ext generator to ExternalCraig Scott2018-11-053-38/+40
| | | | | Ext and External were used inconsistently in the code and the docs. This change converts all uses of Ext to External, including within variable names used by the generator.
* CPack/Deb: Add ability to split out debug symbols into .ddeb packageAndrew Fuller2018-09-212-384/+562
|
* Warnings: Remove unused variablesCraig Scott2018-09-111-4/+0
|
* CPackExt: Add CPACK_EXT_ENABLE_STAGING and CPACK_EXT_PACKAGE_SCRIPTNils Gladitz2018-08-143-21/+52
| | | | | | | | | CPACK_EXT_ENABLE_STAGING enables optional staging and CPACK_EXT_PACKAGE_SCRIPT allows to specify an optional script file that can package staged files via an external packaging tool. Issue: #18236
* Merge topic 'cleanup-find-cstr'Brad King2018-08-091-1/+1
|\ | | | | | | | | | | | | 69ca85cc7f Remove unnecessary c_str() in RegularExpression::find calls Acked-by: Kitware Robot <kwrobot@kitware.com> Merge-request: !2271
| * Remove unnecessary c_str() in RegularExpression::find callsVitaly Stakhovsky2018-08-071-1/+1
| |
* | cmGeneratedFileStream: clang-tidy applied to remove redundant ``c_str`` callsSebastian Holtermann2018-08-078-21/+21
|/ | | | | | | | | | | | | | | After changing the ``cmGeneratedFileStream`` methods to accept ``std::string const&`` instead of ``const char*`` we don't need to call ``std::string::c_str`` anymore when passing a ``std::string`` to a ``cmGeneratedFileStream`` method. This patch removes all redundant ``std::string::c_str`` calls when passing a string to a ``cmGeneratedFileStream`` method. It was generated by building CMake with clang-tidy enabled using the following options: -DCMAKE_CXX_CLANG_TIDY=/usr/bin/clang-tidy-4.0;-checks=-*,readability-redundant-string-cstr;-fix;-fix-errors