Need launchpad integration in non-installed packages

[Philip Peitsch]

Attempting to click any of the launchpad integration menu items results in the following error being displayed in the running terminal:

Creating project directory test-project
Creating bzr repository and commiting
Launching your newly created project!le 0/1
Traceback (most recent call last):
  File "<string>", line 5, in <module>
  File "/usr/share/launchpad-integration/launchpadintegration/main.py", line 48, in main
    pkginfo = PackageInfo.fromPackageName(options.package, logger)
  File "/usr/share/launchpad-integration/launchpadintegration/packageinfo.py", line 216, in fromPackageName
    '"%s"' % package)
launchpadintegration.packageinfo.PackageNotFoundError: Could not look up package info for "test-project"

This problem only occurs during development, and vanishes once the package is installed (obviously)

[Philip Peitsch (philip-peitsch)]

Copied from Bug #534947:

The problem seems quite deeply rooted in how launchpad-integration works. Currently, lpi supports:

* fromProcessID(options.pid, logger)
* fromFilename(options.filename, logger)
* fromPackageName(options.package, logger)
* fromXID(options.xid, logger)

It uses these options to look up the relevant information from a package. Basically, another option needs to be added somehow to allow overriding the lookup functionality to use a hardcoded control file for a particular package.

I've raised a bug upstream for this and will spend some time addressing that. That is the required functionality to address both this issue, and the associated crash as seen when attempting to launch the report a bug feature prior to installing the package under development.

[Philip Peitsch (philip-peitsch)]

I've raised Bug #534968 to track progress in the launchpad integration library

[Philip Peitsch (philip-peitsch)]

I've proposed a patch for launchpad-integration that allows the search file to be overridden using an environment variable. See https://code.launchpad.net/~philip-peitsch/launchpad-integration/allow-info-file-override/+merge/21036

[Didier Roche-Tolomelli (didrocks)]

Sweet! I saw your merge proposal to LP. Maybe, we should ping LP dev on #launchpad-dev on freenode to activate them?

PS: it's possible to add bug task to a bug report to have both a quickly and launchpad-lpi task (and so, we can track that easier :)). You'll know it for next time.

[Philip Peitsch (philip-peitsch)]

If you can find a way of getting things moving that'd be awesome :). I've proposed this patch as well, but it shouldn't be merged until lpi accepts the environment override. If they decide they want to integrate this in a different fashion, this patch will required rewriting (of course). I've dropped by #launchpad-dev but couldn't get any response... though I suspect that is a timezone thing :). If you can push it a bit as well that'd be awesome as then we can get this in sooner!

[Didier Roche-Tolomelli (didrocks)]

Sure, I'll try to do that on Friday (I'm more or less offline until Friday as I'll be at a FLOSS event).

Doing that Friday so ;)

[Didier Roche-Tolomelli (didrocks)]

putting that in progress until we have a proper fix (see above discussion)

[Philip Peitsch (philip-peitsch)]

This is still blocked awaiting some activity on Bug #534968. The bug and associated patch have been waiting since March 2010, so I'm wiping this off my plate.