search

pnlbwh / ukftractography

Other
25 stars 27 forks source link

Superbuild fails #74

Closed reckbo closed 7 years ago

reckbo commented 7 years ago

I've been getting this error recently.

git clone https://github.com/pnlbwh/ukftractography
mkdir build && cd build
cmake ../ukftractography
make
CMake Error at CMakeLists.txt:74 (message):
  CMAKE_RUNTIME_OUTPUT_DIRECTORY is set to a non-existing directory
  [/tmp/build/bin]

-- Configuring incomplete, errors occurred!
See also "/tmp/build/teem-build/CMakeFiles/CMakeOutput.log".
See also "/tmp/build/teem-build/CMakeFiles/CMakeError.log".
make[2]: *** [teem-prefix/src/teem-stamp/teem-configure] Error 1
make[1]: *** [CMakeFiles/teem.dir/all] Error 2
make: *** [all] Error 2
ihnorton commented 7 years ago

What OS? What version of CMake? This error happens because the build script sets the CMAKE_RUNTIME_OUTPUT_DIRECTORY variable, but apparently CMake did not create that directory on your system. (maybe it exists but not readable - permissions issue?)

I just tested cmake 2.8 and 3.7 on Linux (eris cluster), and 3.6 on Mac, and could not reproduce.

ihnorton commented 7 years ago

Hmm, I guess this wouldn't happen until later in the build because of dependencies so I should probably run it for longer. I usually run make -j2 or more, so maybe some other build step creates [superbuild]/bin first. Perhaps with single-process build the ordering is such that teem tries to build before which ever other step creates the directory. I guess we could add an explicit step to create the directory.

ihnorton commented 7 years ago

Fix tested on linux and mac, and committed.