agemuend

Results 7 comments of agemuend

I'm also hit by this using Windows 10 and no Google Drive. I don't see anything special in the logs, apart from this: 2023-04-21 12:22:40:498 [/mediastreamer] MESSAGE Fail to load...

> As you can see, this breaks installation of older ipp versions. Mhm, thats interesting. The error is "directory already exists", but my patch actually _introduces_ the suppression of making...

> Yes it works before this PR. See my fix for the problem above. Sorry, where exactly?

> this should be probed by the buildsystem, https://github.com/VikingScientist/LRsplines/blob/master/CMakeLists.txt#L89 did you build in a clean build directory? Yes, I did. I'll trace the cmake output to see why the check...

Okay, indeed I see it tried: -- Check size of Go::BasisDerivsSf3 - failed It seems to me that CMake doesn't use the include paths with check_type_size()? At least it fails...

It seems a bit redundant, but this works: ``` --- a/CMakeLists.txt +++ b/CMakeLists.txt @@ -84,6 +84,7 @@ IF(GoTools_FOUND) set(LRSpline_DEFINITIONS "${LRSpline_DEFINITIONS} -DHAS_GOTOOLS") include(CheckTypeSize) + set(CMAKE_REQUIRED_INCLUDES ${GoTools_INCLUDE_DIRS}) set(CMAKE_EXTRA_INCLUDE_FILES "GoTools/geometry/SplineSurface.h") set(CMAKE_REQUIRED_FLAGS -std=c++11) check_type_size(Go::BasisDerivsSf3...

I thought because its set in INCLUDES before, but if its right, even better. ;)