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

5d2aff9d ProcessWin32: use strdup() instead of open coding it
cf8beae3 ProcessUNIX: use strdup() instead of open coding it

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

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

3b8fefea remove pointer checks before calling free()
6d7eb3a1 CommandLineArguments: do not check variable before delete[]

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

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

ce55a255 testSystemTools: fix some copy'n'paste issues
73c491e8 processUNIX: fix not null terminated buffer during error reporting
13e02b76 processUNIX: close intermediate file descriptor in error case
862562ce SystemInformation: fix potential off-by-one write

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

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

6599eda6 testRobustEncoding: restore format flags for std::cout before exiting

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

in case readlink uses the complete buffer (1024) it will truncate the
result and return 1024 assigned to ll. the subsequent buf[ll] will then be
out of bounds.

the fix assumes, that a truncated result is as useful as experiencing an
error during readlink, hence falling back to /proc/self/exe.

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

6ca15069 SystemTools: make Getcwd() more efficient on Windows

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

Only do one conversion attempt and write that directly to the target buffer. Do
the uppercasing of the drive letter only in the ASCII domain, not in wide chars.

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

41c93d97 Remove unnecessary testIOS

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

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

a5046656 SystemTool: remove unused Convert*MacroString()
caaef6bb testSystemTools: avoid strcpy()

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

This test was meant to cover the `kwsys/ios/*` headers, but they were
removed in commit 15a16826 (Remove include <kwsys/ios/*> and
kwsys_ios:: compatibility layer, 2015-08-20).

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

aa9f3a82 macOS: Account for deployment target when checking for utimensat

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

The SDK for macOS 10.13 adds `utimensat`, but the symbol is not
available at runtime on older macOS versions.  Check the deployment
target before using the symbol.

Issue: cmake/cmake#17101

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

c8cfe4e0 SystemTools: Drop SplitProgramFromArgs function

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

This was used only by CMake's get_filename_components() command, which
has now been re-implemented independently.  The operation is not
well-defined (string parsing should not depend on filesystem content),
so remove it.

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

80652055 Directory: Use Windows Extended Paths

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

Steven authored 7 years ago and Brad King committed 7 years ago

Teach `Load` to access the filesystem using extended paths to handle
long absolute paths.  Add a test case.

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

da61baff Added cmake settable property to enable the macro
6d73752d Allow KWSYSPE_USE_SELECT macro to be overriden at compile time.

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

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

da8a9e65 SystemTools: make GetFilenameName handle either kind of slash

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

Previously trying to parse a windows-style (backslash separated) path on
Linux would fail.  Make this function handle either kind of separator on
both platforms.

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

dfa13188 Fixed a couple of trivial clang -Wunused-macros warnings

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

Some systems / tools don't play nicely with how select is used here.

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

b61faf89 Fix install rules and usage requirements for interface+object libs

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

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

993f0ea4 Add an option to split object and interface libs.

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

Introduce the KWSYS_SPLIT_OBJECTS_FROM_INTERFACE option.  This will
cause the build to generate a separate interface and object library,
instead of a static or shared library.  This will enable kwsys to be
embedded into another library as objects rather than linked as a
seperate library while still maintaining the propagation of target usage
requirements.

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

7e04a3d2 SystemTools: Remove RemoveEmptyPathElements method

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

This method was added recently but we've found another solution for its
intended use case.  Removing empty path elements can corrupt paths that
depend on a trailing empty component to preserve a trailing slash.

Since this method was meant to complement SplitPath and JoinPath, update
their documentation to explain how they treat empty components.