| Commit message (Collapse) | Author | Age | Files | Lines |
|
|
|
|
|
|
|
| |
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 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>
|
|
|
|
|
|
| |
Fix issues diagnosed by clang-tidy [google-readability-casting]
Signed-off-by: Matthias Maennich <matthias@maennich.net>
|
|
|
|
|
|
|
|
| |
Fix issues diagnosed by clang-tidy
- performance-unnecessary-value-param
- performance-unnecessary-copy-initialization
Signed-off-by: Matthias Maennich <matthias@maennich.net>
|
|
|
|
| |
Signed-off-by: Matthias Maennich <matthias@maennich.net>
|
|\
| |
| |
| |
| |
| |
| | |
9a24ab6b CPackIFW: Add some options
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1298
|
| |
| |
| |
| |
| |
| | |
The `CPackIFW` module `cpack_ifw_configure_component` and
`cpack_ifw_configure_component_group` commands gained a new
`REPLACES` and `CHECKABLE` options.
|
|\ \
| |/
|/|
| |
| |
| |
| | |
37d9387b Replace empty-string comparisons with checking against `empty()`.
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1275
|
| | |
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
b06870e5 CPack: use a distinct checksum file for each generator
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1291
|
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | |
| | | |
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
|
|\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
09166339 CPack: Add missing check for CPACK_INSTALL_SCRIPT variable
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1090
|
| |/
| |
| |
| |
| |
| |
| | |
Also add a test case that uses CPACK_INSTALL_SCRIPT.
Co-Author: Domen Vrankar <domen.vrankar@gmail.com>
Fixes: #15005
|
|\ \
| |/
|/|
| |
| |
| |
| | |
a5279ae5 Use C++11 nullptr (cont.)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1283
|
| |
| |
| |
| |
| |
| |
| | |
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>
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
f43baf69 Meta: modernize old-fashioned loops to range-based `for` (CPack).
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1274
|
| | |
| | |
| | |
| | |
| | |
| | | |
Changes done via `clang-tidy` with some manual fine-tuning
for the variable naming and `auto` type deduction
where appropriate.
|
|\ \ \
| |_|/
|/| |
| | |
| | |
| | |
| | | |
5db3aac1 Meta: replace empty-string assignments with `clear()`.
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1276
|
| |/ |
|
|/
|
|
|
|
|
|
| |
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'
|
| |
|
|\
| |
| |
| |
| |
| |
| | |
8c51dbdd cmCPackDragNDropGenerator: Add missing include
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1206
|
| |
| |
| |
| |
| |
| | |
We use `std::sort` and so must include `<algorithm>`.
Issue: #17233
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
5fe64511 CPackIFW: Add CPACK_IFW_REPOSITORIES_DIRECTORIES variable
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1150
|
| | |
| | |
| | |
| | |
| | |
| | | |
Add CPACK_IFW_REPOSITORIES_DIRECTORIES variable to specify
additional repositories dirs that will be used to resolve and
repack dependent components
|
|\ \ \
| | | |
| | | |
| | | |
| | | |
| | | |
| | | | |
ca2233e3 IWYU: Mark cmConfigure.h with pragma: keep
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !1187
|
| | | |
| | | |
| | | |
| | | | |
Also remove `#include "cmConfigure.h"` from most source files.
|
|/ / /
| | |
| | |
| | |
| | | |
Remove the copy of OpenBSD's 'ar' implementation and instead
use cmArchiveWrite with the "arbsd" format.
|
|/ / |
|
| |
| |
| |
| |
| | |
Add CPACK_IFW_PACKAGE_FILE_EXTENSION variable to customize
target binary format.
|
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| |
| |
| | |
* 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
|
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
| |
| |
| |
| |
| | |
Drop the unused `InstallPrefix` member that shadows a member in a
subclass.
|
|\ \
| | |
| | |
| | |
| | |
| | |
| | | |
2042cae9 CPack-FreeBSD: add a generator for FreeBSD pkg(8)
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !693
|
| |/
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/ |
|
| |
|
| |
|
|
|
|
|
|
| |
Support for setting archive packager specific
per component filenames and monolithic package
filenames.
|
|\
| |
| |
| |
| |
| |
| | |
72ac7ad9 CPackIFW: Internationalization Support
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !748
|
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
|
|
|
|
|
|
|
| |
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
|
|\
| |
| |
| |
| |
| |
| | |
543cf134 CPackIFW: Access refactoring
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !820
|
| |
| |
| |
| |
| |
| | |
Changes:
- Access members with this->;
- Access nested enum values with class name.
|
| | |
|
|/ |
|
|\
| |
| |
| |
| |
| |
| | |
3e027d9d c++: prefer vectors over lists
Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !799
|
| |
| |
| |
| |
| |
| |
| |
| |
| |
| |
| | |
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.
|
|/
|
|
| |
Move common used code to new cmCPackIFWCommon base class.
|