Philip Kaludercic writes: > Eli Zaretskii writes: > >>> Cc: 63337@debbugs.gnu.org >>> Date: Sun, 07 May 2023 11:40:46 -0700 >>> From: Joseph Turner via "Bug reports for GNU Emacs, >>> the Swiss army knife of text editors" >>> >>> > According to the docs, makeinfo has -I to append the search path, and -P >>> > to prepend. I don't know how well either of the two are supported, but >>> > assuming they are, shouldn't -P be preferred? Or wouldn't it have any >>> > effect? >>> >>> I am not sure what difference it would make. I don't know if the default >>> @include search path includes anything besides the working directory. > > I don't know that either, and I can imagine that certain versions of > makeinfo might be patched or this could change in the future. > >> It doesn't, according to the Texinfo manual. Only the current >> directory is searched. >> >>> In the attached diff, I have changed -I to -P. >> >> I think it's a mistake: the current directory should searched first. >> So -I is better. > > What do we mean by the current directory? When building the manual from > an org-file, we switch to a temporary directory (where the .org -> .texi > conversion is stored), so the "actual" directory is not the same as the > default-directory. AFAICT, makeinfo searches the default-directory. See attached patch, where we let-bind default-directory to the docs-directory. In this case, neither -I nor -P is necessary. It's a bit strange to let-bind default-directory twice in the same function, but we can't bind it at the top of the function, the insert-file-contents expects default-directory to be package-desc-dir. Joseph