Hi,

On Fri, Jul 16, 2010 at 11:28:44AM +0200, Bernhard R. Link wrote:
> * Osamu Aoki <[email protected]> [100715 18:24]:
> > > > (Beside its commands are too long to type ...)
> > >
> > > That's always a dilemma. Things being too short makes things cryptic,
> > > things to long make it hard to type. I already have some slightly
> > > shorter aliases and perhaps adding very short aliases makes sense,
> > > but I guess it is better to have the speaking command names in the
> > > documentation.
> >
> > I understand.
> 
> I've added some aliases. Comments and suggestions more than welcome.

I saw them in git repo.  With these alias and typing "git dpm ..." helps
to reduce typing :-)
 
> > By the way, this automatic branch switching caused me quite a bit of 
> > confusion.
> > As I read the following: 
> > [...]
> > What do you think.
> 
> I've added some more stuff there. Could you take a look and tell if you
> think it helps?
> 
> http://git.debian.org/?p=git-dpm/git-dpm.git;a=blob_plain;f=git-dpm.1;hb=HEAD

I think you have addressed all the things which I mentioned.

Few other things I can think of.
 * I wish to have "git-import-dsc" like function. 
   (command or example, it will be noce to have dpatch support)
 * I wish to help people do not read the documentation :-)

In order to reduce surprize, it is best to provide verbose outputs for debug.

Can you think of changing like: 

function gitcmd() {
        "$GIT" "$@"
        if [ "$1" = "add" ] \
        || [ "$1" = "branch" ] \
        || [ "$1" = "checkout" ] \
        || [ "$1" = "cherry-pick" ] \
        || [ "$1" = "merge" ] \
        || [ "$1" = "rebase" ] \
        || [ "$1" = "reset" ] \
        || [ "$1" = "rm" ] \
        || [ "$1" = "tag" ]; then
                debugout "git-dpm: git $@"
        fi
}
function xargsgitcmd() {
        xargs --no-run-if-empty "$GIT" "$@"
        if [ "$1" = "add" ] \
        || [ "$1" = "branch" ] \
        || [ "$1" = "checkout" ] \
        || [ "$1" = "cherry-pick" ] \
        || [ "$1" = "merge" ] \
        || [ "$1" = "rebase" ] \
        || [ "$1" = "reset" ] \
        || [ "$1" = "rm" ] \
        || [ "$1" = "tag" ]; then
                debugout "git-dpm: xargs --no-run-if-empty git $@"
        fi
}

--debug option really helps usability.  (Somehow, I am getting echo even
without it... here.  I will be reinstalling :-)

Thanks

Osamu




-- 
To UNSUBSCRIBE, email to [email protected]
with a subject of "unsubscribe". Trouble? Contact [email protected]

Reply via email to