Message ID | pull.1847.git.1735996601092.gitgitgadget@gmail.com (mailing list archive) |
---|---|
State | New |
Headers | show |
Series | doc: git-restore: migrate to new style format | expand |
On Sat, Jan 04, 2025 at 01:16:40PM +0000, Jean-Noël Avila via GitGitGadget wrote: > From: =?UTF-8?q?Jean-No=C3=ABl=20Avila?= <jn.avila@free.fr> > > The git-restore manpage was converted to the new documentation > format: Commit messages should typically use imperative style, as if asking the code to change. For example: Convert the git-restore(1) man page to our new documentation format. This includes the following conversions: - Switch the synopsis to a 'synopsis' block, which will automatically format placeholders in italics and keywords in monospace. - Use `_<placeholder>_` instead of `<placeholder>` in the description. - Use backticks for keywords and more complex option descriptions. The new rendering engine will apply synopsis rules to these spans. > diff --git a/Documentation/git-restore.txt b/Documentation/git-restore.txt > index 975825b44aa..541a39b5d28 100644 > --- a/Documentation/git-restore.txt > +++ b/Documentation/git-restore.txt > @@ -41,79 +41,79 @@ OPTIONS > If not specified, the contents are restored from `HEAD` if `--staged` is > given, otherwise from the index. > + > -As a special case, you may use `"A...B"` as a shortcut for the > -merge base of `A` and `B` if there is exactly one merge base. You can > -leave out at most one of `A` and `B`, in which case it defaults to `HEAD`. > +As a special case, you may use `"<refA>...<refB>"` as a shortcut for the > +merge base of _<refA>_ and _<refB>_ if there is exactly one merge base. You can > +leave out at most one of _<refA>__ and _<refB>_, in which case it defaults to `HEAD`. This change is a bit surprising to me though. Why was this renamed from A and B to refA and refB, respectively? It should be possible for these to be object IDs and not refs. > @@ -122,30 +122,29 @@ in linkgit:git-checkout[1] for details. > not be updated. Just like linkgit:git-checkout[1], this will detach > `HEAD` of the submodule. > > ---overlay:: > ---no-overlay:: > - In overlay mode, the command never removes files when > - restoring. In no-overlay mode, tracked files that do not > - appear in the `--source` tree are removed, to make them match > - `<tree>` exactly. The default is no-overlay mode. > - > ---pathspec-from-file=<file>:: > - Pathspec is passed in `<file>` instead of commandline args. If > - `<file>` is exactly `-` then standard input is used. Pathspec > - elements are separated by LF or CR/LF. Pathspec elements can be > +`--overlay`:: > +`--no-overlay`:: > + In overlay mode, never remove files when restoring. In no-overlay mode, > + remove tracked files that do not appear in the `--source` tree, to make > + them match _<tree>_ exactly. The default is no-overlay mode. > + > +`--pathspec-from-file=<file>`:: > + Pathspec is passed in _<file>_ instead of commandline args. If > + _<file>_ is exactly `-` then standard input is used. Pathspec > + elements are separated by _LF_ or _CR_/_LF_. Pathspec elements can be The reflowing of these paragraphs makes it a bit hard to see what exactly is changing. Thanks! Patrick
On Monday, 6 January 2025 07:48:27 CET Patrick Steinhardt wrote: > On Sat, Jan 04, 2025 at 01:16:40PM +0000, Jean-Noël Avila via GitGitGadget wrote: > > From: =?UTF-8?q?Jean-No=C3=ABl=20Avila?= <jn.avila@free.fr> > > > > The git-restore manpage was converted to the new documentation > > format: > > Commit messages should typically use imperative style, as if asking the > code to change. For example: > > Convert the git-restore(1) man page to our new documentation format. > This includes the following conversions: > > - Switch the synopsis to a 'synopsis' block, which will > automatically format placeholders in italics and keywords in > monospace. > > - Use `_<placeholder>_` instead of `<placeholder>` in the > description. > > - Use backticks for keywords and more complex option descriptions. > The new rendering engine will apply synopsis rules to these spans. > Will do. > > diff --git a/Documentation/git-restore.txt b/Documentation/git-restore.txt > > index 975825b44aa..541a39b5d28 100644 > > --- a/Documentation/git-restore.txt > > +++ b/Documentation/git-restore.txt > > @@ -41,79 +41,79 @@ OPTIONS > > If not specified, the contents are restored from `HEAD` if `--staged` is > > given, otherwise from the index. > > + > > -As a special case, you may use `"A...B"` as a shortcut for the > > -merge base of `A` and `B` if there is exactly one merge base. You can > > -leave out at most one of `A` and `B`, in which case it defaults to `HEAD`. > > +As a special case, you may use `"<refA>...<refB>"` as a shortcut for the > > +merge base of _<refA>_ and _<refB>_ if there is exactly one merge base. You can > > +leave out at most one of _<refA>__ and _<refB>_, in which case it defaults to `HEAD`. > > This change is a bit surprising to me though. Why was this renamed from > A and B to refA and refB, respectively? It should be possible for these > to be object IDs and not refs. Sorry for mixing things up revs with refs. The "A" and "B" are just placeholders for revisions, and their names need to be more informative. So, it should be <rev-A> and <rev-B> all along. > > > @@ -122,30 +122,29 @@ in linkgit:git-checkout[1] for details. > > not be updated. Just like linkgit:git-checkout[1], this will detach > > `HEAD` of the submodule. > > > > ---overlay:: > > ---no-overlay:: > > - In overlay mode, the command never removes files when > > - restoring. In no-overlay mode, tracked files that do not > > - appear in the `--source` tree are removed, to make them match > > - `<tree>` exactly. The default is no-overlay mode. > > - > > ---pathspec-from-file=<file>:: > > - Pathspec is passed in `<file>` instead of commandline args. If > > - `<file>` is exactly `-` then standard input is used. Pathspec > > - elements are separated by LF or CR/LF. Pathspec elements can be > > +`--overlay`:: > > +`--no-overlay`:: > > + In overlay mode, never remove files when restoring. In no-overlay mode, > > + remove tracked files that do not appear in the `--source` tree, to make > > + them match _<tree>_ exactly. The default is no-overlay mode. > > + > > +`--pathspec-from-file=<file>`:: > > + Pathspec is passed in _<file>_ instead of commandline args. If > > + _<file>_ is exactly `-` then standard input is used. Pathspec > > + elements are separated by _LF_ or _CR_/_LF_. Pathspec elements can be > > The reflowing of these paragraphs makes it a bit hard to see what > exactly is changing. > For the first set of options, there is a conversion to imperative mood, and I triggered a reflow. For the second one, it just appears that all lines are slightly changed by the new formatting. More generally, I kept reflowing the paragraphs to maintain the width, because this was done everywhere before, but I would strongly prefer having each sentence on separate line, regardless of its length. For asciidoc texts, this policy has some advantages: * keep a lexical entity as a single line, and get rid messed up diffs due to reflowing. * unfortunate automatic reflows can create errors, for instance when a part of text is sent to the beginning of line and is interpreted as a list item. JN
diff --git a/Documentation/git-restore.txt b/Documentation/git-restore.txt index 975825b44aa..541a39b5d28 100644 --- a/Documentation/git-restore.txt +++ b/Documentation/git-restore.txt @@ -7,10 +7,10 @@ git-restore - Restore working tree files SYNOPSIS -------- -[verse] -'git restore' [<options>] [--source=<tree>] [--staged] [--worktree] [--] <pathspec>... -'git restore' [<options>] [--source=<tree>] [--staged] [--worktree] --pathspec-from-file=<file> [--pathspec-file-nul] -'git restore' (-p|--patch) [<options>] [--source=<tree>] [--staged] [--worktree] [--] [<pathspec>...] +[synopsis] +git restore [<options>] [--source=<tree>] [--staged] [--worktree] [--] <pathspec>... +git restore [<options>] [--source=<tree>] [--staged] [--worktree] --pathspec-from-file=<file> [--pathspec-file-nul] +git restore (-p|--patch) [<options>] [--source=<tree>] [--staged] [--worktree] [--] [<pathspec>...] DESCRIPTION ----------- @@ -32,8 +32,8 @@ THIS COMMAND IS EXPERIMENTAL. THE BEHAVIOR MAY CHANGE. OPTIONS ------- --s <tree>:: ---source=<tree>:: +`-s <tree>`:: +`--source=<tree>`:: 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. @@ -41,79 +41,79 @@ OPTIONS If not specified, the contents are restored from `HEAD` if `--staged` is given, otherwise from the index. + -As a special case, you may use `"A...B"` as a shortcut for the -merge base of `A` and `B` if there is exactly one merge base. You can -leave out at most one of `A` and `B`, in which case it defaults to `HEAD`. +As a special case, you may use `"<refA>...<refB>"` as a shortcut for the +merge base of _<refA>_ and _<refB>_ if there is exactly one merge base. You can +leave out at most one of _<refA>__ and _<refB>_, in which case it defaults to `HEAD`. --p:: ---patch:: +`-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 + 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 prompt to restore all modified paths. --W:: ---worktree:: --S:: ---staged:: +`-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:: +`-q`:: +`--quiet`:: Quiet, suppress feedback messages. Implies `--no-progress`. ---progress:: ---no-progress:: +`--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`. ---ours:: ---theirs:: +`--ours`:: +`--theirs`:: When restoring files in the working tree from the index, use - stage #2 ('ours') or #3 ('theirs') for unmerged paths. + stage #2 (`ours`) or #3 (`theirs`) for unmerged paths. This option cannot be used when checking out paths from a tree-ish (i.e. with the `--source` option). + -Note that during `git rebase` and `git pull --rebase`, 'ours' and -'theirs' may appear swapped. See the explanation of the same options +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:: +`-m`:: +`--merge`:: When restoring files on the working tree from the index, recreate the conflicted merge in the unmerged paths. This option cannot be used when checking out paths from a tree-ish (i.e. with the `--source` option). ---conflict=<style>:: +`--conflict=<style>`:: The same as `--merge` option above, but changes the way the conflicting hunks are presented, overriding the `merge.conflictStyle` configuration variable. Possible values - are "merge" (default), "diff3", and "zdiff3". + are `merge` (default), `diff3`, and `zdiff3`. ---ignore-unmerged:: +`--ignore-unmerged`:: When restoring files on the working tree from the index, do not abort the operation if there are unmerged entries and neither `--ours`, `--theirs`, `--merge` or `--conflict` is specified. Unmerged paths on the working tree are left alone. ---ignore-skip-worktree-bits:: +`--ignore-skip-worktree-bits`:: In sparse checkout mode, the default is to only update entries - matched by `<pathspec>` and sparse patterns in - $GIT_DIR/info/sparse-checkout. This option ignores the sparse + matched by _<pathspec>_ and sparse patterns in + `$GIT_DIR/info/sparse-checkout`. This option ignores the sparse patterns and unconditionally restores any files in - `<pathspec>`. + _<pathspec>_. ---recurse-submodules:: ---no-recurse-submodules:: - If `<pathspec>` names an active submodule and the restore location +`--recurse-submodules`:: +`--no-recurse-submodules`:: + If _<pathspec>_ names an active submodule and the restore location includes the working tree, the submodule will only be updated if this option is given, in which case its working tree will be restored to the commit recorded in the superproject, and any local @@ -122,30 +122,29 @@ in linkgit:git-checkout[1] for details. not be updated. Just like linkgit:git-checkout[1], this will detach `HEAD` of the submodule. ---overlay:: ---no-overlay:: - In overlay mode, the command never removes files when - restoring. In no-overlay mode, tracked files that do not - appear in the `--source` tree are removed, to make them match - `<tree>` exactly. The default is no-overlay mode. - ---pathspec-from-file=<file>:: - Pathspec is passed in `<file>` instead of commandline args. If - `<file>` is exactly `-` then standard input is used. Pathspec - elements are separated by LF or CR/LF. Pathspec elements can be +`--overlay`:: +`--no-overlay`:: + In overlay mode, never remove files when restoring. In no-overlay mode, + remove tracked files that do not appear in the `--source` tree, to make + them match _<tree>_ exactly. The default is no-overlay mode. + +`--pathspec-from-file=<file>`:: + Pathspec is passed in _<file>_ instead of commandline args. If + _<file>_ is exactly `-` then standard input is used. Pathspec + elements are separated by _LF_ or _CR_/_LF_. Pathspec elements can be quoted as explained for the configuration variable `core.quotePath` (see linkgit:git-config[1]). See also `--pathspec-file-nul` and global `--literal-pathspecs`. ---pathspec-file-nul:: +`--pathspec-file-nul`:: Only meaningful with `--pathspec-from-file`. Pathspec elements are - separated with NUL character and all other characters are taken + separated with _NUL_ character and all other characters are taken literally (including newlines and quotes). -\--:: +`--`:: Do not interpret any more arguments as options. -<pathspec>...:: +`<pathspec>...`:: Limits the paths affected by the operation. + For more details, see the 'pathspec' entry in linkgit:gitglossary[7]. @@ -154,7 +153,7 @@ EXAMPLES -------- The following sequence switches to the `master` branch, reverts the -`Makefile` to two revisions back, deletes hello.c by mistake, and gets +`Makefile` to two revisions back, deletes `hello.c` by mistake, and gets it back from the index. ------------ @@ -165,7 +164,7 @@ $ git restore hello.c <2> ------------ <1> take a file out of another commit -<2> restore hello.c from the index +<2> restore `hello.c` from the index If you want to restore _all_ C source files to match the version in the index, you can say