Bug 104524 - [PATCH] Workaround missing openssl.pc on FreeBSD
Summary: [PATCH] Workaround missing openssl.pc on FreeBSD
Status: RESOLVED FIXED
Alias: None
Product: Spice
Classification: Unclassified
Component: spice-gtk (show other bugs)
Version: unspecified
Hardware: Other FreeBSD
: medium normal
Assignee: Spice Bug List
QA Contact:
URL:
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2018-01-07 10:00 UTC by Ting-Wei Lan
Modified: 2018-01-09 13:49 UTC (History)
2 users (show)

See Also:
i915 platform:
i915 features:


Attachments
build-sys: Workaround missing openssl.pc for FreeBSD (2.15 KB, patch)
2018-01-07 10:18 UTC, Ting-Wei Lan
Details | Splinter Review

Description Ting-Wei Lan 2018-01-07 10:00:48 UTC
FreeBSD always have OpenSSL installed, but .pc files are not available. We can still build spice-gtk by setting SSL_CFLAGS, SSL_LIBS, OPENSSL_CFLAGS, OPENSSL_LIBS variables, but openssl is still put into spice-client-glib-2.0.pc, which can cause problems for other programs because they cannot find openssl.pc.

I will upload to patch to workaround the issue.
Comment 1 Ting-Wei Lan 2018-01-07 10:18:38 UTC
Created attachment 136595 [details] [review]
build-sys: Workaround missing openssl.pc for FreeBSD

FreeBSD has OpenSSL installed in base, but .pc files are not available.
We can still successfully build spice-gtk by setting SSL_CFLAGS and
SSL_LIBS variables manually, but openssl will still be recorded to
Requires.private field of spice-client-glib-2.0.pc, causing problems for
applications using spice-gtk. To workaround the issue, we do the check
again without using SSL_CFLAGS and SSL_LIBS variables and put SSL_LIBS
into Libs.private field when openssl.pc cannot be found.
Comment 2 Christophe Fergeau 2018-01-08 11:31:01 UTC
Sent upstream
https://lists.freedesktop.org/archives/spice-devel/2018-January/041326.html
but it looks good to me, so I'll push it soon.


Use of freedesktop.org services, including Bugzilla, is subject to our Code of Conduct. How we collect and use information is described in our Privacy Policy.