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

sip Unable to find file "QtCore/QtCoremod.sip" #2

Closed
iPaddy opened this issue Oct 18, 2020 · 4 comments
Closed

sip Unable to find file "QtCore/QtCoremod.sip" #2

iPaddy opened this issue Oct 18, 2020 · 4 comments

Comments

@iPaddy
Copy link

iPaddy commented Oct 18, 2020

I don't know exactly if here is the right place to post this issue. But I had problems installing ros-noetic-qt-gui-cpp because of an error in
/opt/ros/noetic/share/python_qt_binding/cmake/sip_configure.py

where in line 57 and 62 both a default_sip_dir is defined but the first one crashed because with the error:
sip Unable to find file "QtCore/QtCoremod.sip"

commenting out lines 57-59 fixed the problem.
There probably just has to be a conditional beforehand checking for python3.
Any idea where to put the corresponding pull request? If it is valid anyways...

@acxz
Copy link
Member

acxz commented Oct 18, 2020

I don't know exactly if here is the right place to post this issue.

Finding the package that owns the file (pacman -Qo /opt/ros/noetic/share/python_qt_binding/cmake/cmake/sip_configure.py) points to ros-noetic-python-qt-binding. Transferring the issue over there.

Seems like this is also a dup of #1

@acxz acxz transferred this issue from ros-noetic-arch/ros-noetic-qt-gui-cpp Oct 18, 2020
@acxz
Copy link
Member

acxz commented Oct 18, 2020

Corresponding ros-melodic issue: ros-melodic-arch/ros-melodic-rviz#9
and corresponding upstream issue: ros-visualization/python_qt_binding#80

@acxz
Copy link
Member

acxz commented Oct 18, 2020

Using upstream sip path patch in 7dcebf0.

@iPaddy can you confirm if the updated package (0.4.3-2) works for you with the downstream packages that depend on it like ros-noetic-qt-gui-cpp?

@acxz
Copy link
Member

acxz commented Jan 31, 2022

Patch has been released in new version

@acxz acxz closed this as completed Jan 31, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants