Bug 186395 - net/freerdp fails to build on sparc64
Summary: net/freerdp fails to build on sparc64
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: Michael Moll
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2014-02-03 01:20 UTC by craig001
Modified: 2015-11-29 20:09 UTC (History)
1 user (show)

See Also:


Attachments
file.diff (3.17 KB, patch)
2014-02-03 01:20 UTC, craig001
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description craig001 2014-02-03 01:20:00 UTC
currently when building freerdp (as a dependency for xrdp) on a sparc64 machine build complains about -mfpu=neon and -mfloat-abi

I don't think sparc64 supports neon

Fix: nasty hack to remove all mention of --mfpu=neon and -mfloat-abi allows the port to build.

I think there maybe a way to pass in WITH_NEON=off or something to stop it trying to build with neon support...

Patch attached with submission follows:
How-To-Repeat: build net/freerdp on a sparc64 machine
Comment 1 Edwin Groothuis freebsd_committer 2014-02-03 01:20:10 UTC
Responsible Changed
From-To: freebsd-ports-bugs->fluffy

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Jan Beich freebsd_committer 2015-08-30 22:31:34 UTC
Reset assignee per ports r380659.
Comment 3 Michael Moll freebsd_committer 2015-11-29 20:04:00 UTC
take
Comment 4 commit-hook freebsd_committer 2015-11-29 20:06:42 UTC
A commit references this bug:

Author: mmoll
Date: Sun Nov 29 20:06:29 UTC 2015
New revision: 402618
URL: https://svnweb.freebsd.org/changeset/ports/402618

Log:
  net/freerdp: fix build on sparc64

  PR:		186395
  Obtained from:	FreeRDP commit 68619b6

Changes:
  head/net/freerdp/files/patch-cmake_ConfigOptions.cmake
Comment 5 Michael Moll freebsd_committer 2015-11-29 20:09:30 UTC
The fix here is to correct the architecture detection, see:
https://github.com/FreeRDP/FreeRDP/commit/68619b6

If anybody wants to the this MFHed, please ping me in the PR and I'll ask for permission to merge into 2015Q4.