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

Split the before-input and after-output cases since the parent must
act between them.

Change-Id: I14b704ea927c7ce7258d40ac105577a355cdcd13

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

Print more information about each failure.

Change-Id: I9c9a0ad628fba6452894eb80251e6fa2bdf1a87f

If only some leading components of the input path exist then convert
those and leave the rest unchanged.  This way if a file path is
constructed using GetActualCaseForPath and created on disk, then a
following GetActualCaseForPath on the same now-existing path will return
the same value.

Change-Id: I6e09a4d411f8bf107312f39d7409ba2d22bf176b
CMake-Issue: 16295

Use ComparePath to perform case-insensitive comparison when needed.

Change-Id: I901b1a12ab43655ac44eee2b3e18c3f2630d6c18

Undefine any `_WIN32_WINNT` value that may be defined on the compiler
command line by clients that are targeting the build at a different
version of Windows than the host.  We must build for the host APIs in
order for the test to be able to configure its console correctly.  Since
the test binary is not installed it does not matter if it is not built
for the client's target Windows version.

Change-Id: Ic3e79550713384c45baf25aa624efbcb5d49d0a4

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

When `GetShortPathNameW` returns a 0 buffer size then we should not call
it again.

Change-Id: I6529013ec3c792796649c25168f6c67b60ac7238

This method checks if a path by the given name exists. The problem with
FileExists is that it returns `false` even for broken (or
self-referential) symlinks which may not be what the user wants.

Change-Id: I71b8af6e12a5ae4b1319a3bec83db6c3cef36f90

Patrick authored 8 years ago and Brad King committed 8 years ago

Take changes used by pkgsrc [1]:

* SCO OpenServer 5.0.7/3.2's command has 711 permission.

[1] http://cdn.netbsd.org/pub/pkgsrc/current/pkgsrc/devel/cmake/patches/

Change-Id: Ib9fd7611e2c0b3e10a75b2322c002a8cb266ea89

Patrick authored 8 years ago and Brad King committed 8 years ago

Take changes used by pkgsrc [1].

[1] http://cdn.netbsd.org/pub/pkgsrc/current/pkgsrc/devel/cmake/patches/

Change-Id: I3b035ad1835755548250b67e2a7cbadaaf6a4657

Patrick authored 8 years ago and Brad King committed 8 years ago

Take changes used by pkgsrc [1]:

* Add more conditional handling for NetBSD, same as others.
* Treat FreeBSD and DragonFly the same way as NetBSD and OpenBSD.

[1] http://cdn.netbsd.org/pub/pkgsrc/current/pkgsrc/devel/cmake/patches/

Change-Id: If94a931dd1609565459eb9891561f341cc990bf0

Remove the last calls to the GetEnv methods with this signature from
KWSys itself.

Change-Id: I9cdb2b060776a00bb698b396c9b33450237cd0b7

Otherwise on e.g. MinGW or Embarcadero it would use POSIX functions
which works only for ASCII paths.

Change-Id: Ib95c79f0bc649e3befb646c6820380ef86108c8a

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

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

Reduce code duplication.

Change-Id: Ic0fcee68286c8e67bcb6fb58cbc7ca71c30f4c36

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

Change-Id: I3cdbcb846a205b211ac0f597e2c1c940dc0b7268

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

Some GetEnv callers only need to know whether an environment variable
exists, so provide a direct query for this.

Change-Id: Ib754cb122b53a2a7cc4801fc7198cf1d612ef6f4

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

On Windows getenv (and putenv) uses ANSI codepage so it needs to be encoded
to internally used encoding (eg. UTF-8). Here we use _wgetenv (and _wputenv)
instead and encode that.

Change-Id: I8cb91f2386eb0efe3ef0a3132d1603217d710b60

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

Revise code layout to make introduction of wide character APIs cleaner.

Change-Id: I22301f07eba1e13b954eb938da3305f269d8b754

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

Change-Id: If7753c828c211998b76a1706ca7fd35fbb4c1f71

When using `putenv("A=")` to approximate removal of `A` from the
environment we must not free the buffer afterward because the string
is still actually in the environment.

Change-Id: I4d2455ca604a2e216123a3d2afafb64ca260d105
Reported-by: James Johnston <johnstonj.public@codenest.com>

Move `kwsysEnv` storage management into a `kwsysEnvSet` base class.

Change-Id: I022a8620e570394bb1871515c8275c969535c693
Inspired-by: Dāvis Mosāns <davispuh@gmail.com>

This header is no longer used by any KWSys clients.

Change-Id: Ia83ed98403c881aa28c9a531111cc1441646b47f

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

Matches the opening of this #ifdef.

Change-Id: I044645285faaed0ac96b5f40af027ec07b73d825

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

If the path is a junction, pass if the junction was successfully
deleted. Not doing so triggers an INVALID_FILE_HANDLE error when
DeleteFileW is called later, not that the path doesn't exist.

Change-Id: I8957a0ca0704f7f950faeac90b15e8c61c5f5a90

Felix Geyer authored 8 years ago and Brad King committed 8 years ago

The Debian package checker tool (lintian) detected typos.

Change-Id: Ic7896c78d5cbac39287bf5e9c925208e53f3d348

_WIN32 is the actual define for Windows. WIN32 is a CMake-ism.

Change-Id: Iea9f69ab1719325703a3c9a6cacdd6ecf13dbc34

Zack Galbreath authored 9 years ago and Brad King committed 9 years ago

Change-Id: I7d588cb060ea17ed4221c55b866cbbebab983e3c

Mariusz Pluciński authored 9 years ago and Brad King committed 9 years ago

Change-Id: If1ef48369feca8bfd0a3831146fae6688f934b0c

Change-Id: Id2f7882d16bf05cfd50983b4fb660fec1d12b5b4

Change-Id: I0c1db70531f1ef3823146fcefc2a8f8ee67f8f2a

Change-Id: I8bd634ecc1bdc035492c145d3a66478dee56bb06

Zack Galbreath authored 9 years ago and Brad King committed 9 years ago

Recompute `TimeoutTime` after `Timeout` changes so that one may modify
the timeout of a process after it has started running by calling the
normal `kwsysProcess_SetTimeout` method.

Change-Id: I460a02d0d3c5f107b844abb56e39f5c844e7e3fb

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

Junction points can exist in directories which may be manipulated within
CMake. The problem is that junction points are currently completely
unsupported within CMake. Since junctions need special API calls to
manipulate, DeleteFileW does not work and instead DeviceIoControl must
be used. Token privilege API calls must also be used to be able to
actually open the junction points.

Change-Id: I3e4ff3b7a600146ac3574d2b17890eef07d06496

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

Change-Id: I694df81c1c62bd538ef02cb2846b6932870dbefe

Costy Petrisor authored 9 years ago and Brad King committed 9 years ago

Update sources using KWSYS_HEADER to make the corresponding ".in"
includes consistent.

Change-Id: I9414edf93928386ffc38e0fd885edd0b7ddb2886

As described in the issues below, Embarcadero compilers emit a spurious warning
when including windows.h, if compiling with the C (not C++) language.

    http://qc.embarcadero.com/wc/qcmain.aspx?d=104595
    [BCC32 Warning] winnt.h(14966)...

    http://qc.embarcadero.com/wc/qcmain.aspx?d=127681
    Annoying warnings upon inclusion of <windows.h> in .C source file

Change-Id: Ie8014edea7e8bbdd76f4555ec2d6cd6d90490032

Use the GetShortPathNameW return value to allocate a buffer of the
required size.  The previous assumption that the "short" path will
always require less space than the original path is incorrect.

CMake-Issue: 15892
Inspired-by: Jan van Dorsten <dorsten@boehme-weihs.de>
Change-Id: I5845752710560b3e71a2f5eaf334dd676f5b3a9d

Jan van Dorsten authored 9 years ago and Brad King committed 9 years ago

Use a `std::string` instead of `new char`.

Change-Id: I88afe09990c6dd881c813df963ccd0031dcea301

The ICC frontend on OSX is clang, and therefore the "__clang__" define exists,
but it doesn't support the clang diagnostic pragma syntax.

Change-Id: I6947bd22c77af8066b85a6b00be7cf6019e7d06e