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 | ||||||
Attachments: |
|
Description
Dmitry Sivachenko
![]() Created attachment 146628 [details]
fix build with clang
Comment on attachment 146628 [details]
fix build with clang
Can you schedule a reports build or do you want me to do it?
(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? sorry, I meant "redport" http://redports.org Its a free testing service for freebsd ports. I'm running a test of this patch now. https://redports.org/buildarchive/20140901191626-49361/ This builds successfully and eliminates a need for gcc. Let's get this committed. Created review: https://reviews.freebsd.org/D718 Passes redports building: https://redports.org/buildarchive/20140901191626-49361/ 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 Log: 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 Changes: head/devel/google-sparsehash/Makefile head/devel/google-sparsehash/files/ head/devel/google-sparsehash/files/patch-src__sparsehash__internal__sparsehashtable.h Thank you for the patch, this has been accepted and committed to ports! |