emenel/doomemacs
1
0
Fork 0
Code Issues Pull requests Projects Releases Packages Wiki Activity

fix(cli): "Argument list too long" error from after-scripts

Browse source 
Because a persisted envvar was larger than MAX_ARG_STRLEN (which is
typically ~2kb).
This commit is contained in:
Henrik Lissner 2024-09-12 16:10:14 -04:00
parent 14189be77c
commit 0e5935f0f7
No known key found for this signature in database
GPG key ID: B60957CA074D39A3
1 changed files with 3 additions and 1 deletions
Download patch file Download diff file Expand all files Collapse all files

4
lisp/doom-cli-lib.el
View file

@ -1179,7 +1179,9 @@ Emacs' batch library lacks an implementation of the exec system call."
"_doomcleanup() {\n rm -f " ,persistent-files "\n}\n"
"_doomrun() {\n " ,command "\n}\n"
,(cl-loop for (var . val) in persisted-env
concat (format "%s=%s \\\n" var (shell-quote-argument val)))
if (<= (length val) 2048) ; Prevent "Argument list too long" errors
concat (format "%s=%s \\\n" var (shell-quote-argument val))
else do (doom-log 1 "restart: wiscarding envvar %S for being too long (%d)" var (length val)))
,(format "PATH=\"%s%s$PATH\" \\\n"
(doom-path doom-emacs-dir "bin")
path-separator)