Re: package-vc-install bug?

[James Thomas <jimjoe@gmx.net>]

Philip Kaludercic wrote:

> James Thomas <jimjoe@gmx.net> writes:
>
>> Philip Kaludercic wrote:
>>
>>> James Thomas <jimjoe@gmx.net> writes:
>>>
>>>> M-x package-vc-install RET <package> RET fails with the following on my
>>>> system:
>>>
>>> Does this issue occur for every package?
>>
>> Yes, for every one I've tried (in my system).
>
> In that case this should be reported as a bug.
>
>>> What I don't get, is what adding a newline to the end of a string should
>>> change?
>>>
>>> (read-from-string "((emacs \"29.1\"))")   ;=> (((emacs "29.1")) . 16)
>>> (read-from-string "((emacs \"29.1\"\n))") ;=> (((emacs "29.1")) . 17)
>>
>> The problem occurs in `package-read-from-string', not `read-from-string'
>
> I understood that, the point is that the cdr of the return value in both
> cases points to the end of the string, so the second `read-from-string'
> call in the `condition-case' will in both cases raise the same
> `end-of-file', as is the case here:
>
>   (read-from-string "")
>   (read-from-string "x" 1)

Yes, you're right: apparently my modification has nothing to do with it
(see below). Sorry.

> Can you confirm this by using edebug?

It turns out that if I do C-M-x on `package-read-from-string' and then
try it, it works. But not without it on a fresh restart, even with emacs
-Q. Even stepping through using edebug without rerunning the defun
works.

I'm not sure what the problem is: maybe the build? Have to dig deeper.

Thank you for all your help.

Regards,
James