From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: from lists.gentoo.org (pigeon.gentoo.org [208.92.234.80]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (2048 bits)) (No client certificate requested) by finch.gentoo.org (Postfix) with ESMTPS id 6A67E15808C for ; Tue, 22 Mar 2022 18:01:27 +0000 (UTC) Received: from pigeon.gentoo.org (localhost [127.0.0.1]) by pigeon.gentoo.org (Postfix) with SMTP id 90087E0827; Tue, 22 Mar 2022 18:01:26 +0000 (UTC) Received: from smtp.gentoo.org (mail.gentoo.org [IPv6:2001:470:ea4a:1:5054:ff:fec7:86e4]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits)) (No client certificate requested) by pigeon.gentoo.org (Postfix) with ESMTPS id 6A55EE0827 for ; Tue, 22 Mar 2022 18:01:26 +0000 (UTC) Received: from oystercatcher.gentoo.org (oystercatcher.gentoo.org [148.251.78.52]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature RSA-PSS (4096 bits) server-digest SHA256) (No client certificate requested) by smtp.gentoo.org (Postfix) with ESMTPS id 9AEC6342F8C for ; Tue, 22 Mar 2022 18:01:25 +0000 (UTC) Received: from localhost.localdomain (localhost [IPv6:::1]) by oystercatcher.gentoo.org (Postfix) with ESMTP id E810D32F for ; Tue, 22 Mar 2022 18:01:23 +0000 (UTC) From: "Sam James" To: gentoo-commits@lists.gentoo.org Content-Transfer-Encoding: 8bit Content-type: text/plain; charset=UTF-8 Reply-To: gentoo-dev@lists.gentoo.org, "Sam James" Message-ID: <1647972071.f94f6f15314c9a4b161efacb57042744346bc234.sam@gentoo> Subject: [gentoo-commits] repo/gentoo:master commit in: net-misc/tigervnc/, net-misc/tigervnc/files/ X-VCS-Repository: repo/gentoo X-VCS-Files: net-misc/tigervnc/files/tigervnc-1.12.0.confd net-misc/tigervnc/files/tigervnc-1.12.0.initd net-misc/tigervnc/tigervnc-1.12.0-r2.ebuild net-misc/tigervnc/tigervnc-1.12.0-r3.ebuild X-VCS-Directories: net-misc/tigervnc/ net-misc/tigervnc/files/ X-VCS-Committer: sam X-VCS-Committer-Name: Sam James X-VCS-Revision: f94f6f15314c9a4b161efacb57042744346bc234 X-VCS-Branch: master Date: Tue, 22 Mar 2022 18:01:23 +0000 (UTC) Precedence: bulk List-Post: List-Help: List-Unsubscribe: List-Subscribe: List-Id: Gentoo Linux mail X-BeenThere: gentoo-commits@lists.gentoo.org X-Auto-Response-Suppress: DR, RN, NRN, OOF, AutoReply X-Archives-Salt: 8cffbd2c-4a9e-47a5-917b-e3a8fb70c28d X-Archives-Hash: 21fa602ee0462f9496695e4744d25850 commit: f94f6f15314c9a4b161efacb57042744346bc234 Author: Viorel Munteanu gmail com> AuthorDate: Tue Mar 22 07:27:03 2022 +0000 Commit: Sam James gentoo org> CommitDate: Tue Mar 22 18:01:11 2022 +0000 URL: https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=f94f6f15 net-misc/tigervnc: Fix the init script Remove $VNC_OPTS from conf.d Make init script POSIX compliant Closes: https://bugs.gentoo.org/835730 Closes: https://bugs.gentoo.org/506522 Signed-off-by: Viorel Munteanu gmail.com> Closes: https://github.com/gentoo/gentoo/pull/24703 Signed-off-by: Sam James gentoo.org> net-misc/tigervnc/files/tigervnc-1.12.0.confd | 4 ++-- net-misc/tigervnc/files/tigervnc-1.12.0.initd | 28 ++++++++++------------ ...-1.12.0-r2.ebuild => tigervnc-1.12.0-r3.ebuild} | 0 3 files changed, 14 insertions(+), 18 deletions(-) diff --git a/net-misc/tigervnc/files/tigervnc-1.12.0.confd b/net-misc/tigervnc/files/tigervnc-1.12.0.confd index a65c991eabcc..d03da01660be 100644 --- a/net-misc/tigervnc/files/tigervnc-1.12.0.confd +++ b/net-misc/tigervnc/files/tigervnc-1.12.0.confd @@ -11,5 +11,5 @@ # TIGERVNC_XSESSION_FILE="/etc/lightdm/Xsession" # TIGERVNC_XSESSION_FILE="/usr/share/slim/Xsession" -# Specify any of vncserver(1) and Xvnc(1) options. See the respective man pages for more information. -# VNC_OPTS="" +# vncsession no longer supports VNC_OPTS +# Use /etc/tigervnc/vncserver-config-defaults or $HOME/.vnc/config instead diff --git a/net-misc/tigervnc/files/tigervnc-1.12.0.initd b/net-misc/tigervnc/files/tigervnc-1.12.0.initd index 59da61eed5f7..c27d9c01bd7e 100644 --- a/net-misc/tigervnc/files/tigervnc-1.12.0.initd +++ b/net-misc/tigervnc/files/tigervnc-1.12.0.initd @@ -2,25 +2,26 @@ # Copyright 1999-2022 Gentoo Authors # Distributed under the terms of the GNU General Public License, v2 or later +# shellcheck shell=sh + depend() { need net } checkconfig() { if [ -n "${DISPLAYS}" ]; then - if [ $1 = "start" ]; then - local user + if [ "$1" = "start" ]; then for user in $DISPLAYS; do if eval [ ! -f "~${user%%:*}/.vnc/passwd" ]; then eerror "There are no passwords defined for user ${user%%:*}." return 1 elif [ -e "/tmp/.X11-unix/X${user##*:}" ]; then eerror "Display :${user##*:} appears to be already in use because of /tmp/.X11-unix/X${user##*:}" - eerror "Remove this file if there is no X server $HOSTNAME:${user##*:}" + eerror "Remove this file if there is no X server for :${user##*:}" return 1 elif [ -e "/tmp/.X${user##*:}-lock" ]; then eerror "Display :${user##*:} appears to be already in use because of /tmp/.X${user##*:}-lock" - eerror "Remove this file if there is no X server $HOSTNAME:${user##*:}" + eerror "Remove this file if there is no X server for :${user##*:}" return 1 elif ! grep -E "^[^#]*:${user##*:}=${user%%:*}" /etc/tigervnc/vncserver.users > /dev/null 2>&1; then eerror "User ${user%%:*} is not defined for display :${user##*:} in /etc/tigervnc/vncserver.users" @@ -30,6 +31,8 @@ checkconfig() { fi return 0 else + # here it is intended for $DISPLAYS to not expand + # shellcheck disable=SC2016 eerror 'Please define $DISPLAYS in /etc/conf.d/tigervnc' return 1 fi @@ -39,15 +42,8 @@ start() { checkconfig start || return 1 ebegin "Starting TigerVNC server" for user in $DISPLAYS; do - usrname=${user%%:*} - usropts=VNC_OPTS_${usrname} - usrdisp=${user##*:} - usrvar=$usropts"["$usrdisp"]" - [ -z ${!usrvar} ] && vnc_opts=$VNC_OPTS || vnc_opts=${!usrvar} - [ -n "${TIGERVNC_XSESSION_FILE}" ] && export TIGERVNC_XSESSION_FILE - - /usr/libexec/vncsession-start :${user##*:} $vnc_opts &>/dev/null + /usr/libexec/vncsession-start ":${user##*:}" >/dev/null 2>&1 done eend $? } @@ -59,10 +55,10 @@ stop() { # vncserver no longer provides a `-kill` option # killing vncsession does not work, we have to kill Xvnc # run pstree to see exactly what's going on - sessionpid=`cat /run/vncsession-\:${user##*:}.pid` - serverpid=`pstree -p $sessionpid | grep Xvnc | sed -e 's/^.*Xvnc(// - s/).*$//'` - kill $serverpid &>/dev/null + sessionpid=$(cat /run/vncsession-:"${user##*:}".pid) + serverpid=$(pstree -p "$sessionpid" | grep Xvnc | sed -e 's/^.*Xvnc(// + s/).*$//') + kill "$serverpid" >/dev/null 2>&1 done # Do not fail if a server is missing /bin/true diff --git a/net-misc/tigervnc/tigervnc-1.12.0-r2.ebuild b/net-misc/tigervnc/tigervnc-1.12.0-r3.ebuild similarity index 100% rename from net-misc/tigervnc/tigervnc-1.12.0-r2.ebuild rename to net-misc/tigervnc/tigervnc-1.12.0-r3.ebuild