Zenaan Harkness <z...@freedbms.net> 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 majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html

Reply via email to