[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#64242: [PATCH] Fix VC package build when there is no docs dir
From: |
Philip Kaludercic |
Subject: |
bug#64242: [PATCH] Fix VC package build when there is no docs dir |
Date: |
Sat, 24 Jun 2023 10:24:47 +0000 |
Philip Kaludercic <philipk@posteo.net> writes:
> Eli Zaretskii <eliz@gnu.org> writes:
>
>>> From: Daniel Semyonov <daniel@dsemy.com>
>>> Cc: 64242@debbugs.gnu.org
>>> Date: Fri, 23 Jun 2023 09:32:01 +0300
>>>
>>> >>>>> Eli Zaretskii writes:
>>>
>>> > What do you mean by "isn't a directory"? FILE is supposed to be a
>>> > documentation file, not a directory, see the doc string of this
>>> > function. Do you mean that FILE is a relative file name so that
>>> > file-name-directory returns nil for it? If so, I think the
>>> > correct fix would be to reverse the order:
>>>
>>> > (docs-directory (file-name-directory (expand-file-name file)))
>>>
>>> Yeah, you're completely right, this was a misunderstanding on my part;
>>> my fix just happened to also work so I made some incorrect assumptions.
>>>
>>> After some further testing, this seems to happen for packages for which
>>> the doc file is in the base directory of the package (which causes
>>> 'file-name-directory' to return nil as the doc file name is relative to
>>> the package dir).
>>>
>>> I attached an updated patch.
>>>
>>> > Can you show a recipe to reproduce this problem, preferably
>>> > starting from "emacs -Q"?
>>>
>>> 1. $ emacs -Q
>>> 2. M-x package-vc-install RET eat RET
>>> 3. A few seconds later,
>>> "package-vc--unpack-1: Wrong type argument: stringp, nil"
>>>
>>> > And finally, this issue exists on the emacs-29 release branch as
>>> > well, doesn't it?
>>>
>>> I haven't tested it, but I don't see a reason it wont.
>>
>> Philip, any comments or objections to installing this on the emacs-29
>> branch?
>
> I have none. There were just come complains last time, because we made
> the change too soon, so I believe the consensus was to wait for a bit
> longer and then it was just forgotten.
Oops, I misread this thread and was under the assumption you were
responding to a different message. Sorry about that! Shouldn't write
messages when I am in a hurry.
Regarding Daniel's patch, I don't believe we need `file-name-directory'
at all, the variable `docs-directory' is just passed to makeinfo -I, and to
my knowledge it should make a difference if the argument ends with a
directory delimiter or not.
--
Philip Kaludercic
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Daniel Semyonov, 2023/06/23
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Eli Zaretskii, 2023/06/23
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Daniel Semyonov, 2023/06/23
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Eli Zaretskii, 2023/06/23
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Philip Kaludercic, 2023/06/23
- bug#64242: [PATCH] Fix VC package build when there is no docs dir,
Philip Kaludercic <=
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Eli Zaretskii, 2023/06/24
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Philip Kaludercic, 2023/06/24
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Eli Zaretskii, 2023/06/24
- bug#64242: [PATCH] Fix VC package build when there is no docs dir, Philip Kaludercic, 2023/06/25