|Summary:||net/freerdp: fix build with libressl-devel|
|Product:||Ports & Packages||Reporter:||Matthew Rezny <rezny>|
|Component:||Individual Port(s)||Assignee:||freebsd-ports-bugs (Nobody) <ports-bugs>|
|Severity:||Affects Some People||CC:||kevans|
Description Matthew Rezny 2017-02-10 21:31:51 UTC
Created attachment 179853 [details] fix build with libressl-devel Add a patch based on one from OpeBSD in order to allow to build with libressl-devel. This patch differs from theirs since OpenBSD has FreeRDP 1.2 in their ports tree. While here, change the SSE option to only be defied for x86 instead of merely excluded from ARM and default it ON on amd64 since SSE2 support is always present there. I have noticed that the build fails when a past revision of the port is installed because the linking of xfreerdp picks up linfreerdp.so from localbase instead of from the build directory, but I have not addressed that issue in this patch.
Comment 1 Kyle Evans 2017-02-12 01:36:17 UTC
Comment on attachment 179853 [details] fix build with libressl-devel Hi, This looks good to me; especially +1 on the SSE change. Test build runs fine w/ OpenSSL, unfortunately I don't have a quick way to build and test this with LibreSSL -- is it safe to assume you've tested this, at least briefly, with such a setup?
Comment 2 Matthew Rezny 2017-02-14 00:37:00 UTC
Yes, I've verified it builds and runs with libressl-devel. I'm confident in the first chunk of the patch since it comes from OpenBSD. The other change was my quick solution to get it to build as I saw no way to directly manipulate that value which is now internal. The affected code is a work-around for an incorrect client, nothing critical.
Comment 3 Kyle Evans 2017-02-14 00:47:06 UTC
Comment on attachment 179853 [details] fix build with libressl-devel Cool, thanks. =) I had kind of figured this was to address your own use-case and thus, tested, but wanted to verify.
Comment 4 commit-hook 2017-02-21 14:34:42 UTC
A commit references this bug: Author: rezny Date: Tue Feb 21 14:33:55 UTC 2017 New revision: 434532 URL: https://svnweb.freebsd.org/changeset/ports/434532 Log: Fix build with libressl-devel. Restrict SSE option to x86, default on amd64. PR: 216980 Approved by: firstname.lastname@example.org (maintainer), swills (mentor) Obtained from: OpenBSD Differential Revision: https://reviews.freebsd.org/D9671 Changes: head/net/freerdp/Makefile head/net/freerdp/files/patch-libfreerdp_crypto_tls.c