Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 27 Jul 2016 07:39:21 +0000 (UTC)
From:      Adam Weinberger <adamw@FreeBSD.org>
To:        ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org
Subject:   svn commit: r419165 - in head/shells/zsh: . files
Message-ID:  <201607270739.u6R7dLTx095946@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: adamw
Date: Wed Jul 27 07:39:21 2016
New Revision: 419165
URL: https://svnweb.freebsd.org/changeset/ports/419165

Log:
  Reorganize, simplify, and improve the Makefile.
  
  A couple options changes:
  The MAILDIR and SECURE_FREE knobs were both removed and switched to
  always-on. Having maildir support does not preclude mbox support, so
  there's no reason not to include it for everybody. The secure-free
  run-time free() safety check is good practice and is a mandatory default
  on a number of other OS's, so now it's enabled for all zsh here as well.
  
  PORTREVISION bump for the above changes.
  
  The rest of the changes are simplifications, and reorganizing things to
  be more legible. An EXTRA_PATCHES that was being applied unconditionally
  was converted to a normal files/patch-, and I added in some comments
  on things I got confused by.
  
  While here, I also rewrote the descr to better describe what Zsh is
  in relation to other shells. I also removed
  
  	"See also zshcompsys(1) manpage. :)"
  
  Not even a smiley face can undo the pain of trying to read zshcompsys(1).

Added:
  head/shells/zsh/files/patch-Completion-Unix-Type-_tar_archive
     - copied, changed from r419157, head/shells/zsh/files/extra-patch-bsdtar
Deleted:
  head/shells/zsh/files/extra-patch-bsdtar
Modified:
  head/shells/zsh/Makefile
  head/shells/zsh/files/patch-Completion-Unix-Command-_sysctl
  head/shells/zsh/pkg-descr

Modified: head/shells/zsh/Makefile
==============================================================================
--- head/shells/zsh/Makefile	Wed Jul 27 07:12:24 2016	(r419164)
+++ head/shells/zsh/Makefile	Wed Jul 27 07:39:21 2016	(r419165)
@@ -3,11 +3,11 @@
 
 PORTNAME=	zsh
 DISTVERSION=	5.2
-PORTREVISION=	2
+PORTREVISION=	3
 CATEGORIES=	shells
-MASTER_SITES=	http://www.zsh.org/pub/:doc \
-		http://www.zsh.org/pub/ \
+MASTER_SITES=	http://www.zsh.org/pub/ \
 		SF \
+		http://www.zsh.org/pub/:doc \
 		SF/${PORTNAME}/${PORTNAME}-doc/${PORTVERSION}:doc
 DISTFILES=	${DISTNAME}${EXTRACT_SUFX}
 
@@ -21,81 +21,62 @@ LICENSE_PERMS=	dist-mirror dist-sell pkg
 
 MAKE_JOBS_UNSAFE=	yes
 
-EXTRA_PATCHES=	${FILESDIR}/extra-patch-bsdtar
-ZSH_VER=	${PORTVERSION}
-PLIST_SUB+=	ZSH_VER="${ZSH_VER}"
-SUB_FILES=	pkg-message
-
 USES=		iconv ncurses shebangfix tar:xz
 
 SHEBANG_LANG=	zsh
-zsh_OLD_CMD=	/bin/zsh
-zsh_CMD=	${PREFIX}/bin/zsh
-SHEBANG_FILES=	Functions/Calendar/calendar_add \
-		Functions/Misc/checkmail \
-		Functions/Misc/run-help \
-		Functions/Misc/sticky-note \
-		Functions/Misc/zcalc \
-		Functions/Misc/zed \
-		Functions/Misc/zkbd
-
-GNU_CONFIGURE=	yes
-TEST_TARGET=	test
+zsh_OLD_CMD=	zsh	# shebang appears as /bin/zsh and just zsh
+SHEBANG_FILES=	Functions/Calendar/* \
+		Functions/Misc/*
 
-CPPFLAGS+=	-I${LOCALBASE}/include -DBOOL_DEFINED
-LDFLAGS+=	-L${LOCALBASE}/lib -Wl,--as-needed
-CONFIGURE_ENV+=	zsh_cv_sys_path_dev_fd=no
 CONFIGURE_ARGS=	--with-tcsetpgrp \
 		--enable-function-subdirs \
+		--enable-maildir-support \
 		--enable-multibyte \
+		--enable-zsh-secure-free \
 		--sysconfdir=${PREFIX}/etc
+CONFIGURE_ENV+=	zsh_cv_sys_path_dev_fd=no
 
-DOCS=		LICENCE META-FAQ README \
-		Etc/BUGS Etc/CONTRIBUTORS Etc/FAQ Etc/completion-style-guide \
-		Doc/zsh*.html Doc/zsh.dvi
-
-PORTDOCS=	*
-PORTEXAMPLES=	zlogin zshenv zshrc
-
-## Vendor completions to be added into the port (none used now)
-#EXTRA_COMPLETION_FILES=	_sockstat
-#EXTRA_COMPLETION_DIR=	Completion/BSD/Command
+GNU_CONFIGURE=	yes
+CPPFLAGS+=	-I${LOCALBASE}/include -DBOOL_DEFINED
+LDFLAGS+=	-L${LOCALBASE}/lib -Wl,--as-needed
+TEST_TARGET=	test
 
-OPTIONS_DEFINE=	DEBUG ETCDIR GDBM MAILDIR MEM PCRE SECURE_FREE STATIC \
+OPTIONS_DEFINE=	DEBUG ETCDIR GDBM MEM PCRE STATIC \
 		DOCS EXAMPLES
-OPTIONS_DEFAULT=SECURE_FREE MAILDIR
 OPTIONS_SUB=	yes
 
-DEBUG_CONFIGURE_ENABLE=	zsh-debug
-
-DOCS_DISTFILES=	${DISTNAME}-doc${EXTRACT_SUFX}:doc
+GDBM_DESC=	Enable GDBM support (GPL)
+ETCDIR_DESC=	System-wide defaults in /etc (instead of $${PREFIX}/etc)
+MEM_DESC=	Enable zsh-mem options
 
-GDBM_DESC=		Enable GDBM support (GPL)
 GDBM_LIB_DEPENDS=	libgdbm.so:databases/gdbm
-GDBM_CONFIGURE_ENABLE=	gdbm
+PCRE_LIB_DEPENDS=	libpcre.so:devel/pcre
 
-ETCDIR_DESC=		System-wide defaults in /etc (instead of $${PREFIX}/etc)
+DEBUG_CONFIGURE_ENABLE=	zsh-debug
+GDBM_CONFIGURE_ENABLE=	gdbm
 ETCDIR_CONFIGURE_OFF=	--enable-etcdir=${PREFIX}/etc
-
-MAILDIR_DESC=			Enable support for Maildirs in MAIL(PATH)
-MAILDIR_CONFIGURE_ENABLE=	maildir-support
-
-MEM_DESC=		Enable zsh-mem options
 MEM_CONFIGURE_ENABLE=	zsh-mem
-
-SECURE_FREE_DESC=		Enable zsh-secure-free
-SECURE_FREE_CONFIGURE_ENABLE=	zsh-secure-free
-
-PCRE_LIB_DEPENDS=	libpcre.so:devel/pcre
 PCRE_CONFIGURE_ENABLE=	pcre
 
-## Some modules can only be built dynamically (such as the regex module).
-## If you enable STATIC, you may get strange errors if scripts/plugins
-## try to use the regex module.
+## Some modules can only be built as a shared library.
+## If you enable STATIC, you may get strange errors if you, a script,
+## or a plugin tries to use the regex module.
 STATIC_LDFLAGS=		-static
 STATIC_CONFIGURE_ON=	--disable-dynamic --with-term-lib="tinfow tinfo"
 STATIC_CONFIGURE_OFF=	--enable-dynamic  --with-term-lib="ncursesw ncurses"
 
+DOCS=		LICENCE META-FAQ README \
+		Etc/BUGS Etc/CONTRIBUTORS Etc/FAQ Etc/completion-style-guide \
+		Doc/zsh*.html Doc/zsh.dvi
+DOCS_DISTFILES=	${DISTNAME}-doc${EXTRACT_SUFX}:doc
+
+PORTDOCS=	*
+PORTEXAMPLES=	zlogin zshenv zshrc
+
+ZSH_VER=	${PORTVERSION}
+PLIST_SUB+=	ZSH_VER="${ZSH_VER}"
+SUB_FILES=	pkg-message
+
 .include <bsd.port.pre.mk>
 
 .if empty(ICONV_LIB)
@@ -103,31 +84,18 @@ CONFIGURE_ENV+=	ac_cv_lib_iconv_libiconv
 .endif
 
 post-patch:
-	@${REINPLACE_CMD} -i '' -e '1s|zsh|${zsh_CMD}|' ${WRKSRC}/Functions/Misc/run-help-ip
-	@${REINPLACE_CMD} -e '/LIBS/s/-lrt//' ${WRKSRC}/configure
-	@${SED} -i "" -e "s,link=dynamic,link=either," \
-		${WRKSRC}/Src/Modules/*.mdd
-	@${SED} -i "" -e "s,/etc/,${LOCALBASE}/etc/," \
+# FreeBSD's clock_gettime(2) is in libc, not librt; see PR 167857
+	@${REINPLACE_CMD} -e '/LIBS/s|-lrt||' ${WRKSRC}/configure
+	@${REINPLACE_CMD} -e "s|/etc/|${LOCALBASE}/etc/|" \
 		${WRKSRC}/Functions/MIME/zsh-mime-setup
 
-# Adding vendor completions into the package
-.ifdef EXTRA_COMPLETION_FILES
-	@${ECHO} "===>  Installing extra completion files"
-.for COMPFILE in ${EXTRA_COMPLETION_FILES}
-	@(if [ -f ${WRKSRC}/${EXTRA_COMPLETION_DIR}/${COMPFILE} ] ; then \
-		${ECHO} "${EXTRA_COMPLETION_DIR}/${COMPFILE} already exists" ; \
-		exit 1 ; \
-	else \
-		${ECHO} ${EXTRA_COMPLETION_DIR}/${COMPFILE} ; \
-		${CP} ${FILESDIR}/${COMPFILE} \
-			${WRKSRC}/${EXTRA_COMPLETION_DIR}/${COMPFILE} ; \
-	fi)
-.endfor
-.endif
-	@${FIND} ${WRKSRC}/Completion -type f -iname '*.orig' -delete
+post-patch-STATIC-on:
+	@${REINPLACE_CMD} -e "s|link=dynamic|link=either|" \
+		${WRKSRC}/Src/Modules/*.mdd
 
 post-build:
 # Fix ".so" macro problem by using "soelim" command.
+# soelim needs the manpages in man1/
 	${LN} -sf ${WRKSRC}/Doc ${WRKSRC}/man1
 	${MV} ${WRKSRC}/Doc/zshall.1 ${WRKSRC}/Doc/zshall.1.source
 	(cd ${WRKSRC} && ${SOELIM} -r ${WRKSRC}/Doc/zshall.1.source > \

Modified: head/shells/zsh/files/patch-Completion-Unix-Command-_sysctl
==============================================================================
--- head/shells/zsh/files/patch-Completion-Unix-Command-_sysctl	Wed Jul 27 07:12:24 2016	(r419164)
+++ head/shells/zsh/files/patch-Completion-Unix-Command-_sysctl	Wed Jul 27 07:39:21 2016	(r419165)
@@ -1,5 +1,5 @@
---- Completion/Unix/Command/_sysctl.orig	2014-10-02 21:28:23.000000000 +0200
-+++ Completion/Unix/Command/_sysctl	2014-10-09 07:57:47.570857223 +0200
+--- Completion/Unix/Command/_sysctl.orig	2015-11-29 21:46:55 UTC
++++ Completion/Unix/Command/_sysctl
 @@ -1,7 +1,7 @@
  #compdef sysctl
  

Copied and modified: head/shells/zsh/files/patch-Completion-Unix-Type-_tar_archive (from r419157, head/shells/zsh/files/extra-patch-bsdtar)
==============================================================================
--- head/shells/zsh/files/extra-patch-bsdtar	Wed Jul 27 02:03:38 2016	(r419157, copy source)
+++ head/shells/zsh/files/patch-Completion-Unix-Type-_tar_archive	Wed Jul 27 07:39:21 2016	(r419165)
@@ -1,6 +1,9 @@
---- ./Completion/Unix/Type/_tar_archive.orig	2009-12-05 18:29:12.000000000 +0100
-+++ ./Completion/Unix/Type/_tar_archive	2011-02-27 23:44:32.000000000 +0100
-@@ -14,17 +14,8 @@
+bsdtar/libarchive knows how to handle compressed tarballs
+natively, without needing fiddly command switches.
+
+--- Completion/Unix/Type/_tar_archive.orig	2014-12-13 23:55:54 UTC
++++ Completion/Unix/Type/_tar_archive
+@@ -14,17 +14,8 @@ local expl
  _description files expl 'archive file'
  
  if [[ "$1" = *[urtx]* ]]; then

Modified: head/shells/zsh/pkg-descr
==============================================================================
--- head/shells/zsh/pkg-descr	Wed Jul 27 07:12:24 2016	(r419164)
+++ head/shells/zsh/pkg-descr	Wed Jul 27 07:39:21 2016	(r419165)
@@ -1,16 +1,15 @@
-Zsh is a UNIX command interpreter (shell) which of the standard shells
-most resembles the Korn shell (ksh), although it is not completely
-compatible.  It includes enhancements of many types, notably in the
-command-line editor, options for customising its behaviour, filename
-globbing, features to make C-shell (csh) users feel more at home and
-extra features drawn from tcsh (another `custom' shell).
+Zsh is the Swiss Army knife of shells. It combines the most popular
+features of every other shell, and then lets you customize every
+inch of it. Users of bourne-style and C-style shells will feel at
+home in it.
 
-If you want to use zsh completion system, you should type the following
-commands:
+Zsh does intelligent completion, spell-checking, has a rich syntax
+for precise globbing, and is fully extensible through plugin
+systems.
+
+To fire up the zsh completion system, type the following commands:
 
 	$ autoload -U compinstall
 	$ compinstall
 
-See also zshcompsys(1) manpage. :)
-
 WWW: http://www.zsh.org/



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201607270739.u6R7dLTx095946>