On 02/23/2015 12:55 PM, Brad King wrote:
On 02/21/2015 04:33 PM, Domen Vrankar wrote:
Attached are patches with fixed patch 3.
Thanks. I'll start with the SystemTools change which I've split
out into its own commit and pushed here:
http://review.source.kitware.com/#/c/19304/
That one
FYI, the proposed GetCanonicalPath already exists in KWSys as
SystemTools::GetRealPath. It returns the unchanged path on error.
That could be extended to take an optional second argument as a
pointer to a buffer describing the error. If the argument is NULL
use the current behavior.
Attached are patches for KWSys Glob and SystemTools (with changes
required by Glob).
I apologize for spamming...
After rewriting CMake tests with directories instead of symlinks I
noticed that patch 3 was not handling basic directory case.
Attached are patches with fixed patch 3.
Thanks,
I'll get the KWSys patch through. Once it is integrated into
CMake then you can rebase the CMake part of the change on that.
However, please revise the logic in the KWSys patch to avoid
using exceptions.
I've noticed that exceptions are not used but was not certain how else
to handle this
On 02/16/2015 02:47 PM, Domen Vrankar wrote:
I've noticed that exceptions are not used but was not certain how else
to handle this
FYI, the proposed GetCanonicalPath already exists in KWSys as
SystemTools::GetRealPath. It returns the unchanged path on error.
That could be extended to take an
On 02/15/2015 03:04 PM, Domen Vrankar wrote:
I tried to register to http://review.source.kitware.com so that I
would be able to commit patches to KWSys repository but after I put in
my OpenID I got a Not Found error.
I'll be in touch off-list about this.
How can I contribute this patch?
Hi everyone,
I am having problems with a patch for Glob functionality because Glob
is part of KWSys repository.
I tried to register to http://review.source.kitware.com so that I
would be able to commit patches to KWSys repository but after I put in
my OpenID I got a Not Found error.
How can I