showing shelf info with 'bzr status'

Parth Malwankar parth.malwankar at gmail.com
Sun Aug 22 10:25:25 BST 2010


Hello,

I was looking into bug #403687 [1] about showing shelved changes
with 'bzr status'. It seems like an important enough wishlist as I can
see 6 votes on it :-)

There is a merge proposal in place[2]. This proposal basically
updates 'bzr status' to include the list of shelves in output.

Sample output with the current patch:

[bzr-grep]% ~/src/bzr.dev/403687-st-shows-shelve/bzr st -S
 M INSTALL
? x
 S 1: <no message>

[bzr-grep]% ~/src/bzr.dev/403687-st-shows-shelve/bzr st
modified:
  INSTALL
unknown:
  x
shelves:
  1: <no message>

While the original report talks about showing per-file shelf information,
just listing the shelves (or just indicating N shelves exist) seems like a
reasonable fix to me that doesn't require too much change.

Vincent suggested that it may be good to discuss this change on the
list as its a UI change. Currently, an S is used in the second column
to indicate a shelf. The report and Vincents comment on the mp also
suggests that it may be better to use a one line output, something
like 'N shelves exist.' instead of listing the shelves. That seems fine
to me. The user can then use 'bzr shelve --list' for actually viewing
the shelves.

I would like to hear your thoughts and suggestions regarding this
change.

Regards,
Parth

[1] https://bugs.launchpad.net/bzr/+bug/403687
[2] https://code.launchpad.net/~parthm/bzr/403687-status-shows-shelves/+merge/31182



More information about the bazaar mailing list