request-sync insists on using LP lib even if I don't use --lp switch !

Bug #416955 reported by أحمد المحمودي (Ahmed El-Mahmoudy)
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Low
Michael Bienia

Bug Description

Binary package hint: ubuntu-dev-tools

Hello,

  When I ran "requestsync -s irssi-scripts karmic" today (that's my first time to run requestsync after upgrading to karmic). I get the following error:

$ requestsync -s irssi-scripts karmic
Traceback (most recent call last):
  File "/usr/bin/requestsync", line 487, in <module>
    if not newsource: (cur_ver, component) = cur_version_component(srcpkg, release)
  File "/usr/bin/requestsync", line 124, in cur_version_component
    src = LpApiWrapper.getUbuntuSourcePackage(sourcepkg, release)
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/lpapiwrapper.py", line 153, in getUbuntuSourcePackage
    series = cls.getUbuntuSeries(str(series))
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/lpapiwrapper.py", line 112, in getUbuntuSeries
    series = cls.getUbuntuDistribution().getSeries(name_or_version = name_or_version)
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/lpapiwrapper.py", line 90, in getUbuntuDistribution
    cls._ubuntu = Launchpad.distributions['ubuntu']
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/lpapiwrapper.py", line 48, in __getattr__
    self.login()
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/lpapiwrapper.py", line 43, in login
    self.__lp = libsupport.get_launchpad('ubuntu-dev-tools')
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/libsupport.py", line 78, in get_launchpad
    credentials = get_credentials(consumer, cred_file, level)
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/libsupport.py", line 74, in get_credentials
    return find_credentials(consumer, files, level)
  File "/usr/lib/python2.6/dist-packages/ubuntutools/lp/libsupport.py", line 56, in find_credentials
    "one for this consumer." % consumer)
IOError: No credentials found for 'ubuntu-dev-tools', please see the manage-credentials manpage for help on how to create one for this consumer.

As I can see, requestsync is trying to use the LP lib python bindings although I did not pass --lp switch to requestsync.

ProblemType: Bug
Architecture: i386
Date: Fri Aug 21 16:41:06 2009
DistroRelease: Ubuntu 9.10
NonfreeKernelModules: wl
Package: ubuntu-dev-tools 0.75
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
ProcVersionSignature: Ubuntu 2.6.31-6.25-generic
SourcePackage: ubuntu-dev-tools
Uname: Linux 2.6.31-6-generic i686

Revision history for this message
أحمد المحمودي (Ahmed El-Mahmoudy) (aelmahmoudy) wrote :
Revision history for this message
Jonathan Davies (jpds) wrote :

We now only use the LP API to file sync requests.

Revision history for this message
Michael Bienia (geser) wrote :

See also bug 406659. I'm working on making requestsync not use the LP API in "mail" mode (i.e. without the --lp option).

Changed in ubuntu-dev-tools (Ubuntu):
status: New → In Progress
importance: Undecided → Low
assignee: nobody → Michael Bienia (geser)
Revision history for this message
Michael Bienia (geser) wrote :

Fix committed to trunk (r484).

Changed in ubuntu-dev-tools (Ubuntu):
status: In Progress → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package ubuntu-dev-tools - 0.76

---------------
ubuntu-dev-tools (0.76) karmic; urgency=low

  [ Nicolas Valcárcel ]
  * mk-sbuild-lv:
    - Add check for built-in dm_snapshot (LP: #398414)

  [ Andreas Moog ]
  * update-maintainer:
    - Don't silently fail when Maintainer-Field contains a comment
      in brackets. (LP: #397144)
    - Don't add second XSBC-Original-Maintainer if Maintainer was set
      to Motu or Core-Dev.

  [ Michael Bienia ]
  * Drop python-launchpad-bugs from Depends.
  * Add python-lazr-restfulclient to Depends.
  * buildd: Add a --batch mode for batch retrying/rescoring of packages.
  * requestsync:
    - Use UBU* environment variables before the DEB* ones (lp: #400133)
    - Split requestsync into a "mail" module and a "lpapi" module and use
      the LP API only when --lp was used. In "mail" mode requestsync has
      to ask some more questions for parts it can't find out without LP API.
      (lp: #406659, #416955)

  [ Iain Lane ]
  * requestsync:
    - Guard some calls when -n is specified
    - Fetch changelog of specified version, not current version. If an
      experimenal upload happened after the unstable one we're syncing, this
      is considered to be current by p.d.o and we would get those changelog
      entries in the sync request
    - Remove trailing fullstop from sync bug title
  * suspicious-source: Add *.hs *.el *.css to whitelist

  [ Siegfried-Angel Gevatter Pujals ]
  * pbuilder-dist:
     - Expand "~" in PBUILDFOLDER to the user's home directory.
     - If there's a "etc/<distro>/apt.conf" file inside the build result
       directory, pass it to pbuilder as --aptconfdir. Thanks to Paul Novotny
       and Ryan Pavlik (LP: #363043).

  [ Luca Falavigna ]
  * Switch to python-support to ease initial import into Debian:
    - debian/control: build-depend on python-support instead of pycentral,
      also remove unneeded XB-Python-Version field from binary stanza.
    - debian/rules: set DEB_PYTHON_SYSTEM to pysupport.
    - ubuntu-dev-tools.preinst: remove stale pycentral files on upgrades.

  [ Nathan Handler ]
  * Add pull-revu-source and doc/pull-revu-source.1
  * Update debian/copyright to include pull-revu-source

 -- Nathan Handler <email address hidden> Sun, 30 Aug 2009 17:24:23 +0000

Changed in ubuntu-dev-tools (Ubuntu):
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.