mollynnobrien / b-tk

Automatically exported from code.google.com/p/b-tk
0 stars 0 forks source link

CMake fails for r1147 #31

Closed GoogleCodeExporter closed 8 years ago

GoogleCodeExporter commented 8 years ago
What steps will reproduce the problem?
1. Checkout revision r1147 under ubuntu 12.04
2. Try to configure using CMake with BTK_USE_VISSUPPORT, BTK_USE_VTK and 
BTK_TOOLS
3.

What is the expected output? What do you see instead?
Proper configure run-through of CMake. Instead it fails with an error of the 
form:
CMake Error at CMakeLists.txt:375 (CONFIGURE_FILE):
  configure_file input location

    /<path_to_btk>/btk-hg/CMake

  is a directory but a file was expected.

Furthermore various header files are not found.

What version of the product are you using? On what operating system?
r1147, Ubuntu 12.04, vtk 5.8 (Ubuntu package)

Please provide any additional information below.
Error is due to wrongly used variable CMAKE_MODULE_PATH near line 373. It 
contains two paths after calling FIND_PACKAGE(VTK REQUIRED) which confuses 
CMake. Proper variable would be CMAKE_CURRENT_SOURCE_DIR/CMake.

Attached a patch that fixes all issues.

Original issue reported on code.google.com by martin.f...@gmail.com on 21 Jun 2012 at 2:52

Attachments:

GoogleCodeExporter commented 8 years ago
This problem was also encountered with VTK-5.10 and was fixed in the branch 
vtk-5100 (temporary branch used to update the code of BTK with the latest 
version of VTK). The fix is in the revision r1125 
(https://code.google.com/p/b-tk/source/diff?spec=svn1125&r=1125&format=side&path
=/BTK/branches/vtk5100/CMakeLists.txt).

This branch will be merged in the trunk as soon as all the functionalities of 
the charts in Mokka will be restored.

Original comment by arnaud.barre on 21 Jun 2012 at 3:28

GoogleCodeExporter commented 8 years ago
All right.

I just tried to use branch 0.1.x however the same issue occurs there. I 
attached a patch that also fixes a missing include in 
Tools/Mokka/LoggerWidget.h and an error that occurs when trying to run "make 
install" for Mokka.

With the attached patch and vtk 5.6 both btk-0.1.x and Mokka can be compiled 
and installed under Ubuntu 12.04. It would be nice if this could be applied to 
the 0.1.x branch.

Original comment by martin.f...@gmail.com on 26 Jun 2012 at 8:26

Attachments:

GoogleCodeExporter commented 8 years ago
I commited the revision r1149 which should fix this issue in the trunk. Could 
you try to compile the code in the trunk? If it works, I will merge the 
revision in the branch btk-0.1.x.

Original comment by arnaud.barre on 27 Jun 2012 at 9:15

GoogleCodeExporter commented 8 years ago
Using revision r1149 and vtk 5.6.1 everything works perfect!

Thanks!

Original comment by martin.f...@gmail.com on 28 Jun 2012 at 1:18

GoogleCodeExporter commented 8 years ago
This issue was fixed in the trunk by the revions r1149 and r1203. The fix was 
also merged in the branch btk-0.1.x (see revision r1277).

Original comment by arnaud.barre on 23 Sep 2012 at 6:13