Content-Transfer-Encoding: 7bit
Content-Type: multipart/mixed; boundary="_----------=_148310482530910"
MIME-Version: 1.0
Date: Fri, 30 Dec 2016 13:33:45 +0000
From: "Thomas Klausner" <wiz@netbsd.org>
Subject: CVS commit: pkgsrc/databases
To: pkgsrc-changes@NetBSD.org
Reply-To: wiz@netbsd.org
Message-Id: <20161230133345.5DB29FBA6@cvs.NetBSD.org>
Sender: pkgsrc-changes-owner@NetBSD.org
Precedence: bulk

This is a multi-part message in MIME format.

--_----------=_148310482530910
Content-Disposition: inline
Content-Transfer-Encoding: 8bit
Content-Type: text/plain; charset="US-ASCII"

Module Name:	pkgsrc
Committed By:	wiz
Date:		Fri Dec 30 13:33:45 UTC 2016

Modified Files:
	pkgsrc/databases/sqsh: Makefile options.mk
	pkgsrc/databases/sqsh-motif: Makefile
	pkgsrc/databases/sqsh-x11: Makefile
Added Files:
	pkgsrc/databases/sqsh: Makefile.common
	pkgsrc/databases/sqsh-motif: DESCR
	pkgsrc/databases/sqsh-x11: DESCR

Log Message:
Fix options framework abuse by sqsh*.


To generate a diff of this commit:
cvs rdiff -u -r1.40 -r1.41 pkgsrc/databases/sqsh/Makefile
cvs rdiff -u -r0 -r1.15 pkgsrc/databases/sqsh/Makefile.common
cvs rdiff -u -r1.3 -r1.4 pkgsrc/databases/sqsh/options.mk
cvs rdiff -u -r0 -r1.1 pkgsrc/databases/sqsh-motif/DESCR
cvs rdiff -u -r1.10 -r1.11 pkgsrc/databases/sqsh-motif/Makefile
cvs rdiff -u -r0 -r1.1 pkgsrc/databases/sqsh-x11/DESCR
cvs rdiff -u -r1.10 -r1.11 pkgsrc/databases/sqsh-x11/Makefile

Please note that diffs are not public domain; they are subject to the
copyright notices on the relevant files.


--_----------=_148310482530910
Content-Disposition: inline
Content-Length: 9177
Content-Transfer-Encoding: binary
Content-Type: text/x-diff; charset=us-ascii

Modified files:

Index: pkgsrc/databases/sqsh/Makefile
diff -u pkgsrc/databases/sqsh/Makefile:1.40 pkgsrc/databases/sqsh/Makefile:1.41
--- pkgsrc/databases/sqsh/Makefile:1.40	Mon Sep 19 13:04:20 2016
+++ pkgsrc/databases/sqsh/Makefile	Fri Dec 30 13:33:44 2016
@@ -1,48 +1,6 @@
-# $NetBSD: Makefile,v 1.40 2016/09/19 13:04:20 wiz Exp $
+# $NetBSD: Makefile,v 1.41 2016/12/30 13:33:44 wiz Exp $
 
-# This section contains variables that might be overridden.
-PKGNAME?=	${PKGNAME_BASE}
-COMMENT?=	SQL shell for Sybase and MS-SQL servers
-
-# Normal package Makefile contents follow.
-DISTNAME=	sqsh-2.1-src
-PKGNAME_BASE=	${DISTNAME:S/-src//}
 PKGREVISION=	15
-CATEGORIES=	databases
-MASTER_SITES=	${MASTER_SITE_SOURCEFORGE:=sqsh/}
-
-CONFLICTS+=	sqsh-[0-9]*
-CONFLICTS+=	sqsh-motif-[0-9]*
-CONFLICTS+=	sqsh-x11-[0-9]*
-
-MAINTAINER=	pkgsrc-users@NetBSD.org
-HOMEPAGE=	http://www.sqsh.org/
-
-.include "options.mk"
-
-WRKSRC=			${WRKDIR}/${DISTNAME:S/-src//}
-GNU_CONFIGURE=		yes
-BUILD_TARGET=		build
-
-CONFIGURE_ARGS+=	--with-readline
-CONFIGURE_ENV+=		INCDIRS=${BUILDLINK_PREFIX.editlinereadline:Q}/include
-CONFIGURE_ENV+=		LIBDIRS=${BUILDLINK_PREFIX.editlinereadline:Q}/lib
-
-CHECK_HEADERS_SKIP=	src/sqsh_config.h
-
-CONF_FILES+=		${PREFIX}/share/examples/sqsh/sample.sqshrc	\
-			${PKG_SYSCONFDIR}/sqshrc
-
-INSTALLATION_DIRS=	bin ${PKGMANDIR}/man1
-
-do-install:
-	${INSTALL_PROGRAM} ${WRKSRC}/src/sqsh ${DESTDIR}${PREFIX}/bin
-	${INSTALL_MAN} ${WRKSRC}/doc/sqsh.1 \
-		${DESTDIR}${PREFIX}/${PKGMANDIR}/man1/sqsh.1
-	${INSTALL_DATA_DIR} ${DESTDIR}${PREFIX}/share/examples/sqsh
-	${INSTALL_DATA} ${WRKSRC}/doc/sample.sqshrc \
-		${DESTDIR}${PREFIX}/share/examples/sqsh
 
-.include "../../mk/readline.buildlink3.mk"
-.include "../../mk/dlopen.buildlink3.mk"
+.include "Makefile.common"
 .include "../../mk/bsd.pkg.mk"

Index: pkgsrc/databases/sqsh/options.mk
diff -u pkgsrc/databases/sqsh/options.mk:1.3 pkgsrc/databases/sqsh/options.mk:1.4
--- pkgsrc/databases/sqsh/options.mk:1.3	Sat Jan 19 20:27:30 2013
+++ pkgsrc/databases/sqsh/options.mk	Fri Dec 30 13:33:45 2016
@@ -1,40 +1,11 @@
-# $NetBSD: options.mk,v 1.3 2013/01/19 20:27:30 riastradh Exp $
+# $NetBSD: options.mk,v 1.4 2016/12/30 13:33:45 wiz Exp $
 
 PKG_OPTIONS_VAR=		PKG_OPTIONS.sqsh
-PKG_SUPPORTED_OPTIONS+=		sybase-openclient
-PKG_OPTIONS_OPTIONAL_GROUPS+=	x11
-PKG_OPTIONS_GROUP.x11+=		sqsh-motif sqsh-xaw
-PKG_OPTIONS_LEGACY_VARS+=	USE_SYBASE_OPENCLIENT:sybase-openclient
+PKG_SUPPORTED_OPTIONS=		sybase-openclient
 
 .include "../../mk/bsd.options.mk"
 
 ###
-### Motif GUI support.
-###
-.if !empty(PKG_OPTIONS:Msqsh-motif)
-.  include "../../mk/motif.buildlink3.mk"
-CONFIGURE_ARGS+=	--with-motif=${MOTIFBASE:Q}
-.endif
-
-###
-### Athena GUI support.
-###
-.if !empty(PKG_OPTIONS:Msqsh-xaw)
-.  include "../../mk/xaw.buildlink3.mk"
-CONFIGURE_ARGS+=	--with-x
-.endif
-
-###
-### Common dependencies for X GUI support.
-###
-.if !empty(PKG_OPTIONS:Msqsh-motif) || !empty(PKG_OPTIONS:Msqsh-xaw)
-.  include "../../x11/libX11/buildlink3.mk"
-.  include "../../x11/libXext/buildlink3.mk"
-.  include "../../x11/libXmu/buildlink3.mk"
-.  include "../../x11/libXt/buildlink3.mk"
-.endif
-
-###
 ### Support for the native Sybase Open Client.
 ###
 .if !empty(PKG_OPTIONS:Msybase-openclient)

Index: pkgsrc/databases/sqsh-motif/Makefile
diff -u pkgsrc/databases/sqsh-motif/Makefile:1.10 pkgsrc/databases/sqsh-motif/Makefile:1.11
--- pkgsrc/databases/sqsh-motif/Makefile:1.10	Tue Oct  2 21:25:54 2012
+++ pkgsrc/databases/sqsh-motif/Makefile	Fri Dec 30 13:33:45 2016
@@ -1,14 +1,14 @@
-# $NetBSD: Makefile,v 1.10 2012/10/02 21:25:54 asau Exp $
+# $NetBSD: Makefile,v 1.11 2016/12/30 13:33:45 wiz Exp $
 
-PKGNAME=	${PKGNAME_BASE:S/-/-motif-/}
+PKGNAME=	sqsh-motif-${VERSION}
 COMMENT=	SQL shell for Sybase and MS-SQL servers (Motif GUI)
 
-FILESDIR=	${.CURDIR}/../../databases/sqsh/files
-PATCHDIR=	${.CURDIR}/../../databases/sqsh/patches
-PKGDIR=		${.CURDIR}/../../databases/sqsh
+CONFIGURE_ARGS+=	--with-motif=${MOTIFBASE:Q}
 
-.include "../../mk/bsd.prefs.mk"
-
-PKG_OPTIONS.sqsh+=	sqsh-motif -sqsh-xaw
-
-.include "../../databases/sqsh/Makefile"
+.include "../../x11/libX11/buildlink3.mk"
+.include "../../x11/libXext/buildlink3.mk"
+.include "../../x11/libXmu/buildlink3.mk"
+.include "../../x11/libXt/buildlink3.mk"
+.include "../../mk/motif.buildlink3.mk"
+.include "../../databases/sqsh/Makefile.common"
+.include "../../mk/bsd.pkg.mk"

Index: pkgsrc/databases/sqsh-x11/Makefile
diff -u pkgsrc/databases/sqsh-x11/Makefile:1.10 pkgsrc/databases/sqsh-x11/Makefile:1.11
--- pkgsrc/databases/sqsh-x11/Makefile:1.10	Tue Oct  2 21:25:54 2012
+++ pkgsrc/databases/sqsh-x11/Makefile	Fri Dec 30 13:33:45 2016
@@ -1,14 +1,14 @@
-# $NetBSD: Makefile,v 1.10 2012/10/02 21:25:54 asau Exp $
+# $NetBSD: Makefile,v 1.11 2016/12/30 13:33:45 wiz Exp $
 
-PKGNAME=	${PKGNAME_BASE:S/-/-x11-/}
+PKGNAME=	sqsh-x11-${VERSION}
 COMMENT=	SQL shell for Sybase and MS-SQL servers (Athena GUI)
 
-FILESDIR=	${.CURDIR}/../../databases/sqsh/files
-PATCHDIR=	${.CURDIR}/../../databases/sqsh/patches
-PKGDIR=		${.CURDIR}/../../databases/sqsh
+CONFIGURE_ARGS+=	--with-x
 
-.include "../../mk/bsd.prefs.mk"
-
-PKG_OPTIONS.sqsh+=	-sqsh-motif sqsh-xaw
-
-.include "../../databases/sqsh/Makefile"
+.include "../../mk/xaw.buildlink3.mk"
+.include "../../x11/libX11/buildlink3.mk"
+.include "../../x11/libXext/buildlink3.mk"
+.include "../../x11/libXmu/buildlink3.mk"
+.include "../../x11/libXt/buildlink3.mk"
+.include "../../databases/sqsh/Makefile.common"
+.include "../../mk/bsd.pkg.mk"

Added files:

Index: pkgsrc/databases/sqsh/Makefile.common
diff -u /dev/null pkgsrc/databases/sqsh/Makefile.common:1.15
--- /dev/null	Fri Dec 30 13:33:45 2016
+++ pkgsrc/databases/sqsh/Makefile.common	Fri Dec 30 13:33:44 2016
@@ -0,0 +1,52 @@
+# $NetBSD: Makefile.common,v 1.15 2016/12/30 13:33:44 wiz Exp $
+#
+# used by databases/sqsh/Makefile
+# used by databases/sqsh-motif/Makefile
+# used by databases/sqsh-x11/Makefile
+
+# This section contains variables that might be overridden.
+PKGNAME?=	${PKGNAME_BASE}
+COMMENT?=	SQL shell for Sybase and MS-SQL servers
+
+# Normal package Makefile contents follow.
+DISTNAME=	sqsh-${VERSION}-src
+PKGNAME_BASE=	${DISTNAME:S/-src//}
+CATEGORIES=	databases
+MASTER_SITES=	${MASTER_SITE_SOURCEFORGE:=sqsh/}
+
+VERSION=	2.1
+
+MAINTAINER=	pkgsrc-users@NetBSD.org
+HOMEPAGE=	http://www.sqsh.org/
+LICENSE=	gnu-gpl-v2
+
+PATCHDIR=	${.CURDIR}/../../databases/sqsh/patches
+DISTINFO_FILE=	${.CURDIR}/../../databases/sqsh/distinfo
+PLIST_SRC=	${.CURDIR}/../../databases/sqsh/PLIST
+
+.include "${.CURDIR}/../../databases/sqsh/options.mk"
+
+WRKSRC=			${WRKDIR}/${DISTNAME:S/-src//}
+GNU_CONFIGURE=		yes
+BUILD_TARGET=		build
+
+CONFIGURE_ARGS+=	--with-readline
+CONFIGURE_ENV+=		INCDIRS=${BUILDLINK_PREFIX.editlinereadline:Q}/include
+CONFIGURE_ENV+=		LIBDIRS=${BUILDLINK_PREFIX.editlinereadline:Q}/lib
+
+CHECK_HEADERS_SKIP=	src/sqsh_config.h
+
+CONF_FILES+=		${PREFIX}/share/examples/sqsh/sample.sqshrc	\
+			${PKG_SYSCONFDIR}/sqshrc
+
+INSTALLATION_DIRS=	bin ${PKGMANDIR}/man1 share/examples/sqsh
+
+do-install:
+	${INSTALL_PROGRAM} ${WRKSRC}/src/sqsh ${DESTDIR}${PREFIX}/bin
+	${INSTALL_MAN} ${WRKSRC}/doc/sqsh.1 \
+		${DESTDIR}${PREFIX}/${PKGMANDIR}/man1/sqsh.1
+	${INSTALL_DATA} ${WRKSRC}/doc/sample.sqshrc \
+		${DESTDIR}${PREFIX}/share/examples/sqsh
+
+.include "../../mk/readline.buildlink3.mk"
+.include "../../mk/dlopen.buildlink3.mk"

Index: pkgsrc/databases/sqsh-motif/DESCR
diff -u /dev/null pkgsrc/databases/sqsh-motif/DESCR:1.1
--- /dev/null	Fri Dec 30 13:33:45 2016
+++ pkgsrc/databases/sqsh-motif/DESCR	Fri Dec 30 13:33:45 2016
@@ -0,0 +1,12 @@
+Sqsh (pronounced skwish) is short for SQshelL (pronounced s-q-shell), it
+is intended as a replacement for the venerable 'isql' program supplied
+by Sybase.  It came about due to years of frustration of trying to do
+real work with a program that was never meant to perform real work.
+
+Sqsh is much more than a nice prompt, it is intended to provide much of
+the functionality provided by a good shell, such as variables,
+redirection, pipes, back-grounding, job control, history, command
+completion, and dynamic configuration.  Also, as a by-product of the
+design, it is remarkably easy to extend and add functionality.
+
+This package uses the Motif frontend.

Index: pkgsrc/databases/sqsh-x11/DESCR
diff -u /dev/null pkgsrc/databases/sqsh-x11/DESCR:1.1
--- /dev/null	Fri Dec 30 13:33:45 2016
+++ pkgsrc/databases/sqsh-x11/DESCR	Fri Dec 30 13:33:45 2016
@@ -0,0 +1,12 @@
+Sqsh (pronounced skwish) is short for SQshelL (pronounced s-q-shell), it
+is intended as a replacement for the venerable 'isql' program supplied
+by Sybase.  It came about due to years of frustration of trying to do
+real work with a program that was never meant to perform real work.
+
+Sqsh is much more than a nice prompt, it is intended to provide much of
+the functionality provided by a good shell, such as variables,
+redirection, pipes, back-grounding, job control, history, command
+completion, and dynamic configuration.  Also, as a by-product of the
+design, it is remarkably easy to extend and add functionality.
+
+This package uses the Xaw frontend.


--_----------=_148310482530910--