git: b20b92686b50 - main - misc/mc: the port had been improved (+)

[Alexey Dokuchaev]

The branch main has been updated by danfe:

URL: https://cgit.FreeBSD.org/ports/commit/?id=b20b92686b504c322dd809c3107acb4edda0eb08

commit b20b92686b504c322dd809c3107acb4edda0eb08
Author:     Alexey Dokuchaev <danfe at FreeBSD.org>
AuthorDate: 2021-05-27 08:42:00 +0000
Commit:     Alexey Dokuchaev <danfe at FreeBSD.org>
CommitDate: 2021-05-27 08:42:43 +0000

    misc/mc: the port had been improved (+)
    
    - GC outdated CONFLICTS (misc/mc-light had been removed on 2019-10-16)
    - Add missing SCREEN_DESC (screen library) option description
    - Offer new option, EXTATTR (ext2fs extended attributes support) and
      ensure, in somewhat ugly way, that dependency on `sysutils/e2fsprogs'
      won't be silently picked up if this option is disabled
    - Tell users that /bin/sh, our base system sh(1), is not supported as
      concurrent subshell because it does not allow "precmd" hooks [1]
    - Fix entering RAR archives after `archivers/unrar` had been updated
      to version 6.x [2]
    - Use helper target and simplify installation of documentation files
    
    PR:     208391 [1], 251404 [2]
---
 misc/mc/Makefile             | 36 +++++++++++++++++++-----------------
 misc/mc/files/pkg-message.in | 17 +++++++++++++++++
 2 files changed, 36 insertions(+), 17 deletions(-)

diff --git a/misc/mc/Makefile b/misc/mc/Makefile
index 4ffb4a71f57e..781b68b1149f 100644
--- a/misc/mc/Makefile
+++ b/misc/mc/Makefile
@@ -2,6 +2,7 @@
 
 PORTNAME=	mc
 PORTVERSION=	4.8.26
+PORTREVISION=	1
 CATEGORIES=	misc shells
 MASTER_SITES=	http://ftp.midnight-commander.org/ \
 		https://ftp.osuosl.org/pub/midnightcommander/
@@ -12,8 +13,6 @@ COMMENT=	Midnight Commander, a free Norton Commander Clone
 LICENSE=	GPLv3+
 LICENSE_FILE=	${WRKSRC}/COPYING
 
-CONFLICTS=	mc-light-4*
-
 USES=		gmake gnome libtool perl5 pkgconfig python shebangfix tar:xz
 SHEBANG_FILES=	${WRKSRC}/src/vfs/extfs/helpers/s3+.in \
 		${WRKSRC}/src/vfs/extfs/helpers/uc1541
@@ -22,18 +21,24 @@ USE_GNOME=	glib20
 GNU_CONFIGURE=	yes
 CONFIGURE_ENV=	ZIP=${ZIP_CMD} UNZIP=${UNZIP_CMD}
 
-OPTIONS_DEFINE=	DOCS EDITOR ICONV NLS SFTP SMB SUBSHELL X11
-OPTIONS_DEFAULT=EDITOR ICONV SFTP SLANG SMB SUBSHELL X11
+PORTDOCS=	AUTHORS FAQ HACKING MAINTAINERS NEWS README *.txt
+
+OPTIONS_DEFINE=	DOCS EDITOR EXTATTR ICONV NLS SFTP SMB SUBSHELL X11
+OPTIONS_DEFAULT=	EDITOR EXTATTR ICONV SFTP SLANG SMB SUBSHELL X11
 OPTIONS_SINGLE=	SCREEN
 OPTIONS_SINGLE_SCREEN=	SLANG NCURSES
 OPTIONS_SUB=	yes
 
 EDITOR_DESC=	Build with internal editor
+EXTATTR_DESC=	Ext2fs extended attributes support
+SCREEN_DESC=	Screen library
 SFTP_DESC=	Support for SFTP (via libssh)
 SUBSHELL_DESC=	Build with subshell support
 
 EDITOR_CONFIGURE_WITH=	internal-edit
 
+EXTATTR_LIB_DEPENDS=	libe2p.so:sysutils/e2fsprogs
+
 ICONV_USES=		iconv
 ICONV_CONFIGURE_ENABLE=	charset
 
@@ -56,6 +61,7 @@ SMB_CONFIGURE_WITH=	smb-configdir=${LOCALBASE}/etc \
 			smb-codepagedir=${LOCALBASE}/etc/codepages
 
 SUBSHELL_CONFIGURE_WITH=subshell
+SUBSHELL_SUB_FILES=	pkg-message
 
 X11_CONFIGURE_ENABLE=	x
 X11_USES=		xorg
@@ -63,25 +69,21 @@ X11_USE=		xorg=x11,xext
 X11_VARS=		CONFLICTS_INSTALL+="mc-nox11-[0-9]*"
 X11_VARS_OFF=		CONFLICTS_INSTALL+="mc-[0-9]*"
 
-PORTDOCS=	*
-
-DOCSRCDIR1=	${WRKSRC}
-DOC_FILES1=	AUTHORS NEWS README
-
-DOCSRCDIR2=	${WRKSRC}/doc
-DOCSDIR2=	${DOCSDIR}/doc
-DOC_FILES2=	FAQ HACKING MAINTAINERS *.txt
-
 post-patch:
 	@${REINPLACE_CMD} -e 's|missing-parameter-type|unused-variable|; \
 		s|but-set-variable|variable|' \
 		${WRKSRC}/configure
+	@${REINPLACE_CMD} -e 's|x$$UNRAR_VERSION = x|$$UNRAR_VERSION -ge |' \
+		${WRKSRC}/src/vfs/extfs/helpers/urar.in
+
+post-patch-EXTATTR-off:
+	@${REINPLACE_CMD} -e '/found_e2p=/s|yes|disabled|; /e2p library/ \
+		{ s| or|,|; s|)|&, or disabled|; }' ${WRKSRC}/configure
 
-post-install:
+post-install-DOCS-on:
 	@${MKDIR} ${STAGEDIR}${DOCSDIR}
-	${INSTALL_DATA} ${DOC_FILES1:S|^|${DOCSRCDIR1}/|} ${STAGEDIR}${DOCSDIR}
-	@${MKDIR} ${STAGEDIR}${DOCSDIR2}
-	${INSTALL_DATA} ${DOC_FILES2:S|^|${DOCSRCDIR2}/|} ${STAGEDIR}${DOCSDIR2}
+	${LN} -sf ../${PORTDOCS:[1]} ${WRKSRC}/doc
+	${INSTALL_DATA} ${PORTDOCS:S|^|${WRKSRC}/doc/|} ${STAGEDIR}${DOCSDIR}
 
 post-install-X11-on:
 	${INSTALL_DATA} ${FILESDIR}/${PORTNAME}.png \
diff --git a/misc/mc/files/pkg-message.in b/misc/mc/files/pkg-message.in
new file mode 100644
index 000000000000..ee7fc70bc1e2
--- /dev/null
+++ b/misc/mc/files/pkg-message.in
@@ -0,0 +1,17 @@
+[
+{ type: install
+  message: <<EOM
+Midnight Commander was built with subshell support, which works with most
+popular shells, e.g. bash(1), dash(1), tcsh(1), zsh(1), but not sh(1) due
+to its lack of "precmd" or equivalent hooks which mc(1) needs to read the
+subshell's current directory.
+
+  $ env SHELL=/bin/sh mc
+  common.c: unimplemented subshell type 1
+  read (subshell_pty...): No such file or directory (2)
+
+Please either use more advanced interactive shell, or start mc(1) with -u
+(--nosubshell) switch if you're confined to /bin/sh for some reason.
+EOM
+}
+]