[MERGE] Two little fixes/cleanups

[Wouter van Heyst]

On Sat, Dec 10, 2005 at 06:27:45PM -0600, John Arbash Meinel wrote:
> Wouter van Heyst wrote:
> > On Sat, Dec 10, 2005 at 04:13:16PM -0600, Michael Ellerman wrote:
> 
> ...
> 
> >>>    bzr.dev> ./bzr cat
> >>>    bzr: ERROR: exceptions.KeyError: 'changeset'
> >>>      at /home/larstiq/src/bzr.dev/bzrlib/commands.py line 207
> >>>      in options
> >>
> >>You don't? I hit this when one of my old plugins skewed with bzr such that the 
> >>option it wanted wasn't there anymore - it took me a couple of minutes to 
> >>work that out though. Having the message at least tell me it was looking for 
> >>an option would have helped I think.
> > 
> > 
> > Oh yes, I agree with noting it is an option, I should have been more
> > explicit. The semi-traceback does more for me knowing where to look then
> > the 'Internal check failed'. Of course, by the time I had figured out
> > what your patch was supposed to do, I had been mucking around on line
> > 207 already.
> > 
> > If there could be information on where the offending option comes from,
> > that would be great.
> 
> I think in general tracebacks are intended to be hidden in ~/.bzr.log,
> and not shown on the screen.

I can understand that, Michael's patch tells us it is an option, and
even it's name, but not _where_ this option is wrongly defined, so I
would still be miffed when this happened due to version skew.

Wouter van Heyst