Bug 165814

Summary: Build multimedia/dirac with clang [patch]
Product: Ports & Packages Reporter: Shane <FreeBSD>
Component: Individual Port(s)Assignee: Martin Wilke <miwi>
Status: Closed FIXED    
Severity: Affects Only Me    
Priority: Normal    
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
file.shar none

Description Shane 2012-03-07 11:20:10 UTC
The attached patches allow multimedia/dirac to be compiled with clang.

Tested with tinderbox to build with combinations of 10-CURRENT and 9-STABLE with gcc and clang and ARCH i386 and amd64

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2012-03-07 11:20:27 UTC
Responsible Changed
From-To: freebsd-ports-bugs->freebsd-multimedia

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Martin Wilke freebsd_committer freebsd_triage 2012-03-08 13:32:53 UTC
Responsible Changed
From-To: freebsd-multimedia->miwi

I'll take it.
Comment 3 Shane 2012-03-13 08:26:54 UTC
After further testing I found that dirac fails to build on

7.4 7-STABLE
8.2 8.3 8-STABLE

with ARCH=i386 and CC=gcc

The error is undefined reference to `__sync_fetch_and_add_4' in the 
linking step inside work/dirac-1.0.2/encoder

Setting CPUTYPE=i486 or higher gets around the issue but there appears 
to be a change included with 9-STABLE and 10-CURRENT that fixes this issue.

I updated and rebuilt 7/8/9 stable tinderbox jails a few hours ago.
Comment 4 Martin Wilke freebsd_committer freebsd_triage 2012-04-11 03:35:04 UTC
State Changed
From-To: open->closed

i close it here since there is no feedback
Comment 5 Shane 2012-04-16 04:06:59 UTC
On 11/04/2012 12:05, miwi@FreeBSD.org wrote:
> Synopsis: Build multimedia/dirac with clang [patch]
>
> State-Changed-From-To: open->closed
> State-Changed-By: miwi
> State-Changed-When: Wed Apr 11 02:35:04 UTC 2012
> State-Changed-Why:
> i close it here since there is no feedback
>
> http://www.freebsd.org/cgi/query-pr.cgi?pr=165814
>

Why would you leave a port broken just because no-one
commented on the fix?