Zenaan Harkness <zen@xxxxxxxxxxxx> writes: > I needed this quite a bit in the last few days, basic script but > serves my need. I think it would be useful for other beginners if in > $git/contrib/ source dir. > > Just a start to a basic script. Needs more tests etc, but it's enough > to get newbies (like me) off to a reasonable start. Handles multiple > input dirs. > > PLEASE CC me, as I am not subscribed. > > (some SMTP server rejected attachment, so pasting below instead) > > Thanks, > Zenaan > > > #!/bin/bash I do not think you need (nor used) any bash-ism in this script. Saying "#!/bin/sh" here is cleaner. > > # Change one or more normal repos into bare repos: > # See also https://git.wiki.kernel.org/index.php/GitFaq#How_do_I_make_existing_non-bare_repository_bare.3F > > for i in "$@"; do for i do You do not have to say 'in "$@"'; it is implied. > echo; echo "----------------------" > echo Processing $i Forgot to dq? > > repo="$i" > repo="`basename $i`" > tmp_repo="${repo}.git" > # Insert here: may be exit if any spaces in repo fqn > # Insert here: check for non-existent repo/.git dir > # Insert here: check that we are not inside the repo > # Insert here: add exit/do-nothing if fail to mv dirs etc > > mv $repo/.git $tmp_repo Forgot to dq? i.e. mv "$repo/.git" "$tmp_repo" The same for all the variable references in the remainder of the script. More importantly, "mv" would fail if $repo is given as a full pathname elsewhere in the filesystem that is different from your current directory where you create $tmp_repo. > git --git-dir=$tmp_repo config core.bare true > mv $repo ${repo}.bak > mv $tmp_repo $repo > done -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html