diff options
| author | Craig Scott <craig.scott@crascit.com> | 2021-03-24 04:43:21 (GMT) |
|---|---|---|
| committer | Craig Scott <craig.scott@crascit.com> | 2021-03-24 05:35:29 (GMT) |
| commit | a6a52563955a5a42e1fec856d65479444ba5fbdc (patch) | |
| tree | e9f8c1124d49cc99b129e486b38eeb5ff9548746 /Source | |
| parent | b7b0fb430397bafae4a7bd80b41d474c91a3b7f3 (diff) | |
| download | CMake-a6a52563955a5a42e1fec856d65479444ba5fbdc.zip CMake-a6a52563955a5a42e1fec856d65479444ba5fbdc.tar.gz CMake-a6a52563955a5a42e1fec856d65479444ba5fbdc.tar.bz2 | |
CPack: Validate and document NSIS branding text trim positions
Diffstat (limited to 'Source')
| -rw-r--r-- | Source/CPack/cmCPackNSISGenerator.cxx | 13 |
1 files changed, 10 insertions, 3 deletions
diff --git a/Source/CPack/cmCPackNSISGenerator.cxx b/Source/CPack/cmCPackNSISGenerator.cxx index 263adfd..f1cc677 100644 --- a/Source/CPack/cmCPackNSISGenerator.cxx +++ b/Source/CPack/cmCPackNSISGenerator.cxx @@ -215,9 +215,16 @@ int cmCPackNSISGenerator::PackageFiles() if (this->IsSet("CPACK_NSIS_BRANDING_TEXT_TRIM_POSITION")) { std::string wantedPosition = this->GetOption("CPACK_NSIS_BRANDING_TEXT_TRIM_POSITION"); - const std::set<std::string> possiblePositions{ "CENTER", "LEFT", - "RIGHT" }; - if (possiblePositions.find(wantedPosition) != possiblePositions.end()) { + if (!wantedPosition.empty()) { + const std::set<std::string> possiblePositions{ "CENTER", "LEFT", + "RIGHT" }; + if (possiblePositions.find(wantedPosition) == + possiblePositions.end()) { + cmCPackLogger(cmCPackLog::LOG_ERROR, + "Unsupported branding text trim position " + << wantedPosition << std::endl); + return false; + } brandingTextPosition = wantedPosition; } } |