Parse feedback page for branch information

[Daniel Holbach]

<ajmitch> in a few cases the submitter has a branch already
 it'd be cool to parse the initial comment & look for a lp: ;)

[Andrew Mitchell (ajmitch)]

This case also comes up when the ARB has asked for source after they've initially submitted a binary package.

[Daniel Holbach (dholbach)]

Maybe we can try to come up with some regexes or a priority of where first to look?

[Daniel Holbach (dholbach)]

Can somebody please help me testing a bunch of myApps pages, so we can figure out if the regexes in http://bazaar.launchpad.net/~dholbach/apps-brancher/1012064/revision/52 are correct? Just throw some HTML from myApps at them and see what they spit out for you.

[Andrew Starr-Bochicchio (andrewsomething)]

In find_ppa_info(), I'd expect to see ""ppa:%s/%s" % (slug_regex, slug_regex)" on line 95 not "lp:%s/%s" % (slug_regex, slug_regex)"

I tried to give this a test, but it's not working for me:

>>> from brancher import web; import urllib
>>>
>>> html=urllib.urlopen("https://myapps.developer.ubuntu.com/dev/apps/500/feedback/")
>>> web.find_ppa_info(html)
Traceback (most recent call last):
  File "<stdin>", line 1, in <module>
  File "brancher/web.py", line 98, in find_ppa_info
    ppas = re.findall("(%s)" % "|".join(ppa_regexes), html)
  File "/usr/lib/python2.7/re.py", line 177, in findall
    return _compile(pattern, flags).findall(string)
TypeError: expected string or buffer

[Daniel Holbach (dholbach)]

On 22.06.2012 16:37, Andrew Starr-Bochicchio wrote:
> In find_ppa_info(), I'd expect to see ""ppa:%s/%s" % (slug_regex,
> slug_regex)" on line 95 not "lp:%s/%s" % (slug_regex, slug_regex)"

I'll have a look into it. Thanks.

> I tried to give this a test, but it's not working for me:
>
>>>> from brancher import web; import urllib
>>>>
>>>> html=urllib.urlopen("https://myapps.developer.ubuntu.com/dev/apps/500/feedback/")

Oops, this is missing a .read() a the end.

Have a great day,
 Daniel