Bug 175709 - build fix for databases/p5-DBD-Oracle (currently marked BROKEN)
Summary: build fix for databases/p5-DBD-Oracle (currently marked BROKEN)
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: Guido Falsi
URL:
Keywords:
Depends on:
Blocks:
 
Reported: 2013-01-30 19:40 UTC by Tim Z
Modified: 2013-02-08 16:50 UTC (History)
0 users

See Also:


Attachments
file.diff (10.63 KB, patch)
2013-01-30 19:40 UTC, Tim Z
no flags Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Tim Z 2013-01-30 19:40:00 UTC
databases/p5-DBD-Oracle is maked as broken.
It fails to build with the current default version of perl.
The attached patches fix this problem.

Fix: add attached patches to the port
(also bump PORTREVISION and remove the BROKEN= in the Makefile of course)

Patch attached with submission follows:
How-To-Repeat: n/a
Comment 1 Edwin Groothuis freebsd_committer freebsd_triage 2013-01-30 19:40:08 UTC
Responsible Changed
From-To: freebsd-ports-bugs->perl

Over to maintainer (via the GNATS Auto Assign Tool)
Comment 2 Guido Falsi freebsd_committer freebsd_triage 2013-02-08 16:37:02 UTC
Responsible Changed
From-To: perl->madpilot

Take PR, approved by: jadawin (perl@)
Comment 3 dfilter service freebsd_committer freebsd_triage 2013-02-08 16:46:08 UTC
Author: madpilot
Date: Fri Feb  8 16:45:55 2013
New Revision: 311944
URL: http://svnweb.freebsd.org/changeset/ports/311944

Log:
  - Fix build and undeprecate [1] (converted to sed expression)
  
  While I'm here:
  
  - Fix plist with OCI8 option enabled
  - Fix some portlint warnings
  - Properly use optionsng
  - Use EXAMPLES macros for examples in place of DOCS ones.
  
  Similar fix as in PR also submitted via email to me independently
  by Riccardo Torrini <riccardo@torrini.org>
  
  PR:		ports/175709
  Submitted by:	Tim <tez@netbsd.org>
  Approved by:	jadawin (perl@)

Modified:
  head/databases/p5-DBD-Oracle/Makefile   (contents, props changed)
  head/databases/p5-DBD-Oracle/pkg-plist   (contents, props changed)

Modified: head/databases/p5-DBD-Oracle/Makefile
==============================================================================
--- head/databases/p5-DBD-Oracle/Makefile	Fri Feb  8 16:33:51 2013	(r311943)
+++ head/databases/p5-DBD-Oracle/Makefile	Fri Feb  8 16:45:55 2013	(r311944)
@@ -11,15 +11,11 @@ PKGNAMEPREFIX=	p5-
 MAINTAINER=	perl@FreeBSD.org
 COMMENT=	DBI driver for Oracle RDBMS server
 
-OPTIONS_DEFINE=	OCI8 DOCS
-OCI8_DESC=	Use OCI8
-
 BUILD_DEPENDS=	p5-DBI>=0:${PORTSDIR}/databases/p5-DBI
 RUN_DEPENDS:=	${BUILD_DEPENDS}
 
-DEPRECATED=	Broken for more than 6 month
-EXPIRATION_DATE=	2013-03-05
-BROKEN=		Does not build
+OPTIONS_DEFINE=	OCI8 EXAMPLES
+OCI8_DESC=	Use OCI8
 
 PERL_CONFIGURE=	yes
 CONFIGURE_ENV=	ORACLE_HOME="${ORACLE_HOME}"
@@ -35,22 +31,28 @@ WRKSRC=	${WRKDIR}/${PORTNAME}-${PORTVERS
 ORACLE_HOME=	${LOCALBASE}/oracle8-client
 BUILD_DEPENDS+=	${ORACLE_HOME}/lib/libclntst8.a:${PORTSDIR}/databases/oracle8-client
 CONFIGURE_ARGS=	-V 8.1.7
+PLIST_SUB+=	MKTA=''
 .else
 PORTVERSION=	1.14
 ORACLE_HOME=	${LOCALBASE}/oracle7
 BUILD_DEPENDS+=	${ORACLE_HOME}/rdbms/demo/oracle.mk:${PORTSDIR}/databases/oracle7-client
 CONFIGURE_ARGS=	-m ${ORACLE_HOME}/rdbms/demo/oracle.mk
+PLIST_SUB+=	MKTA='@comment '
 .endif
 
 post-patch:
-.ifdef WITH_OCI8
+.if ${PORT_OPTIONS:MOCI8}
 	@${MV} ${WRKSRC}/Oracle.ex/README ${WRKSRC}/Oracle.ex/Readme
 .else
 	@${PERL} -pi -e 's!\(void\*\)(SvOK)!$$1!g' ${WRKSRC}/oci7.c
 .endif
+	@${REINPLACE_CMD} -E -e 's!([, ][&!]?)na\)!\1PL_na)!g' \
+		-e 's!([, =][,&]?)sv_(undef|yes|no)!\1PL_sv_\2!g' \
+		-e 's!dirty!PL_dirty!g' -e 's!dowarn!PL_dowarn!g' \
+		${WRKSRC}/dbdimp.c ${WRKSRC}/Oracle.xs ${WRKSRC}/oci[78].c
 
 post-install:
-.if ${PORT_OPTIONS:MDOCS}
+.if ${PORT_OPTIONS:MEXAMPLES}
 	@${ECHO_MSG} "===>   Installing examples for ${PKGNAME}"
 	@${MKDIR} ${EXAMPLESDIR}
 	@${INSTALL_DATA} ${WRKSRC}/Oracle.ex/Readme 	${EXAMPLESDIR}

Modified: head/databases/p5-DBD-Oracle/pkg-plist
==============================================================================
--- head/databases/p5-DBD-Oracle/pkg-plist	Fri Feb  8 16:33:51 2013	(r311943)
+++ head/databases/p5-DBD-Oracle/pkg-plist	Fri Feb  8 16:45:55 2013	(r311944)
@@ -1,6 +1,7 @@
 bin/ora_explain
 %%SITE_PERL%%/%%PERL_ARCH%%/DBD/Oracle.pm
 %%SITE_PERL%%/%%PERL_ARCH%%/DBD/Oracle/GetInfo.pm
+%%MKTA%%%%SITE_PERL%%/%%PERL_ARCH%%/DBD/mkta.pl
 %%SITE_PERL%%/%%PERL_ARCH%%/Oraperl.pm
 %%SITE_PERL%%/%%PERL_ARCH%%/auto/DBD/Oracle/.packlist
 %%SITE_PERL%%/%%PERL_ARCH%%/auto/DBD/Oracle/Oracle.bs
@@ -10,18 +11,18 @@ bin/ora_explain
 %%SITE_PERL%%/%%PERL_ARCH%%/auto/DBD/Oracle/mk.pm
 %%SITE_PERL%%/%%PERL_ARCH%%/auto/DBD/Oracle/ocitrace.h
 %%SITE_PERL%%/%%PERL_ARCH%%/oraperl.ph
-%%PORTDOCS%%%%EXAMPLESDIR%%/Readme
-%%PORTDOCS%%%%EXAMPLESDIR%%/bind.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/commit.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/curref.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/ex.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/japh
-%%PORTDOCS%%%%EXAMPLESDIR%%/mktable.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/oradump.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/proc.pl
-%%PORTDOCS%%%%EXAMPLESDIR%%/sql
-%%PORTDOCS%%%%EXAMPLESDIR%%/tabinfo.pl
-%%PORTDOCS%%@dirrm %%EXAMPLESDIR%%
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/Readme
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/bind.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/commit.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/curref.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/ex.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/japh
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/mktable.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/oradump.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/proc.pl
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/sql
+%%PORTEXAMPLES%%%%EXAMPLESDIR%%/tabinfo.pl
+%%PORTEXAMPLES%%@dirrm %%EXAMPLESDIR%%
 @dirrm %%SITE_PERL%%/%%PERL_ARCH%%/DBD/Oracle
 @dirrm %%SITE_PERL%%/%%PERL_ARCH%%/auto/DBD/Oracle
 @dirrmtry %%SITE_PERL%%/%%PERL_ARCH%%/DBD
_______________________________________________
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 4 Guido Falsi freebsd_committer freebsd_triage 2013-02-08 16:46:11 UTC
State Changed
From-To: open->closed

Committed with changes. Thanks!