[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
bug#55632: [PATCH] Add new user option project-vc-find-tracked-only
From: |
Eli Zaretskii |
Subject: |
bug#55632: [PATCH] Add new user option project-vc-find-tracked-only |
Date: |
Thu, 02 Jun 2022 22:19:24 +0300 |
> Cc: 55632@debbugs.gnu.org, DG <raaahh@gmail.com>
> Date: Thu, 02 Jun 2022 19:01:53 +0000
> From: jan.synacek@posteo.org
>
> ++++
> +*** New user option 'project-vc-include-untracked'.
> +When non-nil, the VC project backend includes the untracked files.
Can we please tell more about what does "include untracked files"
mean? Include where and in what sense? Bonus points for explaining
this without ever alluding to "backend", as that is not necessarily a
user-level concept in this case.
Also, is it "VC project backend" or "Project's VC backend"?
> +(defcustom project-vc-include-untracked t
> + "When non-nil, the VC project backend includes the untracked files."
> + :type 'boolean
> + :safe #'booleanp)
Same here. And new defcustom's should have a :version tag.
> - ;; Include unregistered.
> - (setq args (append args '("-c" "-o" "--exclude-standard")))
> + (setq args (append args
> + '("-c" "--exclude-standard")
> + (when project-vc-include-untracked '("-o"))))
^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^^
I think 'when' is overkill here, because 'if' will do the job.
> - ;; Include unregistered.
> - (setq args (nconc args '("-mcardu" "--no-status" "-0")))
> + (args (list (concat "-mcard" (when project-vc-include-untracked
> "u"))
> + "--no-status"
> + "-0")))
Likewise here.
Thank you for working on this.
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, jan . synacek, 2022/06/01
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, jan . synacek, 2022/06/02
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only,
Eli Zaretskii <=
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, Dmitry Gutov, 2022/06/02
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, Eli Zaretskii, 2022/06/03
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, Dmitry Gutov, 2022/06/03
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, Eli Zaretskii, 2022/06/04
- bug#55632: [PATCH] Add new user option project-vc-find-tracked-only, Dmitry Gutov, 2022/06/04