-
Notifications
You must be signed in to change notification settings - Fork 18
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
Prefix is doubled in cmake INTERFACE_INCLUDE_DIRECTORIES #18
Comments
I'll look into it. I know where to find your build scripts but it would still be nice to know which options where passed to CMake exactly (after all shell variables have been expanded). I'm wondering that you're building c++utilities as static library although you're targeting a GNU/Linux distribution. Is that intended? By the way, it is likely that the support for GNUInstallDirs broke it. So the build system is now aware of the following CMake variables: https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html |
This is where you can find some of the flags we set for cmake based builds: |
I've really meant the concrete arguments after everything is expanded, e.g. a log like https://build.opensuse.org/package/live_build_log/home:mkittler/c++utilities/openSUSE_Tumbleweed/x86_64. |
And is the error only in the config file or are the headers installed into the wrongly prefixed directory, too? By the way, the variable |
Here you go:
The files themselves are installed fine. It's just this very line that seems wrong. |
The problem is that you actually pass an absolute path and my build script assumes that Seems like I should allow absolute paths, too. I guess using |
Passing an absolute path to |
Qtutilities and syncthingtray compile and work fine now. Thanks! |
Same as #18 but for the share directory.
See Martchus/cpp-utilities#18 (cherry picked from commit d7c6c9a)
On NixOS, we are having trouble with the file written to
$prefix/share/c++utilities/cmake/c++utilitiesTargets.cmake
. See this output in that file:As you can see, the prefix is written twice from some reason. This issue makes the build for qtutilities and syncthingtray fail.
The text was updated successfully, but these errors were encountered: