Add 'bugs' property to xmllog?

[Guillermo Gonzalez]

Hi!
This is cool!
can you propose a branch to be merged in bzr-xmloutput?
Also if you can please add a test for this, or ping me and I'll try to do
it.

Thanks!

On Thu, Dec 17, 2009 at 6:51 AM, Frits Jalvingh <jal at etc.to> wrote:

> Hi Jelmer,
>
> Thanks for your response. I modified the code to report the structure as
> follows:
> <bugs><bug><url>http://bugzilla.etc.to/show_bug.cgi?id=12345
> </url><status>fixed</status></bug></bugs>
> where the "bug" tag gets repeated for every line in the bugs property.
> Sadly enough I cannot properly split this into a bug tracker ID and a
> bug number proper because only the URL is stored, not the components it
> was created from. And properly decoding the URL into an ID and a tracker
> key is not supported by TrackerRegistry.
>
> The new diff is as follows:
> === modified file 'logxml.py'
> --- logxml.py   2009-07-02 16:12:44 +0000
> +++ logxml.py   2009-12-17 09:46:25 +0000
> @@ -139,6 +139,26 @@
>              for tag in revision.tags:
>                 self.to_file.write('<tag>%s</tag>' % tag)
>             self.to_file.write('</tags>')
> +        # jal at etc.to expose 'bugs' property
> +        if revision.rev.properties and 'bugs' in
> revision.rev.properties:
> +            bugs = revision.rev.properties['bugs']
> +            if len(bugs) > 0:
> +                self.to_file.write('<bugs>')
> +                bugar = bugs.split('\n')
> +                for bug in bugar:
> +                    if len(bug) > 0:
> +                        self.to_file.write('<bug>')
> +                        lsp = bug.rfind(' ')
> +                        if lsp == -1:
> +                            url = bug
> +                            status = ''
> +                        else:
> +                            url = bug[0:lsp].rstrip()
> +                            status = bug[lsp+1:].strip()
> +                        self.to_file.write('<url>%s</url>' % url)
> +                        self.to_file.write('<status>%s</status>' %
> status)
> +                        self.to_file.write('</bug>')
> +                self.to_file.write('</bugs>')
>          if self.show_ids:
>             self.to_file.write('<revisionid>%s</revisionid>' %
>                                 revision.rev.revision_id)
>
>
> On Sun, 2009-12-06 at 13:41 +0100, Jelmer Vernooij wrote:
> > Hi Frits,
> >
> > On Sun, 2009-12-06 at 12:11 +0100, Frits Jalvingh wrote:
> > > I want to be able to access the 'bugs' property on all revisions that
> were
> > > committed with a --fixes option. (bla bla bla)
>
> > This seems like a good idea. I think it would be nice if we could
> > provide the contents of the bugs property in XML as well rather than
> > just providing the plain contents, to prevent the user from having to
> > implement a parser for the bugs property themselves.
> >
> > Cheers,
> >
> > Jelmer
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: https://lists.ubuntu.com/archives/bazaar/attachments/20091217/ce998372/attachment.htm