Bug 182778 - Build failure for net/freerdp using non-default options
Summary: Build failure for net/freerdp using non-default options
Status: Closed FIXED
Alias: None
Product: Ports & Packages
Classification: Unclassified
Component: Individual Port(s) (show other bugs)
Version: Latest
Hardware: Any Any
: Normal Affects Only Me
Assignee: Dima Panov
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-10-07 03:20 UTC by Kevin Zheng
Modified: 2013-10-07 14:10 UTC (History)
0 users

See Also:

Attachments
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 Kevin Zheng 2013-10-07 03:20:00 UTC 
Several non-default options cause build failures for net/freerdp.

ALSA (off by default) causes a configuration failure:

CMake Error at /usr/local/share/cmake/Modules/FindPackageHandleStandardArgs.cmake:108 (message):
  Could NOT find ALSA (missing: ALSA_LIBRARY ALSA_INCLUDE_DIR)

Build Log: http://quasarp.dyndns.org:8080/logs/bulk/9-i386-default/2013-10-05_20h45m05s/logs/errors/freerdp-1.0.2.log

SSE (off by default) causes a build failure:

/wrkdirs/usr/ports/net/freerdp/work/FreeRDP-FreeRDP-616aed4/libfreerdp-codec/rfx_sse2.c: In function 'rfx_quantization_encode_sse2':
/wrkdirs/usr/ports/net/freerdp/work/FreeRDP-FreeRDP-616aed4/libfreerdp-codec/rfx_sse2.c:276: error: shift must be an immediate

Build Log: http://quasarp.dyndns.org:8080/logs/bulk/9-i386-default/latest/logs/errors/freerdp-1.0.2.log

Note that the machine with the build logs may not be accessible all the time.

Fix: 

The temporary fix is to mark the problematic options with BROKEN or disable them altogether. I have no idea what the long-term fix is.
How-To-Repeat: Build net/freerdp using one or more of the problematic options listed above.
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-10-07 03:20:09 UTC 
Responsible Changed
From-To: freebsd-ports-bugs->fluffy

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 dfilter service freebsd_committer freebsd_triage 2013-10-07 14:07:25 UTC 
Author: fluffy
Date: Mon Oct  7 13:07:16 2013
New Revision: 329700
URL: http://svnweb.freebsd.org/changeset/ports/329700

Log:
  - Add missed dependency on ALSA when option is selected
  
  PR:		182778
  Submitted by:	Kevin Zheng

Modified:
  head/net/freerdp/Makefile

Modified: head/net/freerdp/Makefile
==============================================================================
--- head/net/freerdp/Makefile	Mon Oct  7 12:52:34 2013	(r329699)
+++ head/net/freerdp/Makefile	Mon Oct  7 13:07:16 2013	(r329700)
@@ -53,6 +53,7 @@ PLIST_SUB+=	DIRECTFB="@comment "
 .endif
 
 .if ${PORT_OPTIONS:MALSA}
+LIB_DEPENDS+=	asound:${PORTSDIR}/audio/alsa-lib
 PLIST_SUB+=	ALSA=""
 .else
 PLIST_SUB+=	ALSA="@comment "
_______________________________________________
svn-ports-all@freebsd.org mailing list
http://lists.freebsd.org/mailman/listinfo/svn-ports-all
To unsubscribe, send any mail to "svn-ports-all-unsubscribe@freebsd.org"
Comment 3 Dima Panov freebsd_committer freebsd_triage 2013-10-07 14:09:23 UTC 
State Changed
From-To: open->closed

ALSA dependency fixed, SSE is always builds fine on sse-capable machines