When building statically, PIC may still be wanted in order to be used in
a shared library elsewhere.

Cristian Adam authored 6 years ago and Brad King committed 6 years ago

Add LTO flags when `INTERPROCEDURAL_OPTIMIZATION` is enabled.

VTK's new module system expects alias targets for each actual target.
Support creating such an alias target.

Our use of the `-qsuppress=` flag to suppress this warning was removed
by commit 51e95ba7 (XL: Fix compatibility with newer clang-based XL on
Linux, 2017-04-28) because XL 13.1.[1-6] for Linux does not support the
flag.  XL 16.1 restores support for the flag.  Restore our use of it for
this version.  Suppress the warning on intermediate versions by blocking
all warnings with `-w`.

The check for whether or not to install a namelink depends on
KWSYS_BUILD_SHARED, but this variable wasn't being set until later
in CMakeLists.txt, causing the namelink to not be installed if
KWSYS_BUILD_SHARED is not explicitly set. Fix this by moving the
variable assignment closer to the top of the file.

Namelinks are used by the compiler when compiling a program that uses
a library, so they should be part of the development component.

The include directory usage requirement for the build tree was added by
commit 993f0ea4 (Add an option to split object and interface libs,
2017-06-19) and preserved by commit b61faf89 (Fix install rules and
usage requirements for interface+object libs, 2017-06-28).  However, its
value should be that of `KWSYS_HEADER_ROOT` rather than just its
default.

Ben Boeckel authored 7 years ago and Brad King committed 7 years ago

Non-Windows platforms don't have backslash as a separator, however some
projects may want to support Windows paths on non-Windows platforms
(e.g., paths from data files). For those which don't need this, this
allows the search to be faster since it's searching for a single
character.

Some host projects no longer set `CMAKE_MACOSX_RPATH` explicitly due to
requiring CMake 3.0 or above, and now see CMP0042 warnings.  Set the
policy explicitly in KWSys to avoid the warnings.  Host projects can
still set `CMAKE_MACOSX_RPATH` to `0` to get the old behavior.

Suggested-by: Ken Martin <ken.martin@kitware.com>

On Linux, getpwnam requires a dynamic glibc at runtime so it can't
really be used in a "fully static" build  and will cause an excessive
number of linker warnings pertaining to such.

+ fix documentation of method name
+ fix some source comment typos

CMake now builds as C++11 or higher and uses `unordered_{map,set}`
instead of `hash_{map,set}`.  Skip providing KWSys hashtable to it
since it uses features deprecated by C++17.

Some C++ compilers now warn when a switch case falls through to another
case, but compilers vary in how to suppress the warning with an explicit
fallthrough specifier.  Provide a macro for the specifier.

The `string(APPEND)` command was introduced by CMake 3.4, but we still
support older versions.

UBsan causes our test crash to abort instead.  Switch the test to
explicitly abort when sanitizers are in use.

While a process is running, it is possible for other processes such as
NTP to alter the system clock in large jumps.

To make the measurement of how long a process has taken to run
impervious to such large jumps it is helpful to use a monotonic clock.
This is a clock that takes an arbitrary starting point that does not
change.

clock_gettime() is a POSIX method that implements this clock. This is
not available on all platforms, so this commit adds a check for its
presence. If the monotonic timer is available, use that in
kwsysProcessTimeGetCurrent(), otherwise falls back to previous
non-monotonic behaviour.

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).

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.

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.

KWSys has not been updated for `clang-tidy`.
Until this is done, block use of the tool on our own targets.
This is useful when KWSys is embedded in the source tree of
another project that may activate clang-tidy.

KWSys has not been updated for `include-what-you-use`.
Until this is done, block use of the tool on our own targets.
This is useful when KWSys is embedded in the source tree of
another project that may activate iwyu.

It appears that Visual Studio 2015 Update 1 lacks this flag. Update 3
includes it; Update2 is unknown.

See cmake/cmake#16623.

Without this the linking of code using
Source/kwsys/SystemInformation.cxx will
have undefined Symbols on Solaris 11.

Dāvis Mosāns authored 8 years ago and Brad King committed 8 years ago

The `std::string` and `std::wstring` types are allowed to contain null
(`0`) bytes.  Teach Encoding::{ToNarrow,ToWide} to preserve them.

Change-Id: If9ddb77d275c1365016f2f1fa811d0c06e46d5f2

Chuck Atkins authored 8 years ago and Brad King committed 8 years ago

Rather than just testing for explicit platforms in the dynamic loader,
i.e. old BlueGene and Cray systems, leverage CMake's platform knowlege
to determine if shared libraries are supported.

Change-Id: I6189bcfb6d528cf4c96a40be2b31af981d09dbb2

Make KWSys honor visibility properties set by the consuming project.

Change-Id: I66b982d955089bc78f88e29c4a7bc19bf2abec00

Use a loop to avoid code duplication and comment the purpose of each
policy.

Change-Id: Id81c24868f89236dab3477034653e6a05321c696

Per-source copyright/license notice headers that spell out copyright holder
names and years are hard to maintain and often out-of-date or plain wrong.
Precise contributor information is already maintained automatically by the
version control tool.  Ultimately it is the receiver of a file who is
responsible for determining its licensing status, and per-source notices are
merely a convenience.  Therefore it is simpler and more accurate for
each source to have a generic notice of the license name and references to
more detailed information on copyright holders and full license terms.

Our `Copyright.txt` file now contains a list of Contributors whose names
appeared source-level copyright notices.  It also references version control
history for more precise information.  Therefore we no longer need to spell
out the list of Contributors in each source file notice.

Replace KWSys per-source copyright/license notice headers with a short
description of the license and links to `Copyright.txt` and online
information available from "https://cmake.org/licensing#kwsys".

Run the `filter-notices.bash` script to perform the replacements mechanically.
Manually fix up the shebang line in one file.

Change-Id: I8497f7c868664dcf54a8608ab302ad93c860b334

Currently Microsoft's C++ libraries implementation of std::cout/cerr
can't output Unicode characters but only ASCII or ANSI if locale is set
so we implement and use our own ConsoleBuf which can output Unicode
characters to console and it doesn't matter what locale or console's
codepage is set.

Change-Id: I33053aa229796e84088aa3beb0ebe8bdbebaf3d1

Dāvis Mosāns authored 8 years ago and Brad King committed 8 years ago

std::basic_filebuf::open(const wchar_t *) isn't part of C++ standard
and it's only present for MSVC but it isn't present in libstdc++ (MinGW)
so we implement this functionality using GNU libstdc++ stdio_filebuf
extension and _wfopen function.

Change-Id: I7df7a1b22ba272db23c8d962cb66ec71a278ef06

This header is no longer used by any KWSys clients.

Change-Id: Ia83ed98403c881aa28c9a531111cc1441646b47f

This header is no longer used by any KWSys clients.

Change-Id: I9362a7053b2f24f6bacf2bbe89dbfceeaabe70ae

This header is no longer used by any KWSys clients.

Change-Id: Ifca2c6aec5d4b3068774d8398719376d61706d53