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.

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

UI for specifying Git repo is very confusing #212

Open
talex5 opened this issue Jul 27, 2016 · 3 comments
Open

UI for specifying Git repo is very confusing #212

talex5 opened this issue Jul 27, 2016 · 3 comments

Comments

@talex5
Copy link
Contributor

talex5 commented Jul 27, 2016

Problems:

  1. If you run without arguments, it provides an in-memory-only store. This is only useful for debugging and should not be the default.
  2. If you point it at a bare Git repository, it quietly adds /.git to the path, and thus creates a new empty repository inside the one you wanted to use. We should probably require a --auto-create option or similar to create a new repository, not change the path the user gives, and warn if they give a path with a .git subdirectory.

(discovered while testing with @jdjakub)

@talex5
Copy link
Contributor Author

talex5 commented Feb 3, 2017

This requires a change to Irmin: @samoht will this be in the Irmin 1.0 API?

@samoht
Copy link
Member

samoht commented Feb 3, 2017

It could be added yes, is there an issue upstream?

@talex5
Copy link
Contributor Author

talex5 commented Feb 3, 2017

I've added mirage/irmin#398 for this.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
Development

No branches or pull requests

2 participants