Brad King authored 5 years ago and Kitware Robot committed 5 years ago

5ca03af6 SystemTools: Revert "Reduce scope of 'buf' variable in CollapseFullPath"

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !146

Revert commit 6e36d909 (SystemTools: Reduce scope of 'buf' variable in
CollapseFullPath, 2019-06-18).  A pointer to the buffer is returned by
`Getcwd` and stored in `res_in_base`.  The buffer must remain valid for
the lifetime of that variable.

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

c6f8e24a SystemTools: Fix CollapseFullPath with relative base path
15896025 SystemTools: Use C++11 in SystemToolsAppendComponents

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !145

Although `CollapseFullPath` is meant to construct a full path, it
is possible that a caller will provide a relative base path.
In this case, keep leading `../` components from the base path
instead of discarding them.

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

6c3dfd25 CommandLineArguments: initialize internal class members directly
a93bc28c SystemTools: Drop unnecessary .c_str()
6e36d909 SystemTools: Reduce scope of 'buf' variable in CollapseFullPath
31f5cdeb RegularExpression: Initialize private members on construction
116a4919 RegularExpression: Reduce scope of 'len' variable

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !143

bdvd authored 5 years ago and Brad King committed 5 years ago

The `Realpath` function takes `std::string`.

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

887b1da5 Fixing issues when building with clang 8.0 on Windows

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !142

* Clang with the MSVC STL requires C++14
* Deprecatoion warning suppressions were specific to MSVC and Intel
* An assignment inside a condition resulted in a warning
* kwsysProcessSetExitException was declared, but never defined or used

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

93352b24 SystemInformation: Add Hygon Dhyana Support

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !141

hygonsoc authored 5 years ago and Brad King committed 5 years ago

Add Hygon CPU Vendor ID("HygonGenuine") detection and HasMMXPlus CPU
flag checking.

Signed-off-by: Fan Jinke <fanjinke@hygon.cn>

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

43c6e2fd SystemInformation: Implement backtrace support on Windows

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !140

Ken Martin authored 5 years ago and Brad King committed 5 years ago

The prior code had no implementation on windows. This code relies on
dbghelp.dll which is a standard part of the operating system. It is
implemented for OS from Vista forward when compiled with MSVC version
1800 or later (VS2013).

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

84bd7c9e SystemTools: support hiding the translation map

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !139

Ben Boeckel authored 9 years ago and Brad King committed 5 years ago

When determining full paths, CMake would treat the as-called path for
the source and build directories as real paths even if they were
symlinks. This causes problems when determining the relative path
between the source and build directories when they are symlinks.

Take the following setup as an example:

    /mnt/src/project/
        src/
        build/ -> /mnt/builds/project/build

With the translation map and /mnt/src/project/build/ as the build
directory, CMake would use ../src as the relative path to the source
tree from the build tree rather than ../../../src/project/src and
lookups using ../src would fail. If the build directory were instead
used *as* /mnt/builds/project/build, everything would work just
fine.

This patch makes the core logic which does this logic optional so that
it can be experimented with in CMake itself.

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

5d8af0d6 SystemTools: Optimize range loop in RelativePath method
815cc593 SystemTools: Optimize range loop in FindProgramPath method
ac8d2033 SystemTools: Optimize range loop in FindLibrary method
c7e3ab12 SystemTools: Optimize range loop in FindProgram method
8c9ca162 SystemTools: Optimize range loop in FindProgram method
9992f69b SystemTools: Optimize range loop in FindName method
d6235796 SystemTools: Optimize iterator lifetime in JoinPath
00a3a436 SystemTools: Update buffered env value string only on a change

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !138

In `SystemToolsStatic::GetEnvBuffered` a pointer to the data of an internal
`std::string` is returned.  This string should only be updated on a change to
keep already emitted pointers valid as long as possible.

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

9e8e9ba0 DynamicLoader: avoid the min/max macros from windows.h
efb006b9 DynamicLoader: include stdio.h for _snprintf

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Brad King <brad.king@kitware.com>
Merge-request: !137

Brad King authored 5 years ago and Kitware Robot committed 5 years ago

92334e76 SystemTools: CopyFileAlways: avoid copying file over self

Acked-by: Kitware Robot <kwrobot@kitware.com>
Merge-request: !136

Ben Boeckel authored 5 years ago and Kitware Robot committed 5 years ago

971809c5 DynamicLoader: test the SearchBesideLibrary flag
40d9e482 DynamicLoader: support loading sibling libraries on Windows
d17291ad DynamicLoader: support loading libraries using flags
cf6b5f69 DynamicLoader: use Encoding::ToWindowsExtendedPath for the libname
30198dbc DynamicLoader: fix error reporting on Windows

Acked-by: Kitware Robot <kwrobot@kitware.com>
Reviewed-by: Brad King <brad.king@kitware.com>
Merge-request: !135

Since commit 228c60b8 (SystemTools: CopyFileAlways: try to create a
cheap CoW clone first, 2018-12-07) on Linux copying a file over itself
by specifying `.` as the destination leaves behind an empty file.  This
has long occurred on Windows too.  Move our existing same-file check
to after we know the real destination file.

This converts the path to a format suitable for passing to Windows APIs.

Windows was reporting errors in `wchar_t` which was not reinterpreted as
UTF-8 which is expected from kwsys errors. This basically made error
messages appear as truncated since the UCS-2 byte array had embedded
byte NUL characters.