Skip to content
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

FindXCB cmake module not found #5758

Closed
ycollet opened this issue Nov 1, 2020 · 4 comments
Closed

FindXCB cmake module not found #5758

ycollet opened this issue Nov 1, 2020 · 4 comments
Labels

Comments

@ycollet
Copy link
Contributor

ycollet commented Nov 1, 2020

Under Fedora, the FindXCB module needed by src/3rdparty/qt5-x11embed/CMakeLists.txt is found in:

 /usr/share/ECM/find-modules/ 

So, as a workaround, I added the following line in the principal CMakeLists.txt file of lmms:

SET(CMAKE_MODULE_PATH /usr/share/ECM/find-modules/  ${CMAKE_MODULE_PATH})

The file FindXCB.cmake is provided by the "extra-cmake-modules" package.

@ycollet ycollet added the bug label Nov 1, 2020
@PhysSong
Copy link
Member

PhysSong commented Nov 1, 2020

It's in a recursive submodule, so you should init it properly.

@ycollet
Copy link
Contributor Author

ycollet commented Nov 1, 2020

Yes, but this file: FindXCB.cmake should have been shipped by qt5-x11embed in the first place ...

@tresf
Copy link
Member

tresf commented Nov 11, 2020

Yes, but this file: FindXCB.cmake should have been shipped by qt5-x11embed in the first place ...

Why? It's a source-only submodule which pulls in another source-only submodule? What's wrong with the current technique?

https://github.com/lukas-w/qt5-x11embed/blob/022b39a1d496d72eb3e5b5188e5559f66afca957/CMakeLists.txt#L4

The current technique already relies on the submodule, which means it'll always be available. LMMS relies on a ton of these currently #3963.

If you prefer to use a system-provided package instead, that may be a good justification for a PR against qt5-x11embed, or an issue against that project. Another option may be to switch the project to a git subtree instead if warranted, which has the advantage of including the submodule in tarballs, etc.

@PhysSong
Copy link
Member

PhysSong commented Jun 5, 2021

If you prefer to use a system-provided package instead, that may be a good justification for a PR against qt5-x11embed, or an issue against that project.

Agreed. Closing.

@PhysSong PhysSong closed this as completed Jun 5, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

No branches or pull requests

3 participants