[backportpackage] -sa detection does not work with private PPAs

Bug #1106429 reported by Evan Broder
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Medium
Evan Broder

Bug Description

backportpackage's attempt to detect whether -sa is needed or not is foiled by private PPAs, as hitting https://launchpad.net/~whomever/+archive/whatever/+files/my_file redirects to the login page. httplib2 follows redirects, and the login page returns a 200, so backportpackage concludes it doesn't need to pass -sa.

Not totally sure what the right fix here is. My recollection was that determining this from using the LP API was difficult, but I haven't looked at the options in a while.

Benjamin Drung (bdrung)
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
summary: - [backportpackage] -sa detection does not worth with private PPAs
+ [backportpackage] -sa detection does not work with private PPAs
Evan Broder (broder)
Changed in ubuntu-dev-tools (Ubuntu):
assignee: nobody → Evan Broder (broder)
Martin Pitt (pitti)
Changed in ubuntu-dev-tools (Ubuntu):
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

---------------
ubuntu-dev-tools (0.149) unstable; urgency=low

  [ Marc Deslauriers ]
  * mk-sbuild: allow specifying the schroot profile.
  * ubuntutools/config.py: properly handle name being None.

  [ Loïc Minier ]
  * ubuntutools.config: decode developer names with the current locale's
    encoding and add corresponding test; fixes handling of non-ascii names.

  [ Adam Gandelman ]
  * backportpackage: Fix destination for --dont-sign option. (LP: #1183985)

  [ Evan Broder ]
  * Only conclude that we don't need an orig tarball if we're sent to the
    librarian (as opposed to the login page from a private PPA).
    (LP: #1106429)

 -- Benjamin Drung <email address hidden> Tue, 13 Aug 2013 23:12:42 +0200

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.