Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 22 Jul 2014 23:15:21 +0200
From:      Antoine Brodin <antoine@FreeBSD.org>
To:        Rene Ladan <rene@freebsd.org>
Cc:        "svn-ports-head@freebsd.org" <svn-ports-head@freebsd.org>, "svn-ports-all@freebsd.org" <svn-ports-all@freebsd.org>, "ports-committers@freebsd.org" <ports-committers@freebsd.org>
Subject:   Re: svn commit: r362582 - head/databases/linux-oracle-instantclient-sqlplus
Message-ID:  <CAALwa8kxKa9%2Bun6DBK8e3PUerX%2BrthAKnONcJy=5oa-Ptuw5FQ@mail.gmail.com>
In-Reply-To: <201407222109.s6ML9ptP003749@svn.freebsd.org>
References:  <201407222109.s6ML9ptP003749@svn.freebsd.org>

next in thread | previous in thread | raw e-mail | index | archive | help
On Tue, Jul 22, 2014 at 11:09 PM, Rene Ladan <rene@freebsd.org> wrote:
> Author: rene
> Date: Tue Jul 22 21:09:51 2014
> New Revision: 362582
> URL: http://svnweb.freebsd.org/changeset/ports/362582
> QAT: https://qat.redports.org/buildarchive/r362582/
>
> Log:
>   - Stage support
>   - Enable usage on amd64
>   - Clean up Makefile (especially IGNORE message)
>
>   PR:           191993
>   Submitted by: papowell@astart.com
>
> Modified:
>   head/databases/linux-oracle-instantclient-sqlplus/Makefile
>   head/databases/linux-oracle-instantclient-sqlplus/pkg-plist
>
> Modified: head/databases/linux-oracle-instantclient-sqlplus/Makefile
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
> --- head/databases/linux-oracle-instantclient-sqlplus/Makefile  Tue Jul 2=
2 21:08:01 2014        (r362581)
> +++ head/databases/linux-oracle-instantclient-sqlplus/Makefile  Tue Jul 2=
2 21:09:51 2014        (r362582)
> @@ -3,7 +3,7 @@
>
>  PORTNAME=3D              instantclient
>  PORTVERSION=3D           ${ORACLEVERSION}.${DISTDATE}
> -PORTREVISION=3D          1
> +PORTREVISION=3D          2
>  CATEGORIES=3D            databases
>  MASTER_SITES=3D          #http://www.oracle.com/technology/software/tech=
/oci/instantclient/
>  PKGNAMEPREFIX=3D         linux-oracle-
> @@ -12,17 +12,16 @@ DISTNAME=3D           ${PORTNAME}${PKGNAMESUFFIX}-$
>  DIST_SUBDIR=3D           oracle
>
>  MAINTAINER=3D            ports@FreeBSD.org
> -COMMENT=3D               Oracle 10 InstantClient SQLPlus. Works with Ora=
cle RDBMS 8.1.2+
> +COMMENT=3D               Oracle 10 32 Bit Linux InstantClient SQLPlus fo=
r RDBMS 8.1.2+
>
> -RUN_DEPENDS=3D           ${LINUXBASE}/usr/lib/oracle/${ORACLEVERSION}/cl=
ient/lib/libocci.so.10.1:${PORTSDIR}/databases/linux-oracle-instantclient-b=
asic
> -
> -ONLY_FOR_ARCHS=3D                i386    # amd64(not yet on own)
> +ONLY_FOR_ARCHS=3D                i386 amd64
>  NO_BUILD=3D              yes
> -DISTINFO_FILE=3D         ${MASTERDIR}/distinfo.${ARCH}
> +DISTINFO_FILE=3D         ${MASTERDIR}/distinfo.i386
>
>  RESTRICTED=3D            Packaging prohibited by Oracle license
>
>  USES=3D                  zip
> +USE_LINUX=3D             yes
>  USE_LINUX_PREFIX=3D      yes
>
>  ORACLEVERSION=3D         10.2.0.3
> @@ -35,50 +34,35 @@ LIBS_LIST=3D          glogin.sql      \
>
>  WRKSRC=3D                        ${WRKDIR}/instantclient_10_2
>
> -INSTBINSDIR=3D           ${PREFIX}/usr/lib/oracle/${ORACLEVERSION}/clien=
t/bin
> -INSTLIBSDIR=3D           ${PREFIX}/usr/lib/oracle/${ORACLEVERSION}/clien=
t/lib
> +INSTBINSDIR=3D           ${STAGEDIR}${PREFIX}/usr/bin
> +INSTLIBSDIR=3D           ${STAGEDIR}${PREFIX}/usr/lib/oracle/${ORACLEVER=
SION}/client/lib
> +
> +USE_LDCONFIG=3D${INSTLIBSDIR}

Looks wrong again,  you should have a QA error looking like
Error: 'libdata/ldconfig/linux-oracle-instantclient' is referring to
/path/to/work/stage

>
>  PLIST_SUB=3D             ORACLEVERSION=3D${ORACLEVERSION}
>
> -NO_STAGE=3D      yes
>  .include <bsd.port.pre.mk>
>
> -.if            ${ARCH} =3D=3D "i386"
> +#.if ${ARCH} =3D=3D "i386"
>  LINUXARCH=3D             linux32
>  DISTDATE=3D              20061115
> -.elif          ${ARCH} =3D=3D "amd64"
> -LINUXARCH=3D             linux-x86-64
> -DISTDATE=3D              20070103
> -.endif
> +#.elif ${ARCH} =3D=3D "amd64"
> +#LINUXARCH=3D            linux-x86-64
> +#DISTDATE=3D             20070103
> +#.endif
>
>  .if !exists(${DISTDIR}/${DIST_SUBDIR}/${DISTNAME}${EXTRACT_SUFX})
> -ECHO_MSG=3D              ${PRINTF} %b
> -
> -IGNORE+=3D               \n\n-=3D< MARKED AS IGNORED: >=3D-\n
> -IGNORE+=3D               \n
> -IGNORE+=3D               Due to Oracle license restrictions, you must fe=
tch the source \n
> -IGNORE+=3D               distribution manually.  Please access  \n
> -IGNORE+=3D               http://www.oracle.com/technology/software/tech/=
oci/instantclient/index.html \n
> -IGNORE+=3D               with a web browser and follow the link for the =
\n
> -IGNORE+=3D               "${DISTFILES}".  \n
> -IGNORE+=3D               You will be required to register and log in, \n
> -IGNORE+=3D               but you can create an account on that page.  \n
> -IGNORE+=3D               After registration and accepting the Oracle Lic=
ense, \n
> -IGNORE+=3D               download the distribution file, \n ${DISTFILES}=
, \n
> -IGNORE+=3D               into ${DISTDIR}/${DIST_SUBDIR}/ \n
> -IGNORE+=3D               and then restart this installation\n\n
> +IGNORE+=3D       due to Oracle license restrictions, you must fetch the =
source distribution manually from http://www.oracle.com/technology/software=
/tech/oci/instantclient/index.html , follow the link for ${DISTFILES} and p=
lace them in ${DISTDIR}/${DIST_SUBDIR}/
>  .endif
>
>  do-install:
> -                       ${MKDIR} -m 0755 ${INSTBINSDIR}
> -                       ${MKDIR} -m 0755 ${INSTLIBSDIR}
> -                       ${LN} -s ../lib/oracle/${ORACLEVERSION}/client/bi=
n/sqlplus ${PREFIX}/usr/bin/sqlplus
> +       @${MKDIR} -m 0755 ${INSTBINSDIR}
> +       @${MKDIR} -m 0755 ${INSTLIBSDIR}
>  .for i in ${BINS_LIST}
> -                       ${INSTALL_SCRIPT} ${WRKSRC}/${i} ${INSTBINSDIR}/$=
{i}
> +       ${INSTALL_PROGRAM} ${WRKSRC}/${i} ${INSTBINSDIR}/${i}
>  .endfor
>  .for i in ${LIBS_LIST}
> -                       ${INSTALL_DATA}   ${WRKSRC}/${i} ${INSTLIBSDIR}/$=
{i}
> +       ${INSTALL_DATA} ${WRKSRC}/${i} ${INSTLIBSDIR}/${i}
>  .endfor
> -                       ${PREFIX}/sbin/ldconfig
>
>  .include <bsd.port.post.mk>
>
> Modified: head/databases/linux-oracle-instantclient-sqlplus/pkg-plist
> =3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=3D=
=3D=3D=3D=3D
> --- head/databases/linux-oracle-instantclient-sqlplus/pkg-plist Tue Jul 2=
2 21:08:01 2014        (r362581)
> +++ head/databases/linux-oracle-instantclient-sqlplus/pkg-plist Tue Jul 2=
2 21:09:51 2014        (r362582)
> @@ -1,8 +1,8 @@
>  usr/bin/sqlplus
> -usr/lib/oracle/%%ORACLEVERSION%%/client/bin/sqlplus
>  usr/lib/oracle/%%ORACLEVERSION%%/client/lib/glogin.sql
>  usr/lib/oracle/%%ORACLEVERSION%%/client/lib/libsqlplus.so
>  usr/lib/oracle/%%ORACLEVERSION%%/client/lib/libsqlplusic.so
> -@dirrm usr/lib/oracle/%%ORACLEVERSION%%/client/bin
> -@dirrmtry usr/bin
> -@exec %D/sbin/ldconfig
> +@dirrmtry usr/lib/oracle/%%ORACLEVERSION%%/client/lib
> +@dirrmtry usr/lib/oracle/%%ORACLEVERSION%%/client
> +@dirrmtry usr/lib/oracle/%%ORACLEVERSION%%
> +@dirrmtry usr/lib/oracle
>



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?CAALwa8kxKa9%2Bun6DBK8e3PUerX%2BrthAKnONcJy=5oa-Ptuw5FQ>