Page MenuHomeFreeBSD

net/remmina: fix build with GVNC if gtk-vnc build without PULSEAUDIO and sort out dependencies
ClosedPublic

Authored by vvd on Feb 29 2024, 9:30 AM.
Tags
None
Referenced Files
Unknown Object (File)
Sat, Apr 27, 8:58 PM
Unknown Object (File)
Fri, Apr 26, 3:48 AM
Unknown Object (File)
Thu, Apr 25, 9:48 PM
Unknown Object (File)
Thu, Apr 25, 9:48 PM
Unknown Object (File)
Thu, Apr 25, 9:48 PM
Unknown Object (File)
Thu, Apr 25, 9:48 PM
Unknown Object (File)
Thu, Apr 25, 9:48 PM
Unknown Object (File)
Sun, Apr 21, 4:17 PM
Subscribers
None

Details

Summary

Changes:

  1. If net/gtk-vnc build with PULSEAUDIO off then build of the net/remmina fails if GVNC option is on:
pkg-static: Unable to access file /tmp/work/usr/ports/net/remmina/work-default/stage/usr/local/lib/remmina/plugins/remmina-plugin-gvnc.so:No such file or directory
pkg-static: Unable to access file /tmp/work/usr/ports/net/remmina/work-default/stage/usr/local/share/icons/hicolor/scalable/emblems/org.remmina.Remmina-gvnc-symbolic.svg:No such file or directory

Added patch to fix build remmina-plugin-gvnc.so plugin. Tested run using this plugin without pulse.

  1. Port have 2 options with same name GVNC.
  1. Options GVNC and VNC are mixed up.
  1. Removed unnecessary dependencies (from atk, pcre, jpeg, harfbuzz, gnutls, xkbfile and etc).
  1. Moved dependency from libsoup to WWW only.
  1. Replaced dependency from libfreerdp2.so (not linked) with libfreerdp-client2.so and libwinpr2.so.2 (linked).

Tested build with a lot of different combinations of options in poudriere and in live system 13.2-p10 amd64.
Tested run too.

Diff Detail

Repository
R11 FreeBSD ports repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

vvd requested review of this revision.Feb 29 2024, 9:30 AM
vvd created this revision.
net/remmina/Makefile
37

^

CMAKE_OFF=WITH_APPINDICATOR
vvd marked an inline comment as done.
vvd added inline comments.
net/remmina/Makefile
37

Agree. Done.

Thank you very much for the cleanup.
I had a build test to fix the VNC - GVNC mixup. Unfortunately it requires rebuilding rust and my machine dies since the box is doing other things that can't wait.

Thank you very much for the cleanup.
I had a build test to fix the VNC - GVNC mixup. Unfortunately it requires rebuilding rust and my machine dies since the box is doing other things that can't wait.

So should I wait until you test?

In D44152#1007320, @vvd wrote:

Thank you very much for the cleanup.
I had a build test to fix the VNC - GVNC mixup. Unfortunately it requires rebuilding rust and my machine dies since the box is doing other things that can't wait.

So should I wait until you test?

No need.

So you approve the review and I can commit it? :-)

In D44152#1007330, @vvd wrote:

So you approve the review and I can commit it? :-)

LGTM, but I don't feel comfortable approving without build testing.
I'll let the approval to your mentor.
You can use reviewed by: fernape in the commit log though :-)

I finally manage to do some testing.

Thanks for your work.

This revision is now accepted and ready to land.Feb 29 2024, 10:02 PM