RFE: Packages of projects/distros

TEstscript:

#!/usr/bin/python

import launchpadbugs.connector as Connector
from launchpadbugs.basebuglistfilter import URLBugListFilter
import launchpadbugs.html_bug, launchpadbugs.html_projectlist

def counter_hook(counter,block_size,size):
    print "Counter %d %d %d " % (counter, block_size, size)

projectList=Connector.ConnectProjectPackageList(method="html")
projectList.set_progress_hook(counter_hook,300)
f=URLBugListFilter()
f.add_option("batch",300)
packagelist=projectList(f("http://launchpad.net/leonov/+allpackages"))

for package in packagelist:
  print "Package Name: %s -> Package URL: %s" % (package.package_name, package.package_url)

I did not make a close code review yet, but on the very first view two questions came in my mind:

1.) Naming: Why did you choose 'ConnectProjectPackageList'? - projects can not have packages. So in my opinion it should either be ConnectDistroPackageList or ConnectPackageList

2.) In the example above it should be:
packagelist = projectList(f("https://edge.launchpad.net/ubuntu/+allpackages"))
right?

I will look at the code tomorrow morning.
Thanks again for your work,
Markus

Hi Markus,

2) Yes :) But you can also insert /debian/+allpackages (0 packages)
or a simple project, which will raise the described exception (which is correct)

1) Well, A Distro is also listed under projects, so I think it's quite correct, to call it ProjectPackageList, because I think we will have in the future, projects which could be distros (i.e. derivatives of ubuntu).

More on IRC :)

\sh

Thanks, Good work Stephan.
merged and pushed
------------------------------------------------------------
revno: 137
committer: Markus Korn <email address hidden>
branch nick: main
timestamp: Fri 2008-07-18 10:09:25 +0200
message:
  Stephan Hermann added support package lists, thanks Stephan
  (LP: #249461)
    ------------------------------------------------------------