ros-visualization / rqt_image_view

http://wiki.ros.org/rqt_image_view
26 stars 59 forks source link

`#include <QSet>` is missing #29

Closed cubimon closed 4 years ago

cubimon commented 4 years ago

#include <QSet> is missing in image_view.cpp, which results in a compilation error on the arch linux aur, here is the issue. Do you want a PR or do you solve it by yourself?

fxkrait commented 4 years ago

I added the include statement, but I'm not sure how to test. I'm using the AUR helper trizen to build the package, and it only lets me edit the pkgbuild, not the source files themselves.

...update: I added the fix to the files in /tmp, but they keep getting overridden by the extraction of the .tar.gz during the build. I can't figure out how to edit the pkgbuild to prevent this.

Someone else can take over if they want ;).

cubimon commented 4 years ago

You may change the source in the PKGBUILD to your fork

cubimon commented 4 years ago

In the best case your pr is accepted and you don't have to change anything

fxkrait commented 4 years ago

Yeah, but the source in the pkgfile is a .tar.gz. My edits are raw source code in a directory. So I suppose I would have to create a new .tar.gz file from the new source files, and then point it to that in the pkgbuild?

fxkrait commented 4 years ago

I did the above: created a new .tar.gz from the new source, and update the sha256, and it compiles fine. I can also attach the new .tar.gz and sha256 sum, or I'm not sure if its created by some build server.

cubimon commented 4 years ago

Alternatively:

diff --git a/PKGBUILD b/PKGBUILD
index b708589..0034d1e 100644
--- a/PKGBUILD
+++ b/PKGBUILD
@@ -40,9 +40,9 @@ depends=(
    ${ros_depends[@]}
 )

-_dir="rqt_image_view-${pkgver}"
-source=("${pkgname}-${pkgver}.tar.gz"::"https://github.com/ros-visualization/rqt_image_view/archive/${pkgver}.tar.gz")
-sha256sums=('e24e26dc98404966cc827a4e576a3c7465d82076095d1831eb9da49d3a97c4d5')
+_dir="rqt_image_view"
+source=("git+https://github.com/greghab/rqt_image_view")
+sha256sums=('SKIP')

 build() {
    # Use ROS environment variables.
dirk-thomas commented 4 years ago

Resolved by #31.