Bug 207569

Summary: [patch] deskutils/spice-gtk fix 2 issues.
Product: Ports & Packages Reporter: Koop Mast <kwm>
Component: Individual Port(s)Assignee: Koop Mast <kwm>
Status: Closed FIXED    
Severity: Affects Only Me CC: olevole
Priority: --- Keywords: patch
Version: LatestFlags: bugzilla: maintainer-feedback? (olevole)
Hardware: Any   
OS: Any   
Attachments:
Description Flags
Fix issues with the spice-client-glib-2.0.pc file none

Description Koop Mast freebsd_committer freebsd_triage 2016-02-28 20:06:15 UTC
Created attachment 167528 [details]
Fix issues with the spice-client-glib-2.0.pc file

The current spice-gtk port has two issues both are with the spice-client-glib-2.0.pc file.

1) spice-gtk needs a run dependancy on spice-protocol because it is required in the spice-client-glib-2.0.pc file.

% pkg-config --cflags spice-client-glib-2.0
Package spice-protocol was not found in the pkg-config search path.
Perhaps you should add the directory containing `spice-protocol.pc'
to the PKG_CONFIG_PATH environment variable
Package 'spice-protocol', required by 'spice-client-glib-2.0', not found

2) it also put a requirement on the openssl.pc file in the spice-client-glib-2.0.pc. I just removed it, and vinagre build and run fine.
Comment 1 Koop Mast freebsd_committer freebsd_triage 2016-02-28 20:07:05 UTC
Grab, I have interest in getting this in.
Comment 2 commit-hook freebsd_committer freebsd_triage 2016-03-17 10:28:40 UTC
A commit references this bug:

Author: kwm
Date: Thu Mar 17 10:28:05 UTC 2016
New revision: 411267
URL: https://svnweb.freebsd.org/changeset/ports/411267

Log:
  These two changes allow vinagre to enable spice support.

  Add spice-protocol to run_depends since the spice-client-glib-2.0
    pkgconfig file requires it.
  Also remove the requirement on the openssl from the spice-client-glib-2.0
    pkgconfig file.

  PR:		207569
  Approved by:	maintainer timeout (18 days)

Changes:
  head/deskutils/spice-gtk/Makefile
  head/deskutils/spice-gtk/files/
  head/deskutils/spice-gtk/files/patch-configure