Skip to content

Conversation

@bk2204
Copy link
Member

@bk2204 bk2204 commented Nov 2, 2022

Right now, people have to read the FAQ on the wiki, which makes it hard to discover or to use offline. We want people to have a copy of it installed as part of the documentation for ease of discovery. Add it to the repository, and add a couple of missing entries to help explain some situations we haven't mentioned yet.

Fixes #5034

@bk2204 bk2204 requested a review from a team as a code owner November 2, 2022 20:36
chrisd8088
chrisd8088 previously approved these changes Nov 2, 2022
Copy link
Member

@chrisd8088 chrisd8088 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great! I noted some minor typos only.

To fix this, you can use `git lfs migrate import --everything` with an
appropriate `--include` argument. For example, if you wanted to move your
`.jpg` and `.png` files into Git LFS, you can do that with
`git lfs migrate import --everything --include="\*.jpg,*.png"`. More complicated
Copy link
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Just out of interest, is there a difference here between \* and plain *?

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

This is required to make AsciiDoc not consider this bold. Very fortunately, it's also syntactically correct according to the shell.

@chrisd8088 chrisd8088 dismissed their stale review November 6, 2022 09:34

I realized we should put this in section 7 of the manual.

@chrisd8088 chrisd8088 self-requested a review November 6, 2022 09:34
Copy link
Member

@chrisd8088 chrisd8088 left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Looks great, as I said -- but we should maybe change this to install in man7 and be git-lfs-faq(7).

That will also mean tweaking a couple of extra Makefile lines, plus some bits for the Debian package (here in git-lfs.manpages) and rpm package (here and here in SPECS/git-lfs.spec, I think).

We generally get a lot of requests for problems that are already
answered in the FAQ.  Right now, this lives in the wiki, but this is
hard to find, so let's include it in the documentation with Git LFS
itself.

Add the relevant portions to the binary and Makefile so that it can be
accessed using `git lfs help faq`, and add it as well to the Linux
packages.
Explain what a pointer file is and how to check out files with `git lfs
pull`.
It's been requested that we add documentation to explain how to track
files that already exist.  Do so in a FAQ entry.
Now that the FAQ is stored in the repository, let's point people there
instead of at the wiki.
@bk2204 bk2204 merged commit be47d92 into git-lfs:main Nov 9, 2022
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

Add info on how to start tracking files already in git

2 participants