From debbugs-submit-bounces@debbugs.gnu.org Sun Feb 27 13:32:50 2022 Received: (at 53257) by debbugs.gnu.org; 27 Feb 2022 18:32:50 +0000 Received: from localhost ([127.0.0.1]:59193 helo=debbugs.gnu.org) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nOOLi-0003u5-CK for submit@debbugs.gnu.org; Sun, 27 Feb 2022 13:32:50 -0500 Received: from mail-ed1-f45.google.com ([209.85.208.45]:44630) by debbugs.gnu.org with esmtp (Exim 4.84_2) (envelope-from ) id 1nOOLg-0003tr-IR for 53257@debbugs.gnu.org; Sun, 27 Feb 2022 13:32:49 -0500 Received: by mail-ed1-f45.google.com with SMTP id x5so14516404edd.11 for <53257@debbugs.gnu.org>; Sun, 27 Feb 2022 10:32:48 -0800 (PST) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=gmail.com; s=20210112; h=from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=/T8Jx+6TE87ZLCN509xCG4OplMh/cJoDPPoM4RBOrkI=; b=LYdkrCJOlMwvDMEgg9AGpxcP1IPwcFumdsKay/NK+UrJxQ/+NE6jGZd4+f+NoNxzEU 7/SYT3KirshtWhpoOOXfnd7mfrSLHEIpNNYR5KjMxiAaWG0wKe1REW70KigiZ0/Fbh3d aEJQIFMemGA2vuIC64tgWcp1Juqp9Zxbp/tMoJV3rkj1PB3Ddow1ZPNhvoe0QrbzA1A/ p5fahH/XeewjbUzGK7DYDvemLWlttoMSAALExihPXe0kHZF/0PWaWrF57Usod0mAzkcR p6v7olpOT3X+RnREaPiSmn68J9zsDIjFXrU38KJxDkl7zvwpkOmt3sYUHXG1Mzcf2OW5 nm7g== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20210112; h=x-gm-message-state:from:to:cc:subject:date:message-id:mime-version :content-transfer-encoding; bh=/T8Jx+6TE87ZLCN509xCG4OplMh/cJoDPPoM4RBOrkI=; b=wxiZq52AcrQT2k2WjR16PKOiX3dnZRgQ7arsJmWQmCobtFqdjskWpSNH5DHRxCpUtP o9lmFFq0+fbotdZHhKb08GYvgS6sdbPlzw4goktSbGyDLRrTL2LwebvbNU7wgCEoj782 n/PTVvt6/YHa7AIkNQG+5JCBGke5jLeq/C7BVlmlQc99i+Pt1Hm5sQQDhsbJWAuN7rUq rB8VzsyLNcG/xC1WV4+oIhwrSwYDZ4cfpW5jkq5NxE0NUHf8UkAkxU7Qs7Um/Z1nj2KJ ba/iaKU/4kCnMXYUS1nIdXk83Q2WG2UxrwHPvcIVB9XsjHRq8lm94xjIGVHu+aIsxUwP o6Xw== X-Gm-Message-State: AOAM530fS/TPnAo0aRfqi355AYGZ3FBX7N+fZpgDAXDSqj13ibotsH0s uzie/DaFVO1JAeE73uyrKD+2cFf3WBM= X-Google-Smtp-Source: ABdhPJyMKrcW6UuFppj4KN7QgOQZ2IhCE3WTkojD4D23SWOXQxbN0GTYDX5hvQJH0TGu82ZSsvJvvg== X-Received: by 2002:a05:6402:90b:b0:412:a7cc:f5f9 with SMTP id g11-20020a056402090b00b00412a7ccf5f9mr15947655edz.136.1645986762255; Sun, 27 Feb 2022 10:32:42 -0800 (PST) Received: from localhost.localdomain ([51.154.70.17]) by smtp.gmail.com with ESMTPSA id q10-20020aa7cc0a000000b0040f826f09fdsm5035175edt.81.2022.02.27.10.32.41 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sun, 27 Feb 2022 10:32:41 -0800 (PST) From: Kevin Boulain To: 53257@debbugs.gnu.org Subject: [PATCH] gnu: foot: Wrap the program to expose TERMINFO_DIRS Date: Sun, 27 Feb 2022 19:34:15 +0100 Message-Id: <20220227183415.12487-1-kevinboulain@gmail.com> X-Mailer: git-send-email 2.34.0 MIME-Version: 1.0 Content-Transfer-Encoding: 8bit X-Spam-Score: -0.0 (/) X-Debbugs-Envelope-To: 53257 Cc: Kevin Boulain X-BeenThere: debbugs-submit@debbugs.gnu.org X-Mailman-Version: 2.1.18 Precedence: list List-Id: List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: debbugs-submit-bounces@debbugs.gnu.org Sender: "Debbugs-submit" X-Spam-Score: -1.0 (-) As discussed in https://issues.guix.gnu.org/53257, it appears the preferred way to expose TERMINFO_DIRS to programs running under a terminal is to wrap the terminal with a script that sets TERMINFO_DIRS before exec'ing it. This is less invasive than unconditionally setting TERMINFO_DIRS in the profile (via native-search-paths) as this particular environment variable is only necessary when running the terminal itself. * gnu/packages/terminals.scm (foot): Export TERMINFO_DIRS. Tested: ./pre-inst-env guix install foot cat "$(which foot)" #!/gnu/store/4y5m9lb8k3qkb1y9m02sw9w9a6hacd16-bash-minimal-5.1.8/bin/bash export TERMINFO_DIRS="/gnu/store/6dwhps0cgzk0z7c3q2q596l52ibzdl5c-foot-1.11.0/share/terminfo${TERMINFO_DIRS:+:}$TERMINFO_DIRS" exec -a "$0" "/gnu/store/6dwhps0cgzk0z7c3q2q596l52ibzdl5c-foot-1.11.0/bin/.foot-real" "$@" --- gnu/packages/terminals.scm | 14 +++++++++++++- 1 file changed, 13 insertions(+), 1 deletion(-) diff --git a/gnu/packages/terminals.scm b/gnu/packages/terminals.scm index 06fa341472..65553ac295 100644 --- a/gnu/packages/terminals.scm +++ b/gnu/packages/terminals.scm @@ -852,7 +852,19 @@ (define-public foot ;; also to address a GCC 10 issue when doing PGO builds. #:build-type "release" ;; Enable LTO as recommended by INSTALL.md. - #:configure-flags '("-Db_lto=true"))) + #:configure-flags '("-Db_lto=true") + ;; Ensure the terminfo database is available to programs spawned under + ;; the terminal. + #:phases + (modify-phases %standard-phases + (add-after 'install 'wrap-program + (lambda* (#:key inputs outputs #:allow-other-keys) + (let* ((out (assoc-ref outputs "out"))) + ;; footclient executes programs under the server process, + ;; there is no need to wrap it too. + (wrap-program (string-append out "/bin/foot") + `("TERMINFO_DIRS" ":" prefix + (,(string-append out "/share/terminfo")))))))))) (native-inputs (list ncurses ;for 'tic' pkg-config scdoc wayland-protocols)) -- 2.34.0