Bug 176766 - [patch] audio/dvda-author fix build with clang
Summary: [patch] audio/dvda-author fix build with clang
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: Pawel Pekala
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-03-08 20:00 UTC by Pawel Pekala
Modified: 2013-03-13 15:40 UTC (History)
0 users

See Also:


Attachments
file.diff (815 bytes, patch)
2013-03-08 20:00 UTC, Pawel Pekala
no flags Details | Diff
dvda-author-v2.diff (729 bytes, patch)
2013-03-11 19:17 UTC, Pawel Pekala
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Pawel Pekala freebsd_committer freebsd_triage 2013-03-08 20:00:00 UTC
Simple clang build fix

Fix: Patch attached with submission follows:
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-03-08 20:00:13 UTC
Responsible Changed
From-To: freebsd-ports-bugs->miwi

miwi@ wants this submitter's PRs (via the GNATS Auto Assign Tool)
Comment 2 Edwin Groothuis freebsd_committer freebsd_triage 2013-03-08 20:00:14 UTC
Maintainer of audio/dvda-author,

Please note that PR ports/176766 has just been submitted.

If it contains a patch for an upgrade, an enhancement or a bug fix
you agree on, reply to this email stating that you approve the patch
and a committer will take care of it.

The full text of the PR can be found at:
    http://www.freebsd.org/cgi/query-pr.cgi?pr=ports/176766

-- 
Edwin Groothuis via the GNATS Auto Assign Tool
edwin@FreeBSD.org
Comment 3 Edwin Groothuis freebsd_committer freebsd_triage 2013-03-08 20:00:15 UTC
State Changed
From-To: open->feedback

Awaiting maintainers feedback (via the GNATS Auto Assign Tool)
Comment 4 martin.dieringer 2013-03-09 10:42:19 UTC
results in "cc -std=gnu99 -std=gnu89 ..."
but seems no problem
Comment 5 Martin Wilke freebsd_committer freebsd_triage 2013-03-10 12:02:58 UTC
Responsible Changed
From-To: miwi->pawel

submitter is committer.
Comment 6 Pawel Pekala 2013-03-11 19:17:46 UTC
>results in "cc -std=gnu99 -std=gnu89 ..."
>but seems no problem


Hmm right, didn't look at buildlogs close enough. Here's improved
version which cleanups build flags for entire build and uses
USE_CSTD macro as it supposed to in the first place. While here I also
allowed myself to convert option check to new options framework.

Do you approve?

-- 
pozdrawiam / with regards
Pawe³ Pêkala
Comment 7 martin.dieringer 2013-03-12 20:26:13 UTC
now it's "cc -std=gnu89 -O2 -pipe -fno-strict-aliasing -std=gnu89 ..."
but I wouldn't care ...
Comment 8 dfilter service freebsd_committer freebsd_triage 2013-03-13 15:31:11 UTC
Author: pawel
Date: Wed Mar 13 15:31:03 2013
New Revision: 314091
URL: http://svnweb.freebsd.org/changeset/ports/314091

Log:
  - Respect CFLAGS
  - Fix build with clang
  - Convert to OptionsNG
  - Trim Makefile header
  
  PR:		ports/176766
  Submitted by:	myself
  Approved by:	Martin Dieringer <martin.dieringer@gmx.de> (maintainer)

Modified:
  head/audio/dvda-author/Makefile   (contents, props changed)

Modified: head/audio/dvda-author/Makefile
==============================================================================
--- head/audio/dvda-author/Makefile	Wed Mar 13 15:29:55 2013	(r314090)
+++ head/audio/dvda-author/Makefile	Wed Mar 13 15:31:03 2013	(r314091)
@@ -1,9 +1,5 @@
-# New ports collection makefile for:   dvda-author
-# Date created:        9 October 2007
-# Whom:                dieringe@zedat.fu-berlin.de
-#
+# Created by: dieringe@zedat.fu-berlin.de
 # $FreeBSD$
-#
 
 PORTNAME=	dvda-author
 PORTVERSION=	09.05
@@ -34,8 +30,19 @@ GNU_CONFIGURE=	yes
 CONFIGURE_ARGS=	--with-flac=${LOCALBASE}/lib/libFLAC.so \
 	--with-ogg=${LOCALBASE}/lib/libogg.so \
 	--enable-glibc
-.if defined(NOPORTDOCS)
+USE_CSTD=	gnu89
+
+.include <bsd.port.options.mk>
+
+.if ! ${PORT_OPTIONS:MDOCS}
 CONFIGURE_ARGS+=	--docdir=${WRKDIR}/doc
 .endif
 
+post-patch:
+	@${REINPLACE_CMD} 's|-std=c99||g' \
+		${WRKSRC}/configure \
+		${WRKSRC}/libats2wav/Makefile.in
+	@${REINPLACE_CMD} 's|-std=c99 -O3||' \
+		${WRKSRC}/fixwav/src/Makefile.in
+
 .include <bsd.port.mk>
_______________________________________________
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 9 Pawel Pekala freebsd_committer freebsd_triage 2013-03-13 15:31:33 UTC
State Changed
From-To: feedback->closed

Committed. Thanks!