-
Notifications
You must be signed in to change notification settings - Fork 28
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
Relative or absolute CMAKE_INSTALL_XXXDIR variables. #16
Comments
Hi! Could you provide steps to reproduce and environment (distro, compiler version, etc)? Additionally some background of why you needed to customize Thanks. |
I use Debian unstable, but it seems my compiler is from "experimental". Environment info (with some superfluous data removed):
I don't really need... I just wanted to. :-) Everything worked fine and OGRE 4.0 is working, thanks to you guys. I also got OGRE installed to my Trying to learn Ogre, I have noticed those files I have mentioned, but I don't really know what they are and how they affect me. :-) I guess that regardless of my needs, https://cmake.org/cmake/help/latest/module/GNUInstallDirs.html |
I assume you ran ? cmake .. -DCMAKE_INSTALL_PREFIX=/home/andre/.local/
Gotcha. The thing is this repo is only meant to build dependencies for ogre-next to pick up. By default this repo doesn't even install in any system path: it installs into a local folder (inside the build you told CMake). And the main issue is that we'd have to test all platforms we support to ensure nothing broke. That is why I will have to reject your PR unless you have objections. Cheers |
You do not need to officially support the thing. As per CMake documentation, the current version is wrong and the PR makes it right. This is regardless of anything. If you say you do not support But I do not have "objections". I am to new here to have "objections". But I do hope I have changed your mind. :-) Just keep up the great work and I am happy! Please, feel very comfortable to make your decision. I shall not complain. :-D |
I am not really familiar with
cmake
. I am very sorry if I am reporting something silly. :-)I have used
ogre-next-deps
. I have setupCMAKE_CXX_FLAGS
andCMAKE_INSTALL_PREFIX
. This is the content of mysrc/CMakeLists.txt
:Now, I have realized that the generated files
pkgconfig/shaderc*.pc
give a wronglibdir
:I do not really understand much... but I see two options:
@CMAKE_INSTALL_LIBDIR@
should simply belib
, but it is being configured with an absolute path. The file./build/src/shaderc-prefix/tmp/shaderc-cfgcmd.txt
contains-DCMAKE_INSTALL_PREFIX:PATH=/home/andre/.local;-DCMAKE_INSTALL_LIBDIR:PATH=/home/andre/.local/lib/;-DCMAKE_INSTALL_BINDIR:PATH=/home/andre/.local/bin/
.includedir
,${prefix}
should not be there either. And "two wrongs made a right", because@CMAKE_INSTALL_INCLUDEDIR@
is relative, not absolute.The template files are like this:
Maybe none of them (
libdir
orincludedir
) should have a${prefix}/
. It is strange, in my opinion, that@CMAKE_INSTALL_INCLUDEDIR@
actually gives a relativeinclude
path, instead of an absolute path. But this is also true forSYSCONFDIR
,DATAROOTDIR
, etc.I do not really know which one is the correct one. But making all those
XXXDIR
relative toPREFIX
is what makes more sense. In this case,BINDIR
andLIBDIR
are wrong.The text was updated successfully, but these errors were encountered: