bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[spvk]

Hi,
Loading slime gives the warning:
Warning (initialization): Your ‘load-path’ seems to contain
your ‘.emacs.d’ directory: /home/spvk/.config/emacs
This is likely to cause problems...
Consider using a subdirectory instead, e.g.:
/home/spvk/.config/emacs/lisp

Maybe it's because of the line:
(setq slime-path (file-name-directory load-file-name))
On slime.el, since the doc says "In case a .eln file is being loaded this is unreliable and ‘load-true-file-name’
should be used instead.".

To reproduce:
1. Start emacs -Q
2. Evaluate the following:
;; START
(defvar bootstrap-version)
(let ((bootstrap-file
       (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
      (bootstrap-version 5))
  (unless (file-exists-p bootstrap-file)
    (with-current-buffer
        (url-retrieve-synchronously
         "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
         'silent 'inhibit-cookies)
      (goto-char (point-max))
      (eval-print-last-sexp)))
  (load bootstrap-file nil 'nomessage))

(straight-use-package 'use-package)

(use-package slime
  :straight t)
;; END


In GNU Emacs 28.0.50 (build 3, x86_64-pc-linux-gnu, GTK+ Version 3.24.13, cairo version 1.16.0)
 of 2020-08-17 built on localhost.localdomain
Repository revision: 76faab27cf4055f6ac37b9b05c98bc03939afb7e
Repository branch: feature/native-comp
Windowing system distributor 'Fedora Project', version 11.0.12006000
System Description: Fedora 31 (Workstation Edition)

Recent messages:
Compilation started.
Type = to expand again, - to contract, 0 to reset
Quit [3 times]
Compilation finished. [2 times]
Type = to expand again, - to contract, 0 to reset
 [2 times]
Type = to expand again, - to contract, 0 to reset

Compilation started.
Compilation finished.

Configured using:
 'configure --with-xml2 --with-json --with-nativecomp CFLAGS=-O3'

Configured features:
XPM JPEG TIFF GIF PNG CAIRO SOUND DBUS GSETTINGS GLIB NOTIFY INOTIFY
LIBSELINUX GNUTLS LIBXML2 FREETYPE HARFBUZZ ZLIB TOOLKIT_SCROLL_BARS
GTK3 X11 XDBE XIM MODULES NATIVE_COMP THREADS JSON PDUMPER

Important settings:
  value of $LANG: en_US.UTF-8
  value of $XMODIFIERS: @im=ibus
  locale-coding-system: utf-8-unix

Major mode: ELisp/l

Minor modes in effect:
  global-highlight-parentheses-mode: t
  highlight-parentheses-mode: t
  smartparens-global-mode: t
  smartparens-mode: t
  global-undo-tree-mode: t
  undo-tree-mode: t
  eyebrowse-mode: t
  yas-minor-mode: t
  show-paren-mode: t
  counsel-mode: t
  async-bytecomp-package-mode: t
  shell-dirtrack-mode: t
  display-time-mode: t
  recentf-mode: t
  lispy-mode: t
  ivy-mode: t
  company-quickhelp-mode: t
  company-quickhelp-local-mode: t
  override-global-mode: t
  straight-use-package-mode: t
  straight-package-neutering-mode: t
  tooltip-mode: t
  global-eldoc-mode: t
  eldoc-mode: t
  electric-indent-mode: t
  mouse-wheel-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
  global-visual-line-mode: t
  visual-line-mode: t
  transient-mark-mode: t

Load-path shadows:
/home/spvk/.config/emacs/straight/build/ivy/elpa hides /home/spvk/.config/emacs/straight/build/lispy/elpa
/home/spvk/.config/emacs/straight/build/auth-source-pass/auth-source-pass hides /usr/local/share/emacs/28.0.50/lisp/auth-source-pass
/home/spvk/.config/emacs/straight/build/let-alist/let-alist hides /usr/local/share/emacs/28.0.50/lisp/emacs-lisp/let-alist

Features:
(shadow sort mail-extr emacsbug sendmail cl-print expand-region
text-mode-expansions cc-mode-expansions the-org-mode-expansions
er-basic-expansions expand-region-core expand-region-custom eieio-opt
speedbar ezimage dframe mc-edit-lines multiple-cursors-core rect
windmove highlight-parentheses projectile grep ibuf-ext ibuffer
ibuffer-loaddefs smartparens ob-async undo-tree diff vc-git diff-mode
eyebrowse dired-rainbow dired-hacks-utils spaceline-config
spaceline-segments spaceline powerline powerline-separators
powerline-themes dired-aux ffap docker-tramp tramp-cache tramp
tramp-loaddefs trampver tramp-integration files-x tramp-compat ls-lisp
disp-table display-line-numbers linum whitespace rainbow-mode
yasnippet-snippets yasnippet org-indent ol-eww ol-rmail ol-mhe ol-irc
ol-info ol-gnus nnir gnus-sum shr svg dom gnus-group gnus-undo
gnus-start gnus-cloud nnimap nnmail mail-source utf7 netrc nnoo
parse-time iso8601 gnus-spec gnus-int gnus-range gnus-win gnus nnheader
ol-docview doc-view jka-compr image-mode exif ol-bbdb ol-w3m org-agenda
doom-acario-light-theme org-static-blog-autoloads exwm-randr xcb-randr
exwm-config exwm exwm-input xcb-keysyms xcb-xkb exwm-manage
exwm-floating xcb-cursor xcb-render exwm-layout exwm-workspace exwm-core
xcb-ewmh xcb-icccm xcb xcb-xproto xcb-types xcb-debug server
exwm-autoloads xelb-autoloads ytdl-autoloads elfeed-autoloads
emms-autoloads pass-autoloads password-store-otp-autoloads
password-store-autoloads auth-source-pass-autoloads
command-log-mode-autoloads treemacs-autoloads pfuture-autoloads
expand-region-autoloads eyebrowse-autoloads centaur-tabs-autoloads
dired-rainbow-autoloads dired-hacks-utils-autoloads peep-dired
peep-dired-autoloads dired-x undo-tree-autoloads
multiple-cursors-autoloads buffer-move-autoloads paren
highlight-parentheses-autoloads smartparens-autoloads
spaceline-autoloads powerline-autoloads which-key-autoloads
ido-vertical-mode-autoloads popup-kill-ring-autoloads
visual-regexp-steroids-autoloads visual-regexp-autoloads ivy-avy
ivy-avy-autoloads counsel xdg doom-themes doom-themes-base
doom-themes-autoloads kaolin-themes kaolin-themes-lib autothemer
kaolin-themes-autoloads autothemer-autoloads ivy-bibtex-autoloads
company-auctex-autoloads auctex-autoloads tex-site ox-reveal-autoloads
ox-gfm-autoloads org-roam-server-autoloads org-roam-autoloads
emacsql-sqlite3-autoloads emacsql-autoloads org-ref org-ref-helm-bibtex
package url-handlers org-ref-helm helm-utils helm-bibtex
bibtex-completion org-capture filenotify biblio biblio-download
biblio-dissemin biblio-ieee biblio-hal biblio-dblp biblio-crossref
biblio-arxiv timezone biblio-doi biblio-core let-alist url-queue ido
hl-line helm-net browse-url xml helm-help helm async-bytecomp
helm-global-bindings helm-easymenu helm-source eieio-compat
helm-multi-match helm-lib async helm-config org-ref-core reftex-cite
reftex reftex-loaddefs reftex-vars parsebib ox-odt rng-loc rng-uri
rng-parse rng-match rng-dt rng-util rng-pttrn nxml-parse nxml-ns
nxml-enc xmltok nxml-util ox-latex ox-icalendar ox-html table ox-ascii
ox-publish ox org-ref-glossary org-ref-bibtex org-ref-citeproc
org-element avl-tree message dired dired-loaddefs rfc822 mml mml-sec epa
derived epg epg-config gnus-util rmail rmail-loaddefs
text-property-search mm-decode mm-bodies mm-encode mailabbrev mail-utils
gmm-utils mailheader key-chord doi-utils org-ref-utils org-ref-pdf
url-http url url-proxy url-privacy url-expand url-methods url-history
mailcap url-auth mail-parse rfc2231 rfc2047 rfc2045 mm-util ietf-drums
mail-prsvr url-cookie url-domsuf url-util url-gw url-parse auth-source
password-cache url-vars nsm rmc puny ol-bibtex bibtex htmlize f
org-ref-autoloads pdf-tools-autoloads key-chord-autoloads
helm-bibtex-autoloads bibtex-completion-autoloads biblio-autoloads
biblio-core-autoloads parsebib-autoloads helm-autoloads
helm-core-autoloads ob-async-autoloads org-bullets org-bullets-autoloads
diminish ob-C ob-lisp ob-shell shell ob-python all-the-icons
all-the-icons-faces data-material data-weathericons data-octicons
data-fileicons data-faicons data-alltheicons memoize
all-the-icons-autoloads memoize-autoloads dashboard dashboard-widgets
time recentf tree-widget wid-edit page-break-lines dashboard-autoloads
page-break-lines-autoloads tiny-autoloads yasnippet-snippets-autoloads
yasnippet-autoloads dockerfile-mode-autoloads docker-autoloads
tablist-autoloads json-mode-autoloads json-snatcher-autoloads
json-reformat-autoloads docker-tramp-autoloads yaml-mode-autoloads tide
tide-lv imenu flycheck s dash json map typescript-mode cc-mode cc-fonts
cc-guess cc-menus cc-cmds cc-styles cc-align cc-engine cc-vars comp
warnings rx cc-defs compile tide-autoloads typescript-mode-autoloads
js2-mode-autoloads request-autoloads impatient-mode-autoloads
htmlize-autoloads simple-httpd-autoloads rainbow-mode-autoloads
web-mode-autoloads emmet-mode cl emmet-mode-autoloads lispy hydra lv
swiper ivy ivy-faces ivy-overlay colir color delsel lispy-inline
thingatpt avy etags fileloop generator xref project eieio eieio-core
eieio-loaddefs edebug backtrace help-fns radix-tree lispy-tags
mode-local lispy-autoloads zoutline-autoloads hydra-autoloads
counsel-autoloads swiper-autoloads ivy-autoloads iedit-autoloads
ace-window-autoloads avy-autoloads suggest-autoloads loop-autoloads
slime-company-autoloads slime-autoloads macrostep-autoloads
go-mode-autoloads ccls-autoloads dumb-jump-autoloads popup-autoloads
flycheck-autoloads let-alist-autoloads projectile-autoloads
pkg-info-autoloads epl-autoloads finder-inf company-lsp-autoloads
lsp-mode-autoloads lv-autoloads markdown-mode-autoloads
spinner-autoloads ht-autoloads f-autoloads s-autoloads
dash-functional-autoloads company-quickhelp pos-tip company pcase
company-quickhelp-autoloads pos-tip-autoloads company-autoloads edmacro
kmacro magit-autoloads git-commit-autoloads with-editor-autoloads
transient-autoloads dash-autoloads async-autoloads
use-package-ensure-system-package system-packages use-package
use-package-ensure use-package-delight use-package-diminish
use-package-bind-key use-package-ensure-system-package-autoloads
system-packages-autoloads org ob ob-tangle ob-ref ob-lob ob-table ob-exp
org-macro org-footnote org-src ob-comint org-pcomplete pcomplete comint
ansi-color ring org-list org-faces org-entities time-date noutline
outline org-version ob-emacs-lisp ob-core ob-eval org-table ol org-keys
org-compat advice org-macs org-loaddefs format-spec find-func cal-menu
calendar cal-loaddefs bind-key easy-mmode diminish-autoloads cl-seq
use-package-core use-package-autoloads bind-key-autoloads
straight-autoloads info cl-extra help-mode easymenu seq byte-opt
straight subr-x cl-macs gv cl-loaddefs cl-lib bytecomp byte-compile
cconv tooltip eldoc electric uniquify ediff-hook vc-hooks
lisp-float-type 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 elisp-mode lisp-mode prog-mode register page tab-bar menu-bar
rfn-eshadow isearch timer select scroll-bar mouse jit-lock font-lock
syntax facemenu 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 charscript charprop
case-table epa-hook jka-cmpr-hook help simple abbrev obarray
cl-preloaded nadvice loaddefs button faces cus-face macroexp files
text-properties overlay sha1 md5 base64 format env code-pages mule
custom widget hashtable-print-readable backquote threads dbusbind
inotify dynamic-setting system-font-setting font-render-setting cairo
move-toolbar gtk x-toolkit x multi-tty make-network-process emacs)

Memory information:
((conses 16 1193212 313300)
 (symbols 48 78545 3199)
 (strings 32 219909 72514)
 (string-bytes 1 8369882)
 (vectors 16 98156)
 (vector-slots 8 2635285 395044)
 (floats 8 976 2395)
 (intervals 56 46402 5956)
 (buffers 992 25))

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Stefan Kangas]

spvk <caiohcs0@gmail.com> writes:

> To reproduce:
> 1. Start emacs -Q
> 2. Evaluate the following:
> ;; START
> (defvar bootstrap-version)
> (let ((bootstrap-file
>        (expand-file-name "straight/repos/straight.el/bootstrap.el" user-emacs-directory))
>       (bootstrap-version 5))
>   (unless (file-exists-p bootstrap-file)
>     (with-current-buffer
>         (url-retrieve-synchronously
>          "https://raw.githubusercontent.com/raxod502/straight.el/develop/install.el"
>          'silent 'inhibit-cookies)
>       (goto-char (point-max))
>       (eval-print-last-sexp)))
>   (load bootstrap-file nil 'nomessage))
>
> (straight-use-package 'use-package)
>
> (use-package slime
>   :straight t)
> ;; END

Do you have a recipe to reproduce this without involving the third-party
packages straight.el and use-package?

> In GNU Emacs 28.0.50 (build 3, x86_64-pc-linux-gnu, GTK+ Version 3.24.13, cairo version 1.16.0)
>  of 2020-08-17 built on localhost.localdomain
> Repository revision: 76faab27cf4055f6ac37b9b05c98bc03939afb7e
> Repository branch: feature/native-comp
> Windowing system distributor 'Fedora Project', version 11.0.12006000
> System Description: Fedora 31 (Workstation Edition)

Can you reproduce this without using the native-comp branch, for example
the current master branch?

Best regards,
Stefan Kangas

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Stefan Kangas]

[Please use "Reply to all" so the discussion is in the bug tracker.]

Caio Henrique <caiohcs0@gmail.com> writes:

> Hi Stefan,
>
>> Do you have a recipe to reproduce this without involving the third-party
>> packages straight.el and use-package?
> I don't have it now, but I'll try to provide one using package.el and
> installing slime from melpa with package-install.

Thanks, it would be useful to hear about your findings.

>> Can you reproduce this without using the native-comp branch, for example
>> the current master branch?
> Now I'm using the master branch:
>
> In GNU Emacs 28.0.50 (build 4, x86_64-pc-linux-gnu, GTK+ Version 3.24.13, cairo version 1.16.0)
>  of 2020-08-20 built on localhost.localdomain
> Repository revision: bf60338d6dd02b4d848229878c8e14182f6f861f
> Repository branch: master
> Windowing system distributor 'Fedora Project', version 11.0.12006000
> System Description: Fedora 31 (Workstation Edition)
>
> And I can't reproduce the issue, that recipe works fine on this version.
>
> Cordially,
> Caio Henrique

So this is specific to the nativecomp branch.

Copying in Andrea Corallo.

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Andrea Corallo via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Stefan Kangas <stefankangas@gmail.com> writes:

> Copying in Andrea Corallo.

Thanks Stefan, is my plan to look into this tomorrow.

Should not very difficult to track down as appear we are doing something
wrong with `load-file-name'.

Thanks

  Andrea

-- 
akrl@sdf.org

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Caio Henrique]

Hi,

I was unable to reproduce the bug without using straight.el, but I can
reproduce it without using use-package.

So after bootstraping straight.el with the recipe that I previously
posted and evaluating the expression: (straight-use-package 'slime),
there is an error. Here is the backtrace:

Debugger entered--Lisp error: (wrong-type-argument stringp nil)
  file-name-directory(nil)
  (or (file-name-directory nil) (car load-path))
  (directory-file-name (or (file-name-directory nil) (car load-path)))
  (add-to-list 'load-path (directory-file-name (or (file-name-directory nil) (car load-path))))
  straight--activate-package-autoloads((:type git :flavor melpa :files ("*.el" ("lib" "lib/hyperspec.el") "swank" "*.lisp" "*.asd" ("contrib" "contrib/*") (:exclude "contrib/test" "contrib/Makefile") "doc/slime.texi" "doc/slime.info" "doc/dir" "ChangeLog" "slime-pkg.el") :host github :repo "slime/slime" :package "slime" :local-repo "slime"))
  #f(compiled-function () #<bytecode 0xe3b8013d73a6767>)()
  straight--transaction-exec(use-package-\(:type\ git\ :flavor\ melpa\ :files\ \(\"*\.el\"\ \(\"lib\"\ \"lib/hyperspec\.el\"\)\ \"swank\"\ \"*\.lisp\"\ \"*\.asd\"\ \(\"contrib\"\ \"contrib/*\"\)\ \(:exclude\ \"contrib/test\"\ \"contrib/Makefile\"\)\ \"doc/slime\.texi\"\ \"doc/slime\.info\"\ \"doc/dir\"\ \"ChangeLog\"\ \"slime-pkg\.el\"\)\ :host\ github\ :repo\ \"slime/slime\"\ :package\ \"slime\"\ :local-repo\ \"slime\"\)-nil-nil :now #f(compiled-function () #<bytecode 0xe3b8013d73a6767>))
  straight-use-package(slime)
  (progn (defvar bootstrap-version) (straight-use-package 'slime))
  elisp--eval-last-sexp(nil)
  eval-last-sexp(nil)
  funcall-interactively(eval-last-sexp nil)
  command-execute(eval-last-sexp)

Cordially,
Caio Henrique

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Andrea Corallo via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Hi Caio,

the following should solve this issue

38b0ead7c1 * Back using `load-file-name' when reading '#$' (bug#42961) 

Could you give a try to the latest state of the branch and confirm?

Thanks!

  Andrea

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Caio Henrique]

Hi Andrea,

It is working now, thanks!

Cordially,
Caio Henrique

bug#42961: 28.0.50; [feature/native-comp] slime load-path warning

[Andrea Corallo via Bug reports for GNU Emacs, the Swiss army knife of text editors]

Caio Henrique <caiohcs0@gmail.com> writes:

> Hi Andrea,
>
> It is working now, thanks!
>
> Cordially,
> Caio Henrique

Great! closing

Thank you

  Andrea