Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Remove reliance on mount process from read-object hook #6

Closed
jrbriggs opened this issue Aug 8, 2019 · 4 comments · Fixed by #122
Closed

Remove reliance on mount process from read-object hook #6

jrbriggs opened this issue Aug 8, 2019 · 4 comments · Fixed by #122
Assignees
@derrickstolee @jeffhostetler

Comments

@jrbriggs
Copy link
Member

jrbriggs commented Aug 8, 2019 •
edited
Loading

Rather than sending a message to the mount process, investigate simply acquiring the objects inline.
@github-actions
Copy link

Labeling this issue as stale. There has been no activity for 30 days. Remove stale label or comment or this issue will be closed in 7 days.

@github-actions github-actions bot added the no-activity Stale issue or PR that will be automatically closed soon. label Aug 16, 2019
@derrickstolee derrickstolee removed the no-activity Stale issue or PR that will be automatically closed soon. label Aug 16, 2019
@github-actions
Copy link

Labeling this issue as stale. There has been no activity for 30 days. Remove stale label or comment or this issue will be closed in 7 days.

@github-actions github-actions bot added the no-activity Stale issue or PR that will be automatically closed soon. label Aug 17, 2019
@jrbriggs jrbriggs removed the no-activity Stale issue or PR that will be automatically closed soon. label Aug 17, 2019
@github-actions
Copy link

Labeling this issue as stale. There has been no activity for 30 days. Remove stale label or comment or this issue will be closed in 7 days.

@github-actions github-actions bot added the no-activity Stale issue or PR that will be automatically closed soon. label Aug 19, 2019
@jrbriggs jrbriggs removed the no-activity Stale issue or PR that will be automatically closed soon. label Aug 19, 2019
@jeffhostetler
Copy link
Contributor

We should be able to use the new git-gvfs-helper (with a yet to be written background process mode) to fetch individual items in series (and with only 1 cred/auth setup).

This was referenced Sep 16, 2019
Closed
Closed
Closed
@derrickstolee derrickstolee mentioned this issue Oct 3, 2019
Merged
derrickstolee added a commit that referenced this issue Oct 4, 2019
@derrickstolee
Merge pull request #122: Use GVFS Helper transport instead of read ob…
a8b0b08 
…ject hook

See microsoft/git#191 for the Git code regarding the GVFS transport layer. This can work in tandem with the read-object hook, but it _should_ make the read-object hook irrelevant.

* Delete all references to the read-object hook in the product code. This unblocks #4 and the [Mount Removal] tasks #15,  #132, #133, #135, #136, and possible others.

* Delete the Sparse Verb in favor of `git sparse-checkout set`.

* Delete the `BlobPrefetcher` and all references to `--files` or `--folders` prefetching.

Resolves #6, #7, #36.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@derrickstolee derrickstolee

@jeffhostetler jeffhostetler

Labels
None yet
Projects
None yet
Milestone
No milestone
Development

Successfully merging a pull request may close this issue.

Use GVFS Helper transport instead of read object hook
3 participants
@derrickstolee @jrbriggs @jeffhostetler