Re: [cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Brad King
On 11/08/2016 02:16 PM, Robert Goulet wrote: > + if (cmSystemToolsCMakeRoot.empty()) { > +std::string dir = cmSystemTools::GetFilenamePath(exe_dir); Please see my sibling response for why more logic is needed. -Brad -- Powered by www.kitware.com Please keep messages on-topic and check

Re: [cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Robert Goulet
Anyhow, here's a quick patch that fixes it. Thanks. -Original Message- From: Robert Goulet Sent: Tuesday, November 8, 2016 2:09 PM To: 'Rolf Eike Beer' ; cmake-developers@cmake.org Subject: RE: [cmake-developers] cmake 3.7.0-rc3 regression Not sure I understand why is

Re: [cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Brad King
On 11/08/2016 01:59 PM, Robert Goulet wrote: > Commit: 18bfbc972fd3daf3e973f80072c4de09ec7e852b > Add option to control 'bin' directory of CMake's own installation (#16076) I suspect it is caused by this hunk: > - // Install tree has "/bin/cmake" and "". > - std::string dir =

Re: [cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Rolf Eike Beer
Am Dienstag, 8. November 2016, 18:59:34 schrieb Robert Goulet: > Hi, > > After trying to upgrade to CMake 3.7.0-rc3, we've found the following > regression: > > Previously, we were able to put multiple platform CMake executables in the > same root by just renaming the "bin" folder, so that they

Re: [cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Robert Goulet
Not sure I understand why is this part of a build switch now? And what if we don't build CMake ourselves but instead, using prebuilt binaries from the CMake website? -Original Message- From: Rolf Eike Beer [mailto:e...@sf-mail.de] Sent: Tuesday, November 8, 2016 2:04 PM To:

[cmake-developers] cmake 3.7.0-rc3 regression

2016-11-08 Thread Robert Goulet
Hi, After trying to upgrade to CMake 3.7.0-rc3, we've found the following regression: Previously, we were able to put multiple platform CMake executables in the same root by just renaming the "bin" folder, so that they don't overlap each other, as such: cmake/win/cmake.exe cmake/mac/cmake