* bug#52178: 29.0.50; process-environment not used in async-shell-command
@ 2021-11-29 12:33 Damien Merenne
2021-11-29 14:38 ` Lars Ingebrigtsen
0 siblings, 1 reply; 7+ messages in thread
From: Damien Merenne @ 2021-11-29 12:33 UTC (permalink / raw)
To: 52178
When running an async process, the value of `process-environment' is
lost, in 'emacs -Q' scratch buffer:
(let ((process-environment "EDITOR=test"))
(shell-command "env &"))
and the output is my normal environment, not the limited one. When
removing the &, it works okay. I managed to pinpoint the problem to the
call to `with-current-buffer' in simple.el:4229. As process-environment
is buffer local, it is not passed in the process buffer. This patch
fixes it but I have no idea if this is the correct way to fix it:
@@ -4194,7 +4194,8 @@ shell-command
(or output-buffer shell-command-buffer-name-async)))
(bname (buffer-name buffer))
(proc (get-buffer-process buffer))
- (directory default-directory))
+ (directory default-directory)
+ (process-env process-environment))
;; Remove the ampersand.
(setq command (substring command 0 (match-beginning 0)))
;; Ask the user what to do with already running process.
@@ -4237,7 +4238,7 @@ shell-command
(format "COLUMNS=%d"
async-shell-command-width)))
(comint-term-environment)
- process-environment)))
+ process-env)))
(setq proc
(start-process-shell-command "Shell" buffer command)))
(setq mode-line-process '(":%s"))
In GNU Emacs 29.0.50 (build 5, x86_64-pc-linux-gnu, GTK+ Version 3.24.25, cairo version 1.16.0)
of 2021-11-29 built on workstation.lan
Repository revision: f5498a608fe85b66a4068a5ccf88c9b3d3b88f98
Repository branch: master
Windowing system distributor 'The X.Org Foundation', version 11.0.12101001
System Description: Ubuntu 21.04
Configured using:
'configure --with-native-compilation --with-mailutils --with-gnutls
--with-modules --with-json --without-gpm --with-file-notification=yes
--with-xwidgets --without-selinux --with-libsystemd --with-jpeg
--with-tiff --with-gif --with-png --with-rsvg --with-imagemagick'
Configured features:
ACL CAIRO DBUS FREETYPE GIF GLIB GMP GNUTLS GSETTINGS HARFBUZZ
IMAGEMAGICK JPEG JSON LCMS2 LIBSYSTEMD LIBXML2 MODULES NATIVE_COMP
NOTIFY INOTIFY PDUMPER PNG RSVG SECCOMP SOUND THREADS TIFF
TOOLKIT_SCROLL_BARS X11 XDBE XIM XPM XWIDGETS GTK3 ZLIB
Important settings:
value of $LC_MONETARY: fr_BE.UTF-8
value of $LC_NUMERIC: fr_BE.UTF-8
value of $LC_TIME: fr_BE.UTF-8
value of $LANG: en_US.UTF-8
value of $XMODIFIERS: @im=ibus
locale-coding-system: utf-8-unix
Major mode: C/*lw
Minor modes in effect:
bloom-global-minor-mode: t
flyspell-mode: t
bug-reference-prog-mode: t
magit-wip-after-apply-mode: t
global-git-commit-mode: t
magit-auto-revert-mode: t
tree-sitter-hl-mode: t
global-tree-sitter-mode: t
tree-sitter-mode: t
dtrt-indent-global-mode: t
dtrt-indent-mode: t
pdf-occur-global-minor-mode: t
yas-global-mode: t
yas-minor-mode: t
subword-mode: t
smartparens-global-mode: t
smartparens-mode: t
auto-revert-mode: t
marginalia-mode: t
selectrum-mode: t
corfu-global-mode: t
corfu-mode: t
global-ligature-mode: t
ligature-mode: t
shell-dirtrack-mode: t
envrc-global-mode: t
envrc-mode: t
bepo-global-mode: t
which-key-mode: t
savehist-mode: t
global-so-long-mode: t
override-global-mode: t
straight-use-package-mode: t
straight-package-neutering-mode: t
tooltip-mode: t
global-eldoc-mode: t
show-paren-mode: t
electric-indent-mode: t
mouse-wheel-mode: t
menu-bar-mode: t
file-name-shadow-mode: t
global-font-lock-mode: t
font-lock-mode: t
blink-cursor-mode: t
auto-composition-mode: t
auto-encryption-mode: t
auto-compression-mode: t
line-number-mode: t
auto-fill-function: yas--auto-fill
indent-tabs-mode: t
transient-mark-mode: t
abbrev-mode: t
Load-path shadows:
/home/dam/.emacs.d/straight/build/transient/transient hides /usr/local/share/emacs/29.0.50/lisp/transient
/home/dam/.emacs.d/straight/build/jsonrpc/jsonrpc hides /usr/local/share/emacs/29.0.50/lisp/jsonrpc
/home/dam/.emacs.d/straight/build/xref/xref hides /usr/local/share/emacs/29.0.50/lisp/progmodes/xref
/home/dam/.emacs.d/straight/build/flymake/flymake hides /usr/local/share/emacs/29.0.50/lisp/progmodes/flymake
/home/dam/.emacs.d/straight/build/project/project hides /usr/local/share/emacs/29.0.50/lisp/progmodes/project
/home/dam/.emacs.d/straight/build/let-alist/let-alist hides /usr/local/share/emacs/29.0.50/lisp/emacs-lisp/let-alist
/home/dam/.emacs.d/straight/build/eldoc/eldoc hides /usr/local/share/emacs/29.0.50/lisp/emacs-lisp/eldoc
Features:
(shadow sort mail-extr emacsbug misearch multi-isearch pulse vc-mtn
vc-hg vc-bzr vc-src vc-sccs vc-cvs vc-rcs vc cl-print helpful trace
edebug info-look help-fns elisp-refs mule-util form-feed epa-file slack
slack-company slack-unread slack-websocket slack-thread-event
slack-room-event slack-star-event slack-reaction-event slack-reply-event
slack-typing slack-slash-commands slack-message-event slack-event
slack-dialog-edit-element-buffer slack-dialog-buffer slack-dialog
slack-stars-buffer slack-search-result-buffer
slack-thread-message-compose-buffer slack-file-list-buffer
slack-file-info-buffer slack-all-threads-buffer slack-message-buffer
slack-user-profile-buffer slack-pinned-items-buffer slack-pinned-item
slack-thread-message-buffer slack-room-info-buffer slack-room-buffer
slack-message-share-buffer slack-message-edit-buffer
slack-room-message-compose-buffer slack-message-compose-buffer
slack-message-attachment-preview-buffer slack-action slack-star
slack-reminder slack-search slack-message-reaction slack-message-editor
slack-message-sender slack-message-notification alert log4e
notifications dbus gntp slack-buffer slack-message-formatter
slack-thread slack-im slack-channel slack-group slack-conversations
slack-create-message slack-attachment slack-selectable slack-bot-message
slack-user-message slack-file slack-message slack-message-faces
slack-unescape slack-block slack-mrkdwn slack-usergroup slack-reaction
slack-modeline slack-room slack-counts slack-user slack-bot
slack-dnd-status slack-emoji slack-image slack-request slack-log lui
tracking shorten slack-team slack-team-ws slack-util websocket bindat
diary-lib diary-loaddefs cal-menu calendar cal-loaddefs slack-autoloads
emojify-autoloads ht-autoloads alert-autoloads log4e-autoloads
gntp-autoloads circe-autoloads oauth2-autoloads websocket-autoloads
bloomlife bloom-mode eglot array jsonrpc ert ewoc debug backtrace
bloom-sensor gdb-mi bloom-tools bloom-magit bloom-magit-bitbucket
atlassian-bitbucket bloom-magit-build bloom-project bloom-test-runner
bloom-build bloom-cmake bloom-conan bloom-bbw bloom-target bloom-common
bloom-pkg bloomlife-autoloads sonar sonar-autoloads circleci
circleci-autoloads atlassian atlassian-jira atlassian-common request
atlassian-autoloads request-autoloads magit-async-section
magit-async-section-autoloads terraform-mode-autoloads
hcl-mode-autoloads cargo-autoloads racer smartparens-rust rust-utils
rust-mode rust-rustfmt rust-playpen rust-compile rust-cargo
racer-autoloads pos-tip-autoloads rust-mode-autoloads rubocop
rubocop-autoloads robe etags fileloop generator inf-ruby robe-autoloads
inf-ruby-autoloads rbenv rbenv-autoloads ruby-mode smie
smartparens-python python tramp-sh smartparens-scala scala-mode
scala-mode-prettify-symbols scala-mode-imenu scala-mode-map
scala-mode-fontlock scala-mode-indent scala-mode-paragraph
scala-mode-syntax scala-mode-lib scala-mode-autoloads java-imports
pcache eieio-base java-imports-autoloads form-feed-autoloads
elisp-format flymake-proc flymake flyspell elisp-format-autoloads
test-runner-ert elfmt elfmt-autoloads cask-mode cask-mode-autoloads
docker docker-volume docker-network docker-image docker-container
docker-faces docker-compose docker-utils json-mode json-snatcher
smartparens-javascript js docker-core docker-autoloads
json-mode-autoloads json-snatcher-autoloads docker-tramp-autoloads
docker-tramp tramp-cache dockerfile-mode-autoloads djinni-mode
djinni-mode-autoloads cov-autoloads elquery-autoloads gud-lldb cmake-api
cmake-api-autoloads test-runner-catch2 test-runner-catch2-autoloads
eglot-clangd-autoloads eldoc-cmake eldoc-cmake-autoloads
cmake-font-lock-autoloads cmake-mode smartparens-rst rst
cmake-mode-autoloads android android-autoloads pbuf pbuf-autoloads gud
xref forge-list forge-commands forge-semi forge-bitbucket buck
forge-gogs gogs forge-gitea gtea forge-gitlab glab forge-github
forge-notify forge-revnote forge-pullreq forge-issue forge-topic yaml
bug-reference forge-post forge-repo forge forge-core forge-db
forge-autoloads yaml-autoloads closql emacsql-sqlite emacsql
emacsql-compiler closql-autoloads emacsql-sqlite-autoloads
emacsql-autoloads ghub-graphql treepy gsexp ghub url-http url-gw nsm
url-auth ghub-autoloads treepy-autoloads git-timemachine vc-git
vc-dispatcher git-timemachine-autoloads magit-extras magit-bookmark
magit-submodule magit-obsolete magit-blame magit-stash magit-reflog
magit-bisect magit-push magit-pull magit-fetch magit-clone magit-remote
magit-commit magit-sequence magit-notes magit-worktree magit-tag
magit-merge magit-branch magit-reset magit-files magit-refs magit-status
magit magit-repos magit-apply magit-wip magit-log which-func magit-diff
smerge-mode diff-mode git-commit log-edit message yank-media rmc rfc822
mml mml-sec epa derived epg rfc6068 epg-config mm-decode mm-bodies
mm-encode mailabbrev gmm-utils mailheader pcvs-util add-log magit-core
magit-autorevert magit-margin magit-transient magit-process magit-mode
transient magit-git magit-section magit-utils magit-autoloads
magit-section-autoloads git-commit-autoloads transient-autoloads
tree-sitter-langs tree-sitter-langs-build tar-mode arc-mode archive-mode
tree-sitter-hl tree-sitter tree-sitter-load tree-sitter-cli tsc tsc-dyn
tsc-dyn-get tsc-obsolete tree-sitter-langs-autoloads
tree-sitter-autoloads tsc-autoloads eglot-autoloads flymake-autoloads
project-autoloads xref-autoloads eldoc-autoloads jsonrpc-autoloads
dtrt-indent dtrt-indent-autoloads diff ispell reformatter
reformatter-autoloads test-runner project test-runner-autoloads
vterm-autoloads ag vc-svn find-dired ag-autoloads dired-collapse
dired-collapse-autoloads dired-rainbow dired-rainbow-autoloads
dired-narrow delsel dired-narrow-autoloads dired-ranger-autoloads
dired-subtree-autoloads dired-filter f s dired-hacks-utils dired-x
dired-filter-autoloads dired-hacks-utils-autoloads dired-aux
yaml-mode-autoloads plantuml-mode plantuml-mode-autoloads pdf-occur
ibuf-ext ibuffer ibuffer-loaddefs tablist tablist-filter
semantic/wisent/comp semantic/wisent semantic/wisent/wisent
semantic/util-modes semantic/util semantic semantic/tag semantic/lex
semantic/fw mode-local find-func cedet pdf-isearch let-alist pdf-misc
imenu pdf-tools package pdf-view jka-compr pdf-cache pdf-info tq
pdf-util pdf-macs pdf-tools-autoloads let-alist-autoloads
tablist-autoloads xwwp eww xdg url-queue shr kinsoku svg dom mm-url gnus
nnheader gnus-util rmail rmail-loaddefs xwwp-autoloads xwidget
url-handlers image-mode exif browse-url url url-proxy url-privacy
url-expand url-methods url-history url-cookie url-domsuf url-util
mailcap osx-browse-autoloads browse-url-dwim-autoloads
string-utils-autoloads smartparens-markdown markdown-mode noutline
outline markdown-mode-autoloads nov-autoloads esxml-autoloads
kv-autoloads yasnippet yasnippet-autoloads wgrep grep compile
wgrep-autoloads cap-words superword subword string-inflection
string-inflection-autoloads smartparens-config smartparens-text
smartparens-c smartparens-ruby smartparens dash smartparens-autoloads
cc-mode cc-fonts cc-guess cc-menus cc-cmds cc-styles cc-align cc-engine
cc-vars cc-defs mmm-mode mmm-univ mmm-class mmm-region mmm-auto mmm-vars
mmm-utils mmm-compat mmm-mode-autoloads iedit iedit-lib iedit-autoloads
autorevert filenotify marginalia marginalia-autoloads
embark-consult-autoloads consult-dash consult-selectrum consult bookmark
text-property-search consult-dash-autoloads consult-autoloads
embark-autoloads selectrum minibuf-eldef crm selectrum-autoloads
orderless orderless-autoloads corfu corfu-autoloads
ns-auto-titlebar-autoloads all-the-icons-dired-autoloads all-the-icons
all-the-icons-faces data-material data-weathericons data-octicons
data-fileicons data-faicons data-alltheicons all-the-icons-autoloads
ligature ligature-autoloads solarized-dark-high-contrast-theme
solarized-palettes solarized-theme solarized solarized-faces color
solarized-theme-autoloads use-theme use-theme-autoloads password-store
with-editor server password-store-autoloads with-editor-autoloads
auth-source-pass url-parse url-vars unicode-fonts
unicode-fonts-autoloads ucs-utils-autoloads font-utils-autoloads
persistent-soft-autoloads list-utils-autoloads cl pcache-autoloads
daemons daemons-autoloads smtpmail sendmail mail-utils tramp
tramp-loaddefs trampver tramp-integration files-x tramp-compat shell
pcomplete comint ring parse-time iso8601 time-date ls-lisp auth-source
eieio eieio-core eieio-loaddefs password-cache envrc inheritenv
ansi-color envrc-autoloads inheritenv-autoloads eshell esh-cmd esh-ext
esh-opt esh-proc esh-io esh-arg esh-module esh-groups esh-util bepo
which-key advice which-key-autoloads time savehist recentf tree-widget
edmacro kmacro helpful-autoloads elisp-refs-autoloads f-autoloads
s-autoloads dash-autoloads exec-path-from-shell
exec-path-from-shell-autoloads so-long diminish diminish-autoloads
dash-docs use-package-dash-docs gnutls puny format-spec xml json map
dash-docs-autoloads async async-autoloads no-littering
no-littering-autoloads finder-inf use-package-ensure-system-package
system-packages use-package-ensure-system-package-autoloads
system-packages-autoloads use-package use-package-ensure
use-package-delight use-package-diminish use-package-bind-key bind-key
use-package-core use-package-autoloads bind-key-autoloads cus-edit pp
cus-load wid-edit straight-autoloads straight info autoload radix-tree
easy-mmode pcase checkdoc lisp-mnt mail-parse rfc2231 rfc2047 rfc2045
mm-util ietf-drums mail-prsvr dired dired-loaddefs thingatpt comp
comp-cstr warnings rx cl-seq cl-macs cl-extra help-mode seq gv subr-x
byte-opt cl-loaddefs cl-lib bytecomp byte-compile cconv iso-transl
tooltip eldoc paren electric uniquify ediff-hook vc-hooks
lisp-float-type elisp-mode mwheel term/x-win x-win term/common-win x-dnd
tool-bar dnd fontset image regexp-opt fringe tabulated-list replace
newcomment text-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch easymenu timer select scroll-bar mouse jit-lock
font-lock syntax font-core term/tty-colors frame minibuffer cl-generic
cham georgian utf-8-lang misc-lang vietnamese tibetan thai tai-viet lao
korean japanese eucjp-ms cp51932 hebrew greek romanian slovak czech
european ethiopic indian cyrillic chinese composite emoji-zwj charscript
charprop case-table epa-hook jka-cmpr-hook help simple abbrev obarray
cl-preloaded nadvice button loaddefs faces cus-face macroexp files
window text-properties overlay sha1 md5 base64 format env code-pages
mule custom widget keymap hashtable-print-readable backquote threads
xwidget-internal dbusbind inotify lcms2 dynamic-setting
system-font-setting font-render-setting cairo move-toolbar gtk x-toolkit
x multi-tty make-network-process native-compile emacs)
Memory information:
((conses 16 951887 739595)
(symbols 48 65848 4)
(strings 32 298876 339972)
(string-bytes 1 10134341)
(vectors 16 127606)
(vector-slots 8 3430125 852341)
(floats 8 1324 1784)
(intervals 56 13228 1365)
(buffers 992 35))
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2021-11-29 12:33 bug#52178: 29.0.50; process-environment not used in async-shell-command Damien Merenne
@ 2021-11-29 14:38 ` Lars Ingebrigtsen
2021-11-29 20:34 ` Damien Merenne
0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2021-11-29 14:38 UTC (permalink / raw)
To: Damien Merenne; +Cc: 52178
Damien Merenne <dam@cosinux.org> writes:
> When running an async process, the value of `process-environment' is
> lost, in 'emacs -Q' scratch buffer:
>
> (let ((process-environment "EDITOR=test"))
> (shell-command "env &"))
I think you mean
(let ((process-environment '("EDITOR=test")))
(shell-command "env &"))
> and the output is my normal environment, not the limited one. When
> removing the &, it works okay. I managed to pinpoint the problem to the
> call to `with-current-buffer' in simple.el:4229. As process-environment
> is buffer local, it is not passed in the process buffer.
process-environment is not buffer-local by default, though, so the test
case works fine normally. Do you have a complete case to reproduce the
problem, starting from "emacs -Q"?
> This patch fixes it but I have no idea if this is the correct way to
> fix it:
The patch wasn't complete, so I've recreated it and included below for
reference.
It makes sense to me, but I'd like to know where this fixes anything
before applying.
diff --git a/lisp/simple.el b/lisp/simple.el
index 84928caa31..e38988f916 100644
--- a/lisp/simple.el
+++ b/lisp/simple.el
@@ -4194,7 +4194,8 @@ shell-command
(or output-buffer shell-command-buffer-name-async)))
(bname (buffer-name buffer))
(proc (get-buffer-process buffer))
- (directory default-directory))
+ (directory default-directory)
+ (process-env process-environment))
;; Remove the ampersand.
(setq command (substring command 0 (match-beginning 0)))
;; Ask the user what to do with already running process.
@@ -4237,7 +4238,7 @@ shell-command
(format "COLUMNS=%d"
async-shell-command-width)))
(comint-term-environment)
- process-environment)))
+ process-env)))
(setq proc
(start-process-shell-command "Shell" buffer command)))
(setq mode-line-process '(":%s"))
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply related [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2021-11-29 14:38 ` Lars Ingebrigtsen
@ 2021-11-29 20:34 ` Damien Merenne
2022-01-15 13:07 ` Lars Ingebrigtsen
0 siblings, 1 reply; 7+ messages in thread
From: Damien Merenne @ 2021-11-29 20:34 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: 52178
Le lun. 29 nov. 2021 à 14:38, Lars Ingebrigtsen <larsi@gnus.org> a écrit :
> Do you have a complete case to reproduce the
> problem, starting from "emacs -Q"?
So indeed I thought I was able to reprodce it in `emacs -Q` but it was
due to the bug in my snippet.
Now I can't reproduce it anyore with `emacs -Q`.
The `process-environment` seems to be switch to a buffer-local
variable due to some autoload
because right after initialization, its still a normal variable. At
some point I do something and
whoops, it is converted to a buffer local...
I'll try to find what's triggering it but its not obvious.
--
Damien Merenne <dam@cosinux.org>
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2021-11-29 20:34 ` Damien Merenne
@ 2022-01-15 13:07 ` Lars Ingebrigtsen
2022-01-15 21:39 ` Michael Heerdegen
0 siblings, 1 reply; 7+ messages in thread
From: Lars Ingebrigtsen @ 2022-01-15 13:07 UTC (permalink / raw)
To: Damien Merenne; +Cc: 52178
Damien Merenne <dam@cosinux.org> writes:
> I'll try to find what's triggering it but its not obvious.
Did you make any progress here?
--
(domestic pets only, the antidote for overdose, milk.)
bloggy blog: http://lars.ingebrigtsen.no
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2022-01-15 13:07 ` Lars Ingebrigtsen
@ 2022-01-15 21:39 ` Michael Heerdegen
2022-01-16 10:49 ` Damien Merenne
0 siblings, 1 reply; 7+ messages in thread
From: Michael Heerdegen @ 2022-01-15 21:39 UTC (permalink / raw)
To: Lars Ingebrigtsen; +Cc: 52178, Damien Merenne
Lars Ingebrigtsen <larsi@gnus.org> writes:
> Damien Merenne <dam@cosinux.org> writes:
>
> > I'll try to find what's triggering it but its not obvious.
>
> Did you make any progress here?
Damien, maybe using a variable watcher:
(info "(elisp) Watching Variables")
could help to understand what's going on? They are able to notice
changes in buffer-localness.
Michael.
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2022-01-15 21:39 ` Michael Heerdegen
@ 2022-01-16 10:49 ` Damien Merenne
2022-01-16 10:51 ` Damien Merenne
0 siblings, 1 reply; 7+ messages in thread
From: Damien Merenne @ 2022-01-16 10:49 UTC (permalink / raw)
To: Michael Heerdegen; +Cc: 52178, Lars Ingebrigtsen
Thanks for the reminder and the pointer to the watcher! I finally
found that it happens due to the `envrc` package.
That package is meant to set `process-environment` buffer local for
some buffers. Somehow it sets
process-environment globally local but only when emacs is started as a
daemon from the systemd unit. Running
it normally or with --fg-daemon from a terminal does not set the
variable globally local. I'll check with the envrc
maintainer why that would be but there is still something strange
going on, I would not expect Emacs behaviour to
change when started in systemd.
For people stumbling upon this, here is the code I used to pinpoint the problem:
```
(defvar pltraces (list (cons "early-init.el" (local-variable-p
'process-environment))))
(add-hook 'after-load-functions
(lambda (fn)
(setq pltraces (cons (cons fn (local-variable-p
'process-environment)) pltraces))))
(add-variable-watcher 'process-environment
(lambda (symbol newval operation where)
(when where
(setq pltraces (cons (cons (buffer-name
where) (local-variable-p 'process-environment)) pltraces)))))
```
It records the place where the variable is set and its local status in
the `pltraces` variable.
--
Damien MERENNE <dam@cosinux.org>
Le sam. 15 janv. 2022 à 21:39, Michael Heerdegen
<michael_heerdegen@web.de> a écrit :
>
> Lars Ingebrigtsen <larsi@gnus.org> writes:
>
> > Damien Merenne <dam@cosinux.org> writes:
> >
> > > I'll try to find what's triggering it but its not obvious.
> >
> > Did you make any progress here?
>
> Damien, maybe using a variable watcher:
>
> (info "(elisp) Watching Variables")
>
> could help to understand what's going on? They are able to notice
> changes in buffer-localness.
>
> Michael.
^ permalink raw reply [flat|nested] 7+ messages in thread
* bug#52178: 29.0.50; process-environment not used in async-shell-command
2022-01-16 10:49 ` Damien Merenne
@ 2022-01-16 10:51 ` Damien Merenne
0 siblings, 0 replies; 7+ messages in thread
From: Damien Merenne @ 2022-01-16 10:51 UTC (permalink / raw)
To: Michael Heerdegen; +Cc: 52178, Lars Ingebrigtsen
But still, the envrc use case for making process-environment buffer
local for some buffer is still valid and I think the original problem
should be fixed.
--
Damien MERENNE <dam@cosinux.org>
Le dim. 16 janv. 2022 à 10:49, Damien Merenne <dam@cosinux.org> a écrit :
>
> Thanks for the reminder and the pointer to the watcher! I finally
> found that it happens due to the `envrc` package.
> That package is meant to set `process-environment` buffer local for
> some buffers. Somehow it sets
> process-environment globally local but only when emacs is started as a
> daemon from the systemd unit. Running
> it normally or with --fg-daemon from a terminal does not set the
> variable globally local. I'll check with the envrc
> maintainer why that would be but there is still something strange
> going on, I would not expect Emacs behaviour to
> change when started in systemd.
>
> For people stumbling upon this, here is the code I used to pinpoint the problem:
> ```
> (defvar pltraces (list (cons "early-init.el" (local-variable-p
> 'process-environment))))
> (add-hook 'after-load-functions
> (lambda (fn)
> (setq pltraces (cons (cons fn (local-variable-p
> 'process-environment)) pltraces))))
> (add-variable-watcher 'process-environment
> (lambda (symbol newval operation where)
> (when where
> (setq pltraces (cons (cons (buffer-name
> where) (local-variable-p 'process-environment)) pltraces)))))
> ```
> It records the place where the variable is set and its local status in
> the `pltraces` variable.
>
> --
> Damien MERENNE <dam@cosinux.org>
>
> Le sam. 15 janv. 2022 à 21:39, Michael Heerdegen
> <michael_heerdegen@web.de> a écrit :
> >
> > Lars Ingebrigtsen <larsi@gnus.org> writes:
> >
> > > Damien Merenne <dam@cosinux.org> writes:
> > >
> > > > I'll try to find what's triggering it but its not obvious.
> > >
> > > Did you make any progress here?
> >
> > Damien, maybe using a variable watcher:
> >
> > (info "(elisp) Watching Variables")
> >
> > could help to understand what's going on? They are able to notice
> > changes in buffer-localness.
> >
> > Michael.
^ permalink raw reply [flat|nested] 7+ messages in thread
end of thread, other threads:[~2022-01-16 10:51 UTC | newest]
Thread overview: 7+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2021-11-29 12:33 bug#52178: 29.0.50; process-environment not used in async-shell-command Damien Merenne
2021-11-29 14:38 ` Lars Ingebrigtsen
2021-11-29 20:34 ` Damien Merenne
2022-01-15 13:07 ` Lars Ingebrigtsen
2022-01-15 21:39 ` Michael Heerdegen
2022-01-16 10:49 ` Damien Merenne
2022-01-16 10:51 ` Damien Merenne
Code repositories for project(s) associated with this public inbox
https://git.savannah.gnu.org/cgit/emacs.git
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).