emacs-bug-tracker
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#55632: closed ([PATCH] Add new user option project-vc-find-tracked-o


From: GNU bug Tracking System
Subject: bug#55632: closed ([PATCH] Add new user option project-vc-find-tracked-only)
Date: Sat, 04 Jun 2022 00:38:02 +0000

Your message dated Sat, 4 Jun 2022 03:37:21 +0300
with message-id <ed43d7f9-00ea-7528-4e53-9e0383742f5a@yandex.ru>
and subject line Re: bug#55632: [PATCH] Add new user option 
project-vc-find-tracked-only
has caused the debbugs.gnu.org bug report #55632,
regarding [PATCH] Add new user option project-vc-find-tracked-only
to be marked as done.

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


-- 
55632: https://debbugs.gnu.org/cgi/bugreport.cgi?bug=55632
GNU Bug Tracking System
Contact help-debbugs@gnu.org with problems
--- Begin Message --- Subject: [PATCH] Add new user option project-vc-find-tracked-only Date: Wed, 25 May 2022 09:08:54 +0000
Tags: patch

Tags: patch


Currently, `project-find-file' always includes untracked files, which is
not always the desired behavior. This patch adds a new user option to
make only find the actual project files. By default, the variable is set
to nil, which means the behavior is not changed.

In GNU Emacs 28.1.50 (build 1, x86_64-pc-linux-gnu, GTK+ Version 3.24.33, cairo 
version 1.17.6)
 of 2022-05-21 built on jsynacek-home
Repository revision: 9e7c0cf57d522b50423880f3a846c52c5509fef9
Repository branch: emacs-28
Windowing system distributor 'The X.Org Foundation', version 11.0.12101003
System Description: Arch Linux

Configured using:
 'configure --with-imagemagick --with-json --with-native-compilation
 --prefix=/home/jsynacek/emacs'

Attachment: 0001-Add-new-user-option-project-vc-find-tracked-only.patch
Description: Text Data


--- End Message ---
--- Begin Message --- Subject: Re: bug#55632: [PATCH] Add new user option project-vc-find-tracked-only Date: Sat, 4 Jun 2022 03:37:21 +0300 User-agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.9.1
Version: 29.1

On 03.06.2022 08:44, Eli Zaretskii wrote:
Date: Fri, 3 Jun 2022 02:45:19 +0300
Cc: 55632@debbugs.gnu.org, raaahh@gmail.com
From: Dmitry Gutov <dgutov@yandex.ru>

On 02.06.2022 22:19, Eli Zaretskii wrote:
Can we please tell more about what does "include untracked files"
mean?  Include where and in what sense?

Has them considered to be part of the project. Which in practice means
including them in a project's list of files. For the purposes for
project-find-file, project-find-regexp, and all other commands that
build on top of 'project-files'.

How would you phrase that better?

Something like this:

   If non-nil, files untracked by a VCS are considered to be part of
   the project by a VC project based on that VCS.

A bit unwieldy IMHO, but I don't mind. As long as you only objected to the NEWS entry.

The addition to the manual, which is also usually considered user-level text, only continues the style of the preceding sentence. And the docstring has to be precise either way.

Bonus points for explaining
this without ever alluding to "backend", as that is not necessarily a
user-level concept in this case.

The variable only affects a particular backend. It's only meaningful
when there is a VCS anyway.

I understand.  My point was to avoid using the word "backend" in
user-level documentation, as much as possible.

Also, is it "VC project backend" or "Project's VC backend"?

I would say both mean the same thing, but the latter seems to be more
unambiguous.

Well, if you are okay with my alternative wording above, it solves
this problem as well.

Amended and pushed, thank you both.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]