Updated tags support

[John A Meinel]

Gustavo Niemeyer wrote:
> Hey folks,
> 
> I've published a branch at http://bzr.labix.org/bzr.tags with
> tags support updated to the current codebase. I've also moved
> the workingtree related methods to the WorkingTree class.
> 
> Have fun!
> 

Thanks for keeping this up to date.

Aren't tags a Branch property, not a WorkingTree property, though?

I'm just thinking that if I have a branch without a working tree, it
should still have tags (so that I can 'bzr branch -r tag:release-1.0
http://bazaar-ng.org/bzr/bzr.dev')

The only reason (right now), that tags would be part of WorkingTree, is
because you have to 'commit' them in order for them to show up.

So at best, the uncommitted pending-actions stuff would be part of
WorkingTree, but the actual stored tags should be in the repository.

Also, you need to create a more complete test case. I was able to break
it with:

$ ./bzr set-tag -r -1 previous
$ ./bzr set-tag -r before:tag:previous something-earlier
bzr: ERROR: exceptions.AttributeError: 'BzrBranch' object has no
attribute 'get_tags'
  at
/local/01/system/srv/bzr/public/mirrors/niemeyer/bzr.tags/bzrlib/revisionspec.py
line 246
  in _match_on

And then when I tried to commit, I got:
$ ./bzr commit -m "Adding some tags"
bzr: ERROR: Revision
{gustavo at niemeyer.net-20051223003244-a442517770a310da} not present in
Weave('tags')

John
=:->

-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 249 bytes
Desc: OpenPGP digital signature
Url : https://lists.ubuntu.com/archives/bazaar/attachments/20051222/5f31c05f/attachment.pgp