From: Duy Nguyen Date: Mon, 25 Mar 2019 09:53:22 GMT Subject: Re: [PATCH v1 01/11] checkout: split part of it to new command 'restore' Message-ID: <20190325095322.GA1617@ash> In-Reply-To: (Including Stefan, in case he's still interested in git and planned something for the "submodule" restore part I mention below) On Fri, Mar 08, 2019 at 10:01:25AM -0800, Elijah Newren wrote: > Thanks for working on this; overall looks really good. I've got a few > comments here and there on the wording and combinations of options... OK now that the 'git switch' part is mostly settled, I could go back to 'git restore'. I have a lot more homework to do before sending v2, but how about this? Most of this is minor rephrasing and clarification (thanks!). The big change is `--force` is split into `--keep-unmerged` and `--force-submodules`, the same direction I took in 'git switch'. Submodules actually pose another problem because there are different things in a submodule to restore: - restore HEAD only, leave submodule's worktree alone. This is `--no-recurse-submodules` (default) - restoring `HEAD`, which is basically "git switch --detach " inside the submodule. This is `--recurse-submodules`. - restoring the paths inside the submodules, i.e. the actual "git restore " inside the submodule. - a combinarion of two. Basically you want "git reset --hard" in every selected submodule. I wonder if we ever need to cover the third and forth case (and if so, what will the UI be like). I suppose we could eventually make --recurse-submodules take an argument to specify what to restore, maybe... PS. I just found https://asciidoclive.com/. Could be useful to paste this text and render without going through git's build system. -- 8< -- git-restore(1) ============== NAME ---- git-restore - Restore working tree files SYNOPSIS -------- [verse] 'git restore' [] [--source=] [--staged] [--worktree] ... 'git restore' (-p|--patch) [] [--source=] [--staged] [--worktree] [...] DESCRIPTION ----------- Restore specified paths in the working tree with some contents from a restore source. If a path is tracked but does not exist in the restore source, it will be removed to match the source. The command can also be used to restore the content in the index with `--staged`, or restore both the working tree and the index with `--staged --worktree`. By default, the restore sources for working tree and the index are the index and `HEAD` respectively. `--source` could be used to specify a commit as the restore source. A note about the differences between this command and linkgit:git-reset[1]. `git restore` modifies the working tree (and maybe the index) to change file content to match some other, usually older, version, but does not update your branch. `git reset` is mainly about moving the tip of your branch in order to either add or remove commits from the branch. OPTIONS ------- -s :: --source=:: Restore the working tree files with the content from the given tree. It is common to specify the source tree by naming a commit, branch or tag associated with it. + If not specified, the default restore source for the working tree is the index, and index `HEAD`. When both `--staged` and `--worktree` are specified, `--source` must also be specified. -p:: --patch:: Interactively select hunks in the difference between the restore source and the restore location. See the ``Interactive Mode'' section of linkgit:git-add[1] to learn how to operate the `--patch` mode. + Note that `--patch` can accept no pathspec and will restore all modified paths. -W:: --worktree:: -S:: --staged:: Specify the restore location. If neither option is specified, by default the working tree is restored. Specifying `--staged` will only restore the index. Specifying both restores both. -q:: --quiet:: Quiet, suppress feedback messages. --progress:: --no-progress:: Progress status is reported on the standard error stream by default when it is attached to a terminal, unless `--quiet` is specified. This flag enables progress reporting even if not attached to a terminal, regardless of `--quiet`. -f:: --force:: An alias of `--keep-unmerged` and `--force-submodules`. --keep-unmerged:: If `--source` is not specified, unmerged entries are left alone and will not fail the operation. --force-submodules:: Continue even if restoring `HEAD` of a submodule leads to loss of local changes in that submodule with `--recurse-submodules`. --ours:: --theirs:: Restore from stage #2 ('ours') or #3 ('theirs') for unmerged paths. + Note that during `git rebase` and `git pull --rebase`, 'ours' and 'theirs' may appear swapped. See the explanation of the same options in linkgit:git-checkout[1] for details. -m:: --merge:: Recreate the conflicted merge in the specified paths. --conflict=