[with patch] bzr status doesn't give the file type for renamed files.

OOps, small bug in my patch itself. The last assignment on kind_str should be "kind_str = ''".

Below is the updated patch:

=== modified file 'bzrlib/delta.py'
--- bzrlib/delta.py
+++ bzrlib/delta.py
@@ -120,10 +120,19 @@
                  text_modified, meta_modified) in self.renamed:
                 if meta_modified:
                     newpath += '*'
+ if kind == 'directory':
+ kind_str = '/'
+ elif kind == 'symlink':
+ kind_str = '@'
+ else:
+ kind_str = ''
+
                 if show_ids:
- print >>to_file, ' %s => %s %s' % (oldpath, newpath, fid)
+ print >>to_file, ' %s%s => %s%s %s' % \
+ (oldpath, kind_str, newpath, kind_str, fid)
                 else:
- print >>to_file, ' %s => %s' % (oldpath, newpath)
+ print >>to_file, ' %s%s => %s%s' % \
+ (oldpath, kind_str, newpath, kind_str)

         if self.modified:
             print >>to_file, 'modified:'

You might find entry.kind_character() useful.

Probably. I've just done the same as what was already done for show_list(files) above in the same file. Then both should use entry.kind_character().

I think the kind_character() for a symlink is '', whereas show_list(files) uses '@' (my patch too). That's inconsistant.

I'd much rather we reduce inconsistency rather than add another manual-if block which can then get out of sync with other code.

See also bug #40507,

The current bzr.dev status code never shows kind characters.
The only code path that displays kind characters is 'bzr ls' when in verbose mode.
Because it is consistent, I'm closing this bug. If we want to bring them back, we should probably open a new bug.