Merge lp:~brian-murray/ubuntu-dev-tools/fix-params into lp:~ubuntu-dev/ubuntu-dev-tools/trunk

[Brian Murray]

Instead of this:

seeded-in-ubuntu: Error: The package 'update-manager-core' does not exist in the Ubuntu primary archive in quantal, quantal-security, quantal-updates or quantal-proposed

Report this:

seeded-in-ubuntu: Error: The source package 'update-manage-core' does not exist in the Ubuntu primary archive in quantal, quantal-security, quantal-updates or quantal-proposed

This is useful for figuring out that I was dumb and wanted to use the -b switch.

For some reason name_key was set to name which didn't really make sense to me.

[Stefano Rivera (stefanor)]

I think you are misunderstanding the purpose of name_key. It's the name of the named parameter that will be the source/binary package name.

I don't see what this patch has to do with making seeded-in-ubuntu say "source package", either. That was already fixed in LP: #1029155

review: Disapprove

[Brian Murray (brian-murray)]

I was looking at lines 371+, http://bazaar.launchpad.net/~ubuntu-dev/ubuntu-dev-tools/trunk/view/head:/ubuntutools/lp/lpapicache.py#L371, which show the following:

            if latest is None:
                if name_key == 'binary_name':
                    package_type = "binary package"
                elif name_key == 'source_name':
                    package_type = "source package"
                else:
                    package_type = "package"

Additionally, there is a named_key and name parameter passed to _getPublishedItem. seeded-in-ubuntu calls getSourcePackage which then calls _getPublishedItem with name_key='source_name' so I'm really pretty sure name_key should set to name_key in the parameters and not name.

[Stefano Rivera (stefanor)]

That's name_key, not params[name_key]. params is only used on line 361.

Unmerged revisions

1347. By Brian Murray on 2012-09-21

don't set name_key to the package name

=== modified file 'ubuntutools/lp/lpapicache.py'
--- ubuntutools/lp/lpapicache.py	2012-07-26 19:40:49 +0000
+++ ubuntutools/lp/lpapicache.py	2012-09-21 18:17:19 +0000
@@ -350,7 +350,7 @@
         index = (name, series.name, pockets)
         if index not in cache:
             params = {
-                name_key: name,
+                name_key: name_key,
                 'distro_series': series(),
                 'status': 'Published',
                 'exact_match': True,