Bug 204836 - devel/spice-protocol: update to 0.12.10
Summary: devel/spice-protocol: update to 0.12.10
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: --- Affects Only Me
Assignee: Roman Bogorodskiy
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2015-11-26 22:03 UTC by Oleg Ginzburg
Modified: 2015-11-27 15:01 UTC (History)
0 users

See Also:
bugzilla: maintainer-feedback? (novel)

Attachments
update to 0.12.10 (1.66 KB, patch)
2015-11-26 22:08 UTC, Oleg Ginzburg 		no flags Details | Diff
spice-protocol 0.12.10 update (1.64 KB, patch)
2015-11-26 23:06 UTC, Roman Bogorodskiy 		no flags Details | Diff
spice-protocol 0.12.10 update (2.53 KB, patch)
2015-11-27 11:06 UTC, Roman Bogorodskiy 		no flags Details | Diff
Show Obsolete (2) View All Add an attachment (proposed patch, testcase, etc.)

Note You need to log in before you can comment on or make changes to this bug.
Description Oleg Ginzburg 2015-11-26 22:03:44 UTC 
Trivial update to 0.12.10 (minimal version for spice-gtk 0.30)

* Add VD_AGENT_CAP_MONITORS_CONFIG_POSITION to handle multi-monitor
  configurations that are not multi-head
* Add protocol code generation scripts from spice-common
* Endianness and compiler portability fixes (clang)
Comment 1 Roman Bogorodskiy freebsd_committer freebsd_triage 2015-11-26 22:07:15 UTC 
Was that supposed to have a patch?

Anyway, I'm aware of 0.12.10 release availability. I was actually waiting for the next release that would include this patch:

http://lists.freedesktop.org/archives/spice-devel/2015-October/022148.html

Because it currently installs stuff into a wrong place. We could add a workaround for that though if that's blocking you somehow.
Comment 2 Oleg Ginzburg 2015-11-26 22:08:19 UTC 
Created attachment 163563 [details]
update to 0.12.10
Comment 3 Roman Bogorodskiy freebsd_committer freebsd_triage 2015-11-26 23:06:43 UTC 
Created attachment 163564 [details]
spice-protocol 0.12.10 update

So here's an updated patch that uses location for scripts that would be used in the upcoming upstream versions. Let me know what you think about this change.
Comment 4 Oleg Ginzburg 2015-11-27 10:14:31 UTC 
(In reply to Roman Bogorodskiy from comment #3)

Yes, your patch is more correct for FreeBSD hierarchy. However in this case, we need find and fix follow result from pkg-config:

% pkg-config --variable=codegendir spice-protocol

By default, it return /usr/local/lib/spice-protocol values. 
This execution and result used some configure scripts that use the spice-protocol (e.g spice-gtk 0.30) for searching spice.proto (and other) files. 

At the moment I do not know where pkg-config get values ( /usr/local/lib/spice-protocol )
Comment 5 Roman Bogorodskiy freebsd_committer freebsd_triage 2015-11-27 11:06:50 UTC 
Created attachment 163578 [details]
spice-protocol 0.12.10 update

How about this one?
Comment 6 Oleg Ginzburg 2015-11-27 11:43:36 UTC 
(In reply to Roman Bogorodskiy from comment #5)

work perfectly!
Comment 7 commit-hook freebsd_committer freebsd_triage 2015-11-27 14:58:57 UTC 
A commit references this bug:

Author: novel
Date: Fri Nov 27 14:58:10 UTC 2015
New revision: 402487
URL: https://svnweb.freebsd.org/changeset/ports/402487

Log:
  devel/spice-protocol: update to 0.12.10

  PR:		204836 (based on)
  Submitted by:	olevole@olevole.ru

Changes:
  head/devel/spice-protocol/Makefile
  head/devel/spice-protocol/distinfo
  head/devel/spice-protocol/files/
  head/devel/spice-protocol/files/patch-spice-protocol.pc.in
  head/devel/spice-protocol/pkg-plist