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

[qtbase] Should have msg to install the correct dependencies for ubuntu #30329

Closed
talregev opened this issue Mar 21, 2023 · 5 comments · Fixed by #30854
Closed

[qtbase] Should have msg to install the correct dependencies for ubuntu #30329

talregev opened this issue Mar 21, 2023 · 5 comments · Fixed by #30854
Assignees
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist

Comments

@talregev
Copy link
Contributor

talregev commented Mar 21, 2023

I install qtbase port from part to install gazebo port.
And I encounter the error:

ERROR: Feature "xcb": Forcing to "ON" breaks its condition:
    QT_FEATURE_thread AND TARGET XCB::XCB AND TEST_xcb_syslibs AND QT_FEATURE_xkbcommon_x11
Condition values dump:
    QT_FEATURE_thread = "ON"
    TARGET XCB::XCB found
    TEST_xcb_syslibs = "FALSE"
    QT_FEATURE_xkbcommon_x11 not evaluated

I found the solution in old issue:
#16486
We should tell the user to install this like we do in many ports:
We should put msg inside qtbase and qt5-base in portfile.cmake and let the user to install the correct dependencies in ubuntu:
sudo apt-get install '^libxcb.*-dev' libx11-xcb-dev libglu1-mesa-dev libxrender-dev libxi-dev libxkbcommon-dev libxkbcommon-x11-dev

@talregev talregev changed the title [qt5-base] Should have msg to install the correct dependencies for ubuntu [qtbase] Should have msg to install the correct dependencies for ubuntu Mar 21, 2023
@Cheney-W Cheney-W added the category:port-bug The issue is with a library, which is something the port should already support label Mar 21, 2023
@Cheney-W
Copy link
Contributor

@talregev Do below tips in qt5-base/portfile.cmake meet your expectations?

CMake Warning at ports/qt5-base/portfile.cmake:4 (message):
  qt5-base currently requires some packages from the system package manager,
  see https://doc.qt.io/qt-5/linux-requirements.html

CMake Warning at ports/qt5-base/portfile.cmake:5 (message):
  qt5-base for qt5-x11extras requires several libraries from the system
  package manager.  Please refer to
    https://github.com/microsoft/vcpkg/blob/master/scripts/azure-pipelines/linux/provision-image.sh
    for a complete list of them.

@talregev
Copy link
Contributor Author

@talregev Do below tips in qt5-base/portfile.cmake meet your expectations?

CMake Warning at ports/qt5-base/portfile.cmake:4 (message):
  qt5-base currently requires some packages from the system package manager,
  see https://doc.qt.io/qt-5/linux-requirements.html

CMake Warning at ports/qt5-base/portfile.cmake:5 (message):
  qt5-base for qt5-x11extras requires several libraries from the system
  package manager.  Please refer to
    https://github.com/microsoft/vcpkg/blob/master/scripts/azure-pipelines/linux/provision-image.sh
    for a complete list of them.

@Cheney-W Thank you for the replay! I prefer that it tell me which packages I need to install other just send me to run the script. Because in this script there many stuff that not connected to the qt5-base nor qtbase.
Can you change it to the line I wrote?

@Cheney-W
Copy link
Contributor

'^libxcb.*-dev' include 34 packages, are they all necessary?

@talregev
Copy link
Contributor Author

talregev commented Mar 31, 2023

'^libxcb.*-dev' include 34 packages, are they all necessary?

I try some of them and it didn't work. I am not sure if needed all, but all is working.
Also all the 34 packages is a small size.

@talregev
Copy link
Contributor Author

talregev commented Mar 31, 2023

This line:
sudo apt-get install '^libxcb.*-dev' libx11-xcb-dev libglu1-mesa-dev libxrender-dev libxi-dev libxkbcommon-dev libxkbcommon-x11-dev
Should be on qt5-base and qtbase ports.

@Cheney-W Cheney-W added category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist and removed category:port-bug The issue is with a library, which is something the port should already support labels Apr 6, 2023
fwcd added a commit to fwcd/m1xxx that referenced this issue Nov 9, 2023
fwcd added a commit to fwcd/m1xxx that referenced this issue Nov 12, 2023
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
category:port-feature The issue is with a library, which is requesting new capabilities that didn’t exist
Projects
None yet
2 participants