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

cmake fails due to unify already existing #1

Closed
StrangeTcy opened this issue Jul 9, 2019 · 6 comments
Closed

cmake fails due to unify already existing #1

StrangeTcy opened this issue Jul 9, 2019 · 6 comments

Comments

@StrangeTcy
Copy link

Trying to cmake .. from /ure/build leads to the following error:

-- Build type: Release
-- CogUtil version 2.0.3 found.
-- AtomSpace found.
-- Boost version: 1.65.1
-- Found the following Boost libraries:
--   date_time
--   filesystem
--   serialization
--   system
--   thread
--   chrono
--   atomic
-- Boost version 106501 found.
-- CxxTest found.
-- Guile (2.2.3 >= 2.2.2) was found.
-- Python 3.7.3 interpreter found.
-- Python 3.7.3 libraries found.
-- Cython ( 0.29.1 >= 0.23.0) found.
-- Python destination dir found: /usr/local/lib/python3.7/dist-packages
-- Python install dir: /usr/local/lib/python3.7/dist-packages/opencog
-- Using nosetests executable /usr/bin/nosetests3
-- Valgrind Prefix: 
-- VALGRIND was found.
-- VALGRIND devel headers found.
CMake Error at opencog/unify/CMakeLists.txt:1 (ADD_LIBRARY):
  ADD_LIBRARY cannot create target "unify" because an imported target with
  the same name already exists.


CMake Error at opencog/unify/CMakeLists.txt:5 (TARGET_LINK_LIBRARIES):
  Cannot specify link libraries for target "unify" which is not built by this
  project.

@ngeiswei
Copy link
Member

ngeiswei commented Jul 9, 2019

You probably need to clean you system from existing cmake config files in /usr/local/include/cmake/AtomSpace. But it will still not work because the PR removing the ure from the atomspace hasn't been merged yet (I wanted to do that today but problem showed up with circleci, maybe I should do it anyway).

@ngeiswei
Copy link
Member

Also, please this repo is not ready yet, I'm gonna rewrite its git history pretty soon.

ngeiswei pushed a commit to ngeiswei/ure that referenced this issue Jul 10, 2019
@StrangeTcy
Copy link
Author

StrangeTcy commented Jul 10, 2019

Okay, but, as far as I can understand it, the main opencog as it is now doesn't build NLP, PLN and Miner, being unable to find URE.
Is there some way around that? Perhaps there's an older branch of opencog that doesn't yet depend on URE?

Anyway, am I right to think that the current best advice is along the lines of "Just wait a bit"?

@ngeiswei
Copy link
Member

@StrangeTcy, I've merged opencog/atomspace#2256 however NLP, PLN and Miner will not be built by circleci till it is fixed. I'm looking into that.

BTW, I'm happy to take care of the oc->sn to not create overhead on your side, unless you prefer to do it for some reason.

@StrangeTcy
Copy link
Author

StrangeTcy commented Jul 11, 2019

Ok, cmake .. succeeds now, and so do make and sudo make install as well, so I should perhaps close the issue.
Then again, perhaps this a temporary state :-)

@ngeiswei
Copy link
Member

It should be permanent now (just need to fix the circleci for ure).

ngeiswei added a commit to ngeiswei/ure that referenced this issue Jul 15, 2019
Fix negative max iter (which means infinity) + add utest
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants