From: Stephen Berman <stephen.berman@gmx.net>
To: Hongyi Zhao <hongyi.zhao@gmail.com>
Cc: help-gnu-emacs <help-gnu-emacs@gnu.org>
Subject: Re: let*: Wrong type argument: stringp, nil
Date: Sat, 25 Sep 2021 13:04:59 +0200 [thread overview]
Message-ID: <87ee9csyp0.fsf@gmx.net> (raw)
In-Reply-To: <CAGP6POLZE2OhpGppikRyBCx_sJzXD1TKCco8zBD3oVSGAZx3HA@mail.gmail.com> (Hongyi Zhao's message of "Sat, 25 Sep 2021 18:45:28 +0800")
On Sat, 25 Sep 2021 18:45:28 +0800 Hongyi Zhao <hongyi.zhao@gmail.com> wrote:
> On Sat, Sep 25, 2021 at 6:09 PM Stephen Berman <stephen.berman@gmx.net> wrote:
>>
>> On Fri, 24 Sep 2021 22:52:45 +0800 Hongyi Zhao <hongyi.zhao@gmail.com> wrote:
>>
>> > According to the idea represented here [1], I wrote the following
>> > function to activate the python virtual environment automatically
>> > using the content of the ".python-version" file located in the project
>> > directory of the current opened python file:
>> >
>> > (defun pyvenv-autoload ()
>> > (require 'projectile)
>> > (let* ((pdir (file-name-directory buffer-file-name)) (pfile
>> > (concat pdir ".python-version")))
>> > (if (file-exists-p pfile)
>> > (pyvenv-workon (with-temp-buffer
>> > (insert-file-contents pfile)
>> > (nth 0 (split-string (buffer-string))))))))
>> > (add-hook 'python-mode-hook 'pyvenv-autoload)
>> >
>> > It seems the corresponding python virtual environment can be activated
>> > automatically when I open the python file in Emacs. But when I try to
>> > evaluate the file, the following message will be triggered, and defeat
>> > the evaluate operation:
>> >
>> > let*: Wrong type argument: stringp, nil
>> >
>> > [1] https://github.com/jorgenschaefer/pyvenv/issues/51#issuecomment-253902450
>> >
>> > Any hints for fixing this problem?
>>
>> Evidently buffer-file-name is nil when the sexp `(file-name-directory
>> buffer-file-name)' is evaluated, which indicates the current buffer when
>> that function is called is not visiting a file (e.g., you get the same
>> error when you evaluate `(file-name-directory buffer-file-name)' in
>> *scratch*).
>
> Thank you for pointing this out. Based on your above comment, I fixed
> the problem with the code below, and it does the trick:
>
> (defun pyvenv-autoload ()
> (require 'projectile)
> (if (buffer-file-name)
> (let* ((pdir (file-name-directory buffer-file-name)) (pfile
> (concat pdir ".python-version")))
> (if (file-exists-p pfile)
> (pyvenv-workon (with-temp-buffer
> (insert-file-contents pfile)
> (nth 0 (split-string (buffer-string)))))))))
> (add-hook 'python-mode-hook 'pyvenv-autoload)
>
> BTW, can I use `when' to write the above code?
Certainly, indeed `when' is typically favored over `if' when there's no
'else' branch, as in the above code. On the other hand, since the point
of the function is activate a virtual environment on visiting a Python
file, you might want to add an 'else' branch that returns a suitable
message when the current buffer is not visiting a file. Without that,
in such a case the function just returns nil, so there's no indication
the activation failed.
BTW, your code would be easier to read if you began the second clause of
the let* on a new line:
(let* ((pdir (file-name-directory buffer-file-name))
(pfile (concat pdir ".python-version")))
Steve Berman
next prev parent reply other threads:[~2021-09-25 11:04 UTC|newest]
Thread overview: 35+ messages / expand[flat|nested] mbox.gz Atom feed top
2021-09-24 14:52 let*: Wrong type argument: stringp, nil Hongyi Zhao
2021-09-24 14:57 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-25 1:29 ` Hongyi Zhao
2021-09-25 23:42 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-25 10:09 ` Stephen Berman
2021-09-25 10:45 ` Hongyi Zhao
2021-09-25 11:04 ` Stephen Berman [this message]
2021-09-25 13:49 ` Hongyi Zhao
2021-09-25 14:45 ` Stephen Berman
2021-09-25 15:03 ` Hongyi Zhao
2021-09-28 3:08 ` Hongyi Zhao
2021-09-28 3:21 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-28 3:24 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-28 5:25 ` Hongyi Zhao
2021-09-28 7:20 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-28 8:02 ` Yuri Khan
2021-09-28 8:24 ` Hongyi Zhao
2021-09-29 1:43 ` Hongyi Zhao
2021-09-29 2:03 ` Hongyi Zhao
2021-09-29 3:02 ` Hongyi Zhao
2021-09-29 3:57 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-29 6:15 ` Hongyi Zhao
2021-09-29 6:29 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-29 7:37 ` Hongyi Zhao
2021-09-29 8:21 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-29 9:02 ` Hongyi Zhao
2021-09-29 9:14 ` Yuri Khan
2021-09-29 10:29 ` Hongyi Zhao
2021-09-29 12:29 ` Yuri Khan
2021-09-29 12:49 ` Hongyi Zhao
2021-10-08 3:36 ` Hongyi Zhao
2021-10-10 14:05 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-30 3:38 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-29 4:35 ` Emanuel Berg via Users list for the GNU Emacs text editor
2021-09-29 6:17 ` Hongyi Zhao
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
List information: https://www.gnu.org/software/emacs/
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=87ee9csyp0.fsf@gmx.net \
--to=stephen.berman@gmx.net \
--cc=help-gnu-emacs@gnu.org \
--cc=hongyi.zhao@gmail.com \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
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).