[Top][All Lists]

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

Re: lilycontrib.tcl, was Re: the "r" in "git pull -r"

From: Johannes Schindelin
Subject: Re: lilycontrib.tcl, was Re: the "r" in "git pull -r"
Date: Tue, 11 Aug 2009 16:06:18 +0200 (CEST)
User-agent: Alpine 1.00 (DEB 882 2007-12-20)


On Tue, 11 Aug 2009, Maximilian Albert wrote:

> > I pushed a new version to git://  You 
> > can download it directly here:
> >
> >;f=lilycontrib.tcl;hb=lilycontrib
> Cool, works like a charm now. Thanks a lot!

I actually had to change the --work-tree things, since it is utter 
garbage.  I _know_ why I was opposed to its inclusion.

> One minor comment: For new contributors with no experience in git (or 
> version control in general) it may seem as though nothing happens after 
> pressing the button. Thus I'd suggest to add two messages to indicate 
> that the process was started, along the lines of:
> ==>
> proc update_lilypond {} {
>       global lily_dir
>       if {![file exists $lily_dir]} {
>               write_to_output "Starting to clone LilyPond repository (this
> can take some time) ...\n"
>               file mkdir $lily_dir
>               git init
>               git config core.bare false
>               git remote add -t master \
>                       origin git://
>               git fetch --depth 1
>               git reset --hard origin/master
>               git config branch.master.remote origin
>               git config branch.master.merge refs/heads/master
>               .update configure -text "Update LilyPond"
>       } else {
>               write_to_output "Starting to update LilyPond repository ...\n"
>               git fetch origin
>               git merge origin/master
>       }
>       write_to_output "Done.\n"
> }
> <==

I added two shorter notices, and I also set the busy cursor now.


reply via email to

[Prev in Thread] Current Thread [Next in Thread]