Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Search key at find_program-ing nosetests' binary might be too specific #461

Closed
130s opened this issue Jul 8, 2013 · 3 comments
Closed
Labels

Comments

@130s
Copy link
Member

130s commented Jul 8, 2013

On the platform I'm trying (QNX 6.5 with pkgsrc), cmake-ing the workspace fails as follows. Binary of nosetests is found as /usr/pkg/bin/nosetests-2.7.

$ cd catkinws && mkdir build && cmake ../src
  :
-- Using CATKIN_TEST_RESULTS_DIR: /home/rosuser/catkin_ws/build/test_results
-- Found gtest sources under '/home/rosuser/catkin_ws/src/gtest': gtests will be built
CMake Warning at catkin/cmake/test/nosetests.cmake:94 (message):
  nosetests not found, Python tests can not be run (try installing package
  'python-nose')
Call Stack (most recent call first):
  catkin/cmake/all.cmake:134 (include)
  CMakeLists.txt:12 (include)


CMake Error at catkin/cmake/assert.cmake:17 (message):


  Assertion failed: check for file existence, but filename
  (RT_LIBRARY-NOTFOUND) unset.  Message: RT Library

Call Stack (most recent call first):
  catkin/cmake/tools/rt.cmake:42 (assert_file_exists)
  catkin/cmake/all.cmake:134 (include)
  CMakeLists.txt:12 (include)


-- Configuring incomplete, errors occurred!

catkin 0.5.69

I'll send a patch right away.

@dirk-thomas
Copy link
Member

According to your output the finding nosetests only results in a warning. The real error comes from being unable to find the rt library. Do you have a fix for this as well?

@130s
Copy link
Member Author

130s commented Jul 8, 2013

Not yet, but I'm trying to make a patch for rt issue too.

@dirk-thomas
Copy link
Member

Closing due to lack of activity and since pending stuff is not at all related to this issues subject.

eitanme pushed a commit to eitanme/catkin that referenced this issue Nov 15, 2013
)

I'm not sure if this is the best patch for the targetted issue, since I'm not an expert with `cmake`. But this seems to find `nosetests-2.7` on my `QNX`.
eitanme pushed a commit to eitanme/catkin that referenced this issue Nov 15, 2013
Add another major version number pattern in find_program (fix to ros#461)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

2 participants