storm icon indicating copy to clipboard operation
storm copied to clipboard

Cudd installed via home-brew clash

Open sjunges opened this issue 1 year ago • 4 comments

If i try to make storm, i run into the following issue.

[ 39%] Building CXX object src/storm/CMakeFiles/storm.dir/abstraction/AbstractionInformation.cpp.o
...
/Users/junges/storm/src/storm/storage/dd/cudd/InternalCuddBdd.h:46:67: error: use of undeclared identifier 'cudd'; did you mean 'Cudd'?
    InternalBdd(InternalDdManager<DdType::CUDD> const* ddManager, cudd::BDD cuddBdd);
                                                                  ^~~~
                                                                  Cudd
/opt/homebrew/include/cuddObj.hh:448:7: note: 'Cudd' declared here
class Cudd {

The only way i managed to work around it so far is by uninstalling yices2 from brew as this uninstalled cudd3 from brew....

sjunges avatar Jun 27 '23 13:06 sjunges

Would it be an option to rename resources/3rdparty/cudd-3.0.0/cplusplus/cuddObj.hh to something like resources/3rdparty/cudd-3.0.0/cplusplus/storm_cuddObj.hh ?

tquatmann avatar Jun 28 '23 08:06 tquatmann

Indeed, that could be one option. Another option could be to change the include dir and make sure that we have to include <storm-cudd/cplusplus/cuddObj.h>? I dont know what is better.

sjunges avatar Jun 28 '23 08:06 sjunges

Yes, that sounds reasonable, too. I also don't know what is better 😄

tquatmann avatar Jun 28 '23 12:06 tquatmann

Dagstuhl agrees: storm-cudd/cplusplus/cuddObj.h is better.

sjunges avatar Mar 25 '24 18:03 sjunges