move LP tools from ubuntu-dev-tools to lptools

Bug #708886 reported by Benjamin Drung
24
This bug affects 3 people
Affects Status Importance Assigned to Milestone
lptools
Fix Released
Medium
Jelmer Vernooij
ubuntu-dev-tools (Ubuntu)
Fix Released
Wishlist
Jelmer Vernooij

Bug Description

Binary package hint: ubuntu-dev-tools

ubuntu-dev-tools contains a bunch of scripts that are specific for Launchpad, but not only for Ubuntu. These tools should be moved from ubuntu-dev-tools to lptools. The tools in question:

get-branches
grab-attachments
lp-list-bugs
lp-project-upload
lp-set-dup
lp-shell

Related branches

Benjamin Drung (bdrung)
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Wishlist
Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :

Should we perhaps make the names of get-branches, grab-attachments, and manage-credentials more lp specific while we're at it? Same reasoning as in Bug #500399

Revision history for this message
Benjamin Drung (bdrung) wrote :

Agreed.

Revision history for this message
Andrew Starr-Bochicchio (andrewsomething) wrote :

Also note that some of these currently depend on the ubuntutools module in ubuntu-dev-tools:

grep -r "from ubuntutools" .
./bin/manage-credentials:from ubuntutools.lp.libsupport import Launchpad, translate_api_web, LEVEL
./bin/grab-attachments:from ubuntutools.config import UDTConfig
./bin/grab-attachments:from ubuntutools.lp.libsupport import get_launchpad
./bin/lp-list-bugs:from ubuntutools.lp.libsupport import get_launchpad
./bin/get-branches:from ubuntutools.lp.lpapicache import PersonTeam
./bin/lp-project-upload:from ubuntutools.lp.libsupport import get_launchpad
./bin/lp-set-dup:from ubuntutools.config import UDTConfig

Revision history for this message
Stefano Rivera (stefanor) wrote :

The dependencies aren't that strong.

libsupport is a very thin launchpadlib wrapper (in many cases just returning a launchpad object)
UDTConfig is part of my recent work, adding common configuration-file support to u-d-t. The uses here are pretty minor (selecting LP instance).
lpapicache is more of an issue, as it's a non-trivial launchpadlib wrapper. Although the use here is pretty basic, and can easily be rewritten for launchpadlib

description: updated
Jelmer Vernooij (jelmer)
Changed in lptools:
status: New → In Progress
Changed in ubuntu-dev-tools (Ubuntu):
status: New → In Progress
Changed in lptools:
assignee: nobody → Jelmer Vernooij (jelmer)
Changed in ubuntu-dev-tools (Ubuntu):
assignee: nobody → Jelmer Vernooij (jelmer)
Jelmer Vernooij (jelmer)
Changed in lptools:
status: In Progress → Fix Committed
importance: Undecided → Medium
Benjamin Drung (bdrung)
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.129

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

  [ Colin Watson ]
  * syncpackage: Convert to new LP API, with --no-lp available for the old
    style of operation.
  * syncpackage: Require -f/--force option to overwrite Ubuntu changes.

  [ Jelmer Vernooij ]
  * Remove several tools not specific to Ubuntu that have been migrated to
    lptools (LP: #708886):
   - get-branches (renamed to lp-get-branches)
   - grab-attachments (renamed to lp-grab-attachments)
   - lp-project-upload
   - lp-list-bugs
   - lp-set-dup
   - lp-shell

  [ Stefano Rivera ]
  * syncpackage: Show changes to be synced when performing native syncs.
  * syncpackage: Check the sync blacklist.
  * syncpackage: Support --bug (extra bugs to be closed by the sync) with
    native syncs. (Bugs are closed one individually, via the API, post-sync)
  * dgetlp, submittodebian, 404main: Use unicode strings for literal strings
    containing non-ASCII characters (LP: #836661)
  * Recommend pbuilder | aptitude for get-build-deps, and exit with an error
    if neither are installed (LP: #799368)
  * get-build-deps: Tell aptitude not to follow Recommends (LP: #817500)
  * doc/requestsync.1: Document the -C option (LP: #833408)
  * ubuntutools.archive: Don't write .dsc files until we pull the entire
    source package, just hold it in memory. Avoids littering the current
    directory (LP: #838361)
  * Run harvest as part of sponsor-patch (LP: #833699)

  [ Julian Taylor ]
  * requestsync: omit dups when checking for duplicate requests (LP: #842217)

  [ Benjamin Drung ]
  * sponsor-patch: Default to not upload the package.
  * requestsync: Do not crash on user abort (Closes: #637168).

 -- Benjamin Drung <email address hidden> Tue, 06 Sep 2011 14:31:31 +0200

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

Duplicates of this bug

Other bug subscribers

Remote bug watches

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