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

Storm should not write the (shipped) carl project to the cmake registry #62

Open
tquatmann opened this issue Dec 19, 2019 · 2 comments
Open

Comments

@tquatmann
Copy link
Member

If we have to build a shipped version of carl and one of the following happens:

  • something goes wrong in cmake after the configuration step of carl
  • something goes wrong in make before libcarl was build
  • the user runs cmake .. again without doing make (maybe because the user wants to change a cmake option)

every subsequent call to cmake leads to

CMake Error at resources/3rdparty/CMakeLists.txt:250 (message):
  File STORM_DIR/build/resources/3rdparty/carl/libcarl.so.14.19 does
  not exist, did you build carl?

Can this be fixed (with moderate effort)?
If not my suggestion would be to improve the error message.

A possible workaround is to delete the contents of the build folder and start again (which can be annoying).

@sjunges
Copy link
Contributor

sjunges commented Dec 19, 2019

Storm writes the carl project to the cmake registry. That causes the problem.

One problem is that pycarl currently needs this.
moves-rwth/pycarl#2 (comment)

To change that, storm needs to write where it put carl. That would be good anyway.
To improve the error message in case that does not work, we can then check whether the carl version is in some storm-subfolder and adapt the error message accordingly.

@tquatmann tquatmann changed the title If building shipped carl fails, we get CMake Error: libcarl does not exist, did you build carl? Storm should not write the (shipped) carl project to the cmake registry Sep 3, 2020
i2git pushed a commit that referenced this issue Sep 3, 2020
… shipped carl was aborted (bypassing github issue #62).
@tquatmann
Copy link
Member Author

tquatmann commented Sep 3, 2020

I now bypassed this problematic as follows:
In cases where the above-mentioned error would occur, we now check whether the directory $STORM_DIR/build/resources/3rdparty/carl exists. If yes, only a warning is printed stating that we try to build a shipped version of carl.

I know this is not a perfect solution (it might not work if there is a shipped version and a broken system version) but it should be fine in most cases. Of course, in the long run, the cmake registry thing mentioned by @sjunges needs to be fixed.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants