bug#58447: closed ([PATCH] In project-find-file, add absolute file name to history)

[GNU bug Tracking System]

Your message dated Sat, 26 Nov 2022 04:28:50 +0200
with message-id <c0666132-9837-17e1-c9e1-576c718a867a@yandex.ru>
and subject line Re: bug#58447: [PATCH] In project-find-file, add absolute file 
name to history
has caused the debbugs.gnu.org bug report #58447,
regarding [PATCH] In project-find-file, add absolute file name to history
to be marked as done.

(If you believe you have received this mail in error, please contact
help-debbugs@gnu.org.)


-- 
58447: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=58447
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems

--- Begin Message --- Subject: [PATCH] In project-find-file, add absolute file name to history Date: Tue, 11 Oct 2022 20:29:24 +0200 User-agent: Gnus/5.13 (Gnus v5.13)

Tags: patch

`project-find-file' and related commands share their history with more
general commands like `find-file', so they should add the absolute file
name to history.  Currently, file names relative to the project root are
added.  The attached patch fixes this.

I've also replaced `concat' with `expand-file-name' at a certain point.
I see no reason why one shouldn't do that, but please have a look.

0001-In-project-find-file-and-the-like-add-absolute-file-.patch
Description: Text Data

--- End Message ---

> --- Begin Message ---
>
>
>
> Subject: 
>
> Re: bug#58447: [PATCH] In project-find-file, add absolute file name to history
>
>
>
>
> Date: 
>
> Sat, 26 Nov 2022 04:28:50 +0200
>
>
>
>
> User-agent: 
>
> Mozilla/5.0 (X11; Linux x86_64; rv:102.0) Gecko/20100101 Thunderbird/102.4.2
>
>
>
>
> On 18/11/22 18:28, Augusto Stoffel wrote:
> > On Wed,  2 Nov 2022 at 00:14, Dmitry Gutov wrote:
> >
> > > On 01.11.2022 19:30, Juri Linkov wrote:
> > > > > I'm not quite sure what Juri meant here.  If from some project I do
> > > > >
> > > > >     C-x p f /tmp RET RET
> > > > >
> > > > > then Emacs opens the /tmp dired buffer (RET twice because REQUIRE-MATCH
> > > > > is 'confirm), as I think it should.
> > > > Actually, that [Confirm] part is not quite a hassle, so your patch
> > > > already handles that in both ways, thanks for this.
> > > Thanks for verifying.
> > >
> > > LGTM then.
> > FTR, someone would have to install the patch on my behalf.  I've
> > attached it again for your convenience.
>
> Sorry, I figured you had commit access.
>
> Now pushed, thanks.
>
>
>
> --- End Message ---