Bug 193218

Summary: Allow to build devel/google-sparsehash with clang compiler
Product: Ports & Packages Reporter: Dmitry Sivachenko <demon>
Component: Individual Port(s)Assignee: Vasil Dimov <vd>
Status: Closed FIXED    
Severity: Affects Many People CC: sbruno, vd
Priority: ---    
Version: Latest   
Hardware: Any   
OS: Any   
Description Flags
fix build with clang none

Description Dmitry Sivachenko freebsd_committer 2014-09-01 12:50:20 UTC
Allow to build devel/google-sparsehash port with clang compiler.
Comment 1 Dmitry Sivachenko freebsd_committer 2014-09-01 12:50:54 UTC
Created attachment 146628 [details]
fix build with clang
Comment 2 Sean Bruno freebsd_committer 2014-09-01 17:08:54 UTC
Comment on attachment 146628 [details]
fix build with clang

Can you schedule a reports build or do you want me to do it?
Comment 3 Dmitry Sivachenko freebsd_committer 2014-09-01 17:53:29 UTC
(In reply to Sean Bruno from comment #2)
> Comment on attachment 146628 [details]
> fix build with clang
> Can you schedule a reports build or do you want me to do it?

What is reports build?
Comment 4 Sean Bruno freebsd_committer 2014-09-01 18:13:11 UTC
sorry, I meant "redport"


Its a free testing service for freebsd ports.  I'm running a test of this patch now.
Comment 5 Sean Bruno freebsd_committer 2014-09-01 19:44:59 UTC

This builds successfully and eliminates a need for gcc.  Let's get this committed.
Comment 6 Sean Bruno freebsd_committer 2014-09-03 17:36:17 UTC
Created review:

Passes redports building:
Comment 7 commit-hook freebsd_committer 2014-09-15 15:35:23 UTC
A commit references this bug:

Author: sbruno
Date: Mon Sep 15 15:35:19 UTC 2014
New revision: 368270
URL: http://svnweb.freebsd.org/changeset/ports/368270

  Fixup templates and remove gcc dependency in this port.

  Side effect, allows compilation on armv6.

  PR:		193218
  Submitted by:	demon@FreeBSD.org
  Reviewed by:	bdrewery (mentor)
  Approved by:	maintainer timeout

Comment 8 Sean Bruno freebsd_committer 2014-09-15 15:36:13 UTC
Thank you for the patch, this has been accepted and committed to ports!