[svk-devel] Review log message when push'ing to mirror

Ruslan Zakirov ruz at bestpractical.com
Wed Oct 17 10:26:02 EDT 2007


On 10/17/07, John Peacock <john.peacock at havurah-software.org> wrote:
> Ruslan Zakirov wrote:
> > As far as I understand 'svk sm -l //src //dst' is exactly what you
> > want. It merges everything at once and opens an editor for you to edit
> > merged log message. For those who don't want to edit log there is
> > option -m in smerge command that allow them to replace log with a
> > message or add heading to merged log. So we have four variants at
> > least:
> > 'svk sm -l //src //dst' - merged log, opens editor
> > 'svk sm -m 'xxx' //src //dst' - replace log with a message, no editor
> > 'svk sm -l -m 'xxx' //src //dst' - add a message before log, no editor
> > 'svk sm -l -m '' //src //dst' - just merge logs without an editor
> > And the last variant is what's 'svk push --lump' does.
>
> Ah, so it does!  That isn't well documented, then.  So I can revise my
> patch to the much simpler:
>
> === lib/SVK/Command/Push.pm
> ==================================================================
> --- lib/SVK/Command/Push.pm     (revision 2544)
> +++ lib/SVK/Command/Push.pm     (local)
> @@ -63,6 +63,7 @@
>        'S|sign'          => 'sign',
>        'P|patch=s'        => 'patch',
>        'verbatim'                => 'verbatim',
> +     'review-message'   => 'review_message'
>       );
>   }
>
> @@ -83,6 +84,11 @@
>           $self->{message} = '';
It even can be much simpler, just add "unless $self->{review_message}
" to the above statement.

>           delete $self->{incremental};
>       }
> +    elsif ( $self->{review_message} ) {
> +       $self->{lump} = 1;
> +       $self->{log}++;
> +       delete $self->{incremental};
> +    }
>       else {
>           $self->{incremental}++;
>       }
>
> Since it still much nicer for the user not to have to remember the two
> URL's involved...
>
> John
> _______________________________________________
> svk-devel mailing list
> svk-devel at bestpractical.com
> http://lists.bestpractical.com/cgi-bin/mailman/listinfo/svk-devel
>


-- 
Best regards, Ruslan.


More information about the svk-devel mailing list