mirrors/git-annex
2
0
Fork 0
Code Issues Pull requests Projects Releases Packages 1 Wiki Activity Actions

comment

Browse source
This commit is contained in:
Joey Hess 2021-04-20 11:57:10 -04:00
parent fee6504388
commit 752c389849
No known key found for this signature in database
GPG key ID: DB12DB0FF05F8F38
1 changed files with 30 additions and 0 deletions
Download patch file Download diff file Expand all files Collapse all files

30
doc/bugs/poor_choice_of_name_for_adjusted_branches/comment_1_adbbc90054eafefa7de58663868986a3._comment Normal file
View file

@ -0,0 +1,30 @@
[[!comment format=mdwn
username="joey"
subject="""comment 1"""
date="2021-04-20T15:43:03Z"
content="""
One of the constraints on naming these is that the name needs to not be one
that is likely to get in the way of something the user is doing with
other branches, eg conflicting with another branch they have for
something unrelated to git-annex.
Another constraint is that the name of the branch (either the whole thing
or sometimes just the part after the slash) often appears in the
user's prompt and so it would be good if it were reasonably short and also
reasonably human readable and clear about what thing is checked out.
Those constraints are what led to this name choice. I am well aware there
are reasons people won't typically use () in branch names, which along with
the "adjusted/", helps make a naming conflict unlikely. And it avoids needing
to also put "git-annex" in the branch name to avoid conflicts, so keeps it
short, and the parens imply a relationship to master rather clearly.
I decided that needing to properly quote a shell parameter in the (somewhat
uncommon) case of manually checking the branch out was a reasonable
tradeoff.
It does seem like a bug in bash completion that it doesn't tab complete
this correctly. I notice that tab completing a similar filename does escape
the parens, so it may be that the bug can be fixed in the git completion
file somehow.
"""]]