{"action":"created","issue":{"url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/159","labels_url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/159/labels{/name}","comments_url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/159/comments","events_url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/159/events","html_url":"https://github.com/OPM/dune-cornerpoint/issues/159","id":97446662,"number":159,"title":"Cannot create a module thats depends on dune-cornerpoint","user":{"login":"berndflemisch","id":13166052,"avatar_url":"https://avatars.githubusercontent.com/u/13166052?v=3","gravatar_id":"","url":"https://api.github.com/users/berndflemisch","html_url":"https://github.com/berndflemisch","followers_url":"https://api.github.com/users/berndflemisch/followers","following_url":"https://api.github.com/users/berndflemisch/following{/other_user}","gists_url":"https://api.github.com/users/berndflemisch/gists{/gist_id}","starred_url":"https://api.github.com/users/berndflemisch/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/berndflemisch/subscriptions","organizations_url":"https://api.github.com/users/berndflemisch/orgs","repos_url":"https://api.github.com/users/berndflemisch/repos","events_url":"https://api.github.com/users/berndflemisch/events{/privacy}","received_events_url":"https://api.github.com/users/berndflemisch/received_events","type":"User","site_admin":false},"labels":[],"state":"open","locked":false,"assignee":null,"milestone":null,"comments":6,"created_at":"2015-07-27T11:57:52Z","updated_at":"2015-07-27T16:30:23Z","closed_at":null,"body":"When I try to build a Dune module that depends on dune-cornerpoint, cmake fails because apparently the variable \"dune-cornerpoint_PREFIX\" is not set correctly, see the output below. The output was generated for a fresh module named \"test\" created with duneproject that should depend on dune-grid and dune-cornerpoint. The problem should be easy to reproduce.\r\n\r\nI use DUNE 2.3.1 and the 2015.04 release of dune-cornerpoint.\r\n\r\n--- calling all for test ---\r\n--- calling vcsetup for test ---\r\n--- calling autogen for test ---\r\nSkipping dune-autogen because of CMake\r\n--- calling configure for test ---\r\n----- using default flags $CONFIGURE_FLAGS from /temp/bernd/DUMUX23/cmake5.0.opts -----\r\nWARNING: could not find module parallelgrid,\r\n       module is also unknown to pkg-config.\r\n       Maybe you need to adjust PKG_CONFIG_PATH!\r\n       parallelgrid is suggested by dune-cornerpoint\r\nSkipping parallelgrid!\r\n/temp/bernd/local/DUNE23CMake50/lib/dunegeometryam2cmake.lib for converting options does not exist\r\n/temp/bernd/local/DUNE23CMake50/lib/duneistlam2cmake.lib for converting options does not exist\r\n/temp/bernd/DUMUX23/opm-parser/lib/opmparseram2cmake.lib for converting options does not exist\r\n/temp/bernd/DUMUX23/opm-core/lib/opmcoream2cmake.lib for converting options does not exist\r\n/temp/bernd/DUMUX23/dune-cornerpoint/lib/dunecornerpointam2cmake.lib for converting options does not exist\r\n/temp/bernd/DUMUX23/test/lib/testam2cmake.lib for converting options does not exist\r\n CXXFLAGS=\"-Wall -Wunused -Wmissing-include-dirs -Wcast-align -Wno-sign-compare -Wno-packed-bitfield-compat -Wno-unused-parameter -Wno-unused-local-typedefs -Wno-deprecated-declarations -Wno-literal-suffix -Wunused-but-set-variable -fno-strict-aliasing -DHAVE_NULLPTR=1\" cmake -DCMAKE_MODULE_PATH=\"\"  -DUSE_MPI=ON \"-Ddune-common_DIR=/temp/bernd/local/DUNE23CMake50/lib64/cmake/dune-common\" \"-Ddune-geometry_DIR=/temp/bernd/local/DUNE23CMake50/lib64/cmake/dune-geometry\" \"-Ddune-grid_DIR=/temp/bernd/local/DUNE23CMake50/lib64/cmake/dune-grid\" \"-Ddune-istl_DIR=/temp/bernd/local/DUNE23CMake50/lib//cmake/dune-istl\" \"-Dopm-parser_DIR=/temp/bernd/DUMUX23/opm-parser/build-cmake-gcc50\" \"-Dopm-core_DIR=/temp/bernd/DUMUX23/opm-core/build-cmake-gcc50\" \"-Ddune-cornerpoint_DIR=/temp/bernd/DUMUX23/dune-cornerpoint/build-cmake-gcc50\" -DDUNE_GRID_EXPERIMENTAL_GRID_EXTENSIONS=1 -DDUMUX_BUILD_ALL_TESTS:BOOL=FALSE -DCMAKE_MODULE_PATH=/temp/bernd/local/DUNE23CMake50/share/cmake/modules -DCMAKE_PREFIX_PATH=\"/usr/lib64/mpi/gcc/openmpi;/temp/bernd/external/metis-5.1.0;/temp/bernd/external/ALUGrid-1.52;/temp/bernd/external/ug-3.12.1;/temp/bernd/DUMUX23/ert;/temp/bernd/local/DUNE23CMake50;/temp/bernd/local/DUNE23CMake50/share;/temp/bernd/local/DUNE23CMake50/share/cmake/modules;/temp/bernd/local/DUNE23CMake50/share/cmake/scripts;\" -DCMAKE_BUILD_TYPE=release -DCMAKE_CXX_COMPILER=/usr/bin/g++-5 -DCMAKE_C_COMPILER=/usr/bin/gcc-5 -Wno-dev --verbose --release -DCMAKE_DISABLE_FIND_PACKAGE_MPI=false \"/temp/bernd/DUMUX23/test\"\r\n-- Dependencies for test: dune-grid;dune-cornerpoint\r\ndune-grid, /temp/bernd/local/DUNE23CMake50\r\n-- Dependencies for dune-grid: dune-common (>= 2.3);dune-geometry (>= 2.3)\r\ndune-cornerpoint, \r\nCMake Error at /temp/bernd/local/DUNE23CMake50/share/dune/cmake/modules/DuneMacros.cmake:196 (message):\r\n  Could not find dune.module file for module dune-cornerpoint in ,\r\n  /lib/dunecontrol/dune-cornerpoint/\r\nCall Stack (most recent call first):\r\n  /temp/bernd/local/DUNE23CMake50/share/dune/cmake/modules/DuneMacros.cmake:350 (find_dune_package)\r\n  /temp/bernd/local/DUNE23CMake50/share/dune/cmake/modules/DuneMacros.cmake:393 (dune_process_dependency_leafs)\r\n  /temp/bernd/local/DUNE23CMake50/share/dune/cmake/modules/DuneMacros.cmake:417 (dune_create_dependency_leafs)\r\n  /temp/bernd/local/DUNE23CMake50/share/dune/cmake/modules/DuneMacros.cmake:546 (dune_create_dependency_tree)\r\n  CMakeLists.txt:19 (dune_project)\r\n\r\n\r\n-- Configuring incomplete, errors occurred!\r\nSee also \"/temp/bernd/DUMUX23/test/build-cmake-gcc50/CMakeFiles/CMakeOutput.log\".\r\n"},"comment":{"url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/comments/125265166","html_url":"https://github.com/OPM/dune-cornerpoint/issues/159#issuecomment-125265166","issue_url":"https://api.github.com/repos/OPM/dune-cornerpoint/issues/159","id":125265166,"user":{"login":"andlaus","id":1837735,"avatar_url":"https://avatars.githubusercontent.com/u/1837735?v=3","gravatar_id":"","url":"https://api.github.com/users/andlaus","html_url":"https://github.com/andlaus","followers_url":"https://api.github.com/users/andlaus/followers","following_url":"https://api.github.com/users/andlaus/following{/other_user}","gists_url":"https://api.github.com/users/andlaus/gists{/gist_id}","starred_url":"https://api.github.com/users/andlaus/starred{/owner}{/repo}","subscriptions_url":"https://api.github.com/users/andlaus/subscriptions","organizations_url":"https://api.github.com/users/andlaus/orgs","repos_url":"https://api.github.com/users/andlaus/repos","events_url":"https://api.github.com/users/andlaus/events{/privacy}","received_events_url":"https://api.github.com/users/andlaus/received_events","type":"User","site_admin":false},"created_at":"2015-07-27T16:30:23Z","updated_at":"2015-07-27T16:30:23Z","body":"> Just to be clear on the use case, are we talking about using an opm module in dumux as a dependency?\r\n\r\nI guess yes: As far as I can see, Bernd wants to write a simulator for Dumux using `Dune::CpGrid` as grid. IMO that's a legitimate use case -- the problem is that the Dune cmake build system (used by Dumux) and OPM's build system are currently mutually exclusive.\r\n\r\nI can see two fixes for this:\r\n\r\n1) keep the current build system, but change the naming conventions for all CMake variables to the ones of Dune.\r\n2) fully switch to the Dune cmake build system.\r\n\r\nboth are major endeavours, but I'd chose (2) if I had to decide this. (Also note that maybe I'm too pessimistic and may be sufficient to write a separate `Finddune-cornerpoint.cmake` module for the Dune build system. I would have to investigate the feasibility of that, though.)\r\n\r\n> I may be rusty on this, but I thought that it was possible to use dunecontrol to build opm and dune modules as one big happy family, have I misunderstood this?\r\n\r\nyes, it is possible to use dunecontrol to build OPM, and OPM modules can also specify dependencies on Dune modules. As far as I can see, the reverse is not easily possible, though."}}