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

Ruslan Zakirov ruz at bestpractical.com
Wed Oct 17 11:28:03 EDT 2007


I really think that it should work in the following way:
svk push --lump
svk push --lump --review-message
svk push --patch --review-message
svk push --review-message - makes not much sense and should issue
warning or even die with error message.
Yes, it's more typing, but flexible. Works as modifier instead of new
sub-command.


On 10/17/07, John Peacock <john.peacock at havurah-software.org> wrote:
> Ruslan Zakirov wrote:
> >> @@ -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;
>
> Except that it doesn't make much sense to have --review-message if lump
> isn't active, and I don't think we would want to require --lump on the
> commandline independent of this flag.
>
> 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