Re: Missing dependency for emacs-magit

[Liliana Marie Prikler]

Hi Zelphir,

Am Dienstag, dem 15.02.2022 um 09:45 +0000 schrieb Zelphir Kaltstahl:
> Hello Liliana,
> 
> On 2/14/22 20:33, Liliana Marie Prikler wrote:
> > Hi Zelphir,
> > 
> > Am Montag, dem 14.02.2022 um 18:41 +0000 schrieb Zelphir Kaltstahl:
> > > [...]
> > I think we need some multi-level printf debugging here.
> > 
> > First, after  `source "${GUIX_PROFILE}/etc/profile"`, is
> > EMACSLOADPATH
> > correctly pointing to the share/emacs/site-lisp of
> > ${GUIX_EXTRA_PROFILES}/emacs-test-profile?
> > 
> > Second, before and after executing your init.el snippet, what are
> > the
> > contents of load-path?
> > Do they contain /gnu/store/SOME_LONG_HASH-magit-MAGIT_VERSION?
> > 
> > Cheers
> 
> To simplify things, I created a shell script, which does the calls:
> 
> ~~~~
> #!/usr/bin/env bash
> 
> set -Eeuxo pipefail
> 
> # source the environment
> GUIX_PROFILE="${GUIX_EXTRA_PROFILES}/emacs-test-profile"
> . "${GUIX_PROFILE}/etc/profile"
> printf "emacs load path: %s\n" "${EMACSLOADPATH}"
> 
> # run emacs
> env XMODIFIERS='' emacs
> ~~~~
> 
> This outputs the following:
> 
> ~~~~
> [...]
> ~~~~
> 
> Magit line is:
> 
> ~~~~
> "/gnu/store/f0461m96rhnpkmhjlj06yz058pqyj02d-emacs-magit-
> 3.3.0/share/emacs/site-lisp/magit-3.3.0"
> ~~~~
Okay, this means that subdirs.el is correctly expanded.  The next
question regards autoloads.  Is (featurep 'magit-autoloads) t or nil?

> Seems to be the same as without running the `init.el` file regarding
> magit at least. This makes sense, as I don't add things to the load-
> path in the whole `init.el` file, except for:
> 
> ~~~~
> (defconst xl-home "~")
> (defconst xl-guix-profiles (concat xl-home "/" ".guix-extra-
> profiles"))
> (defconst xl-emacs-profile (concat xl-guix-profiles "/" "emacs-test-
> profile"))
> (defconst xl-site-lisp-dir (concat xl-emacs-profile "/"
> "share/emacs/site-lisp/"))
> (add-to-list 'load-path
>               xl-site-lisp-dir)
> ~~~~
That snippet should not be needed.  As can be seen from EMACSLOADPATH,
xl-site-lisp-dir should already be in load-path.

> (Not sure if defconst is the best thing to use, but emacs was
> complaining when using setq, that I was assigning to variables, which
> are free, and those strings are actually constant, so I figured that
> would be appropriate to do and emacs does not complain about it any
> longer.)
Let-binding would be nicer, but this is not the coding academy awards,
so anything goes.

Cheers