[MERGE] add shelve --destroy

James Westby jw+debian at jameswestby.net
Thu Mar 12 01:39:22 GMT 2009


On Thu, 2009-03-12 at 11:26 +1000, Aaron Bentley wrote:
> -----BEGIN PGP SIGNED MESSAGE-----
> Hash: SHA1
> 
> Hi all,
> 
> This patch adds the option of destructive shelving, by doing shelve
> - --destroy.  This is equivalent to bzr shelve && bzr unshelve --delete.
> This was requested by an old colleague of mine (Hi Maru!).
> 
> Obviously, there's potential for foot-shooting, but I hope that the
> option name is sufficiently scary that people using it will understand
> the risk.

What do you think about making this "revert --interactive"? I realise it
would be more plumbing, but to me it is closer to the meaning, and
moving it to "revert" would make it clearer that changes were being
destroyed?

> +                    if self.destroy:
> +                        creator.transform()
> +                    else:
> +                        shelf_id =
> self.manager.shelve_changes(creator,
> +
> self.message)
> +                        trace.note('Changes shelved with id "%d".' %
> shelf_id)

Should there be a trace.note on the destroy case as well?

>                  if (self.auto_apply or self.prompt_bool(
>                      'Shelve %d change(s)?' % changes_shelved)):

Should the messages be changed for --destroy? I would say yes if it
was "revert --interactive", here I'm not so sure, "Destroy %d change(s)"
would be pretty clear I think.

Thanks,

James




More information about the bazaar mailing list