-
Notifications
You must be signed in to change notification settings - Fork 23
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
moc-ng wants to ovewrite Qt's moc under /usr/bin/ #13
Comments
This patch helped but still, is there a better way? From 62b06474c31f9d0674f5139e584711e21f4fb702 Mon Sep 17 00:00:00 2001
From: Petros Siligkounas <[email protected]>
Date: Wed, 13 Feb 2019 20:48:13 +0200
Subject: [PATCH] Rename moc target as moc-ng, so it doesn't overwrite Qt's moc
---
src/CMakeLists.txt | 4 ++++
1 file changed, 4 insertions(+)
diff --git a/src/CMakeLists.txt b/src/CMakeLists.txt
index e38fbd1..bcf57ea 100644
--- a/src/CMakeLists.txt
+++ b/src/CMakeLists.txt
@@ -98,6 +98,10 @@ configure_file(embedded_includes.h.in embedded_includes.h)
include_directories(${CMAKE_CURRENT_BINARY_DIR})
include("GNUInstallDirs")
+#Installing moc under /usr/bin will erase Qt's moc
+if(${CMAKE_BUILD_TYPE} MATCHES "Gentoo")
+ set_target_properties(moc PROPERTIES OUTPUT_NAME "moc-ng")
+endif()
install(TARGETS moc mocng_plugin
RUNTIME DESTINATION ${CMAKE_INSTALL_BINDIR}
LIBRARY DESTINATION ${CMAKE_INSTALL_LIBDIR} |
The idea is to overwrite Qt's moc so it is used instead, as it is supposed to be a drop-in replacement. I admit not everybody wants that, but if we give it another name, it makes it harder to use. |
I am thinking about another idea. Maybe This has to be supported from the distribution and Qt's packaging so I don't hold my breath. Thank you anyway. Feel free to close this issue is there is nothing to be done (at least now). |
Trying to install moc-ng with Portage gives me this :
The text was updated successfully, but these errors were encountered: