dgetlp crashes

Bug #979117 reported by Scott Kitterman on 2012-04-11
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Medium
Unassigned

Bug Description

From current precise:

dgetlp http://launchpadlibrarian.net/101339287/kde-workspace_4.8.2a-0ubuntu2.dsc
Getting kde-workspace_4.8.2a-0ubuntu2.dsc
gpg: directory `/home/kitterma/.gnupg' created
gpg: new configuration file `/home/kitterma/.gnupg/gpg.conf' created
gpg: WARNING: options in `/home/kitterma/.gnupg/gpg.conf' are not yet active during this run
gpg: keyring `/home/kitterma/.gnupg/secring.gpg' created
gpg: keyring `/home/kitterma/.gnupg/pubring.gpg' created
gpg: Signature made Wed Apr 11 16:33:39 2012 UTC using DSA key ID DD4D5088
gpg: Can't check signature: public key not found
Getting kde-workspace_4.8.2a-0ubuntu2.debian.tar.gz
Getting kde-workspace_4.8.2a.orig.tar.xz
Traceback (most recent call last):
  File "/usr/bin/dgetlp", line 332, in <module>
    main()
  File "/usr/bin/dgetlp", line 326, in main
    download(dscinfo, number-2, dscinfo.files[0][-1]) # .orig.tar.gz
  File "/usr/bin/dgetlp", line 222, in download
    status(u"Failed to fetch «%s» file, aborting.", ftype)
  File "/usr/bin/dgetlp", line 201, in status
    print msg % tuple(args)
UnicodeEncodeError: 'ascii' codec can't encode character u'\xab' in position 16: ordinal not in range(128)

Benjamin Drung (bdrung) wrote :

We could switch to Python 3 to get rid of these bugs (if all addional libraries support Python 3).

Stefano Rivera (stefanor) wrote :

Switching to Python3 doesn't magically remove all unicode issues in Python code.

The first bug here is that dgetlp doesn't know about xz files, and a bunch of other things.

The second one is that error message printing won't work in non-UTF-8 locales.

Benjamin Drung (bdrung) on 2012-04-12
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
Benjamin Drung (bdrung) on 2012-05-12
Changed in ubuntu-dev-tools (Ubuntu):
status: New → Fix Committed
Launchpad Janitor (janitor) wrote :

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

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

  [ Stefano Rivera ]
  * mk-sbuild: Support kmod, when checking for overlayfs availability.
  * pbuilder-dist: improve bash_completion for *.dsc files. Thanks Maarten
    Bezemer. (Closes: #670924, LP: #770529)
  * check-mir, check-symbols, grep-merges, pbuilder-dist-simple,
    setup-packaging-environment, submittodebian, ubuntu-iso:
    Do enough argument parsing to handle --help (LP: #988009)
  * dgetlp: Require a UTF-8 locale, or it'll crash when displaying errors
    (LP: #979117)
  * pbuilder-dist: Don't try to enable -updates for Debian testing
    (LP: #993006)
  * pbuilder-dist, pull-debian-source, pull-lp-source, requestsync,
    reverse-depends, submittodebian, syncpackage:
    Handle outdated distro-info data. Fall back to sane defaults where
    possible. (Closes: #673957)
  * backportpackage: Avoid uploading orig tarballs if they are already present
    in the destination PPA (LP: #691897)
  * Allow mk-sbuild to be run by root if a configuration file exists
    (LP: #888736)
  * backportpackage: Allow unsigned backports (LP: #992739)
  * update-maintainer: Add a function to restore the original maintainer.
    - Update sponsor-patch to use the new API resulting from this change
      (LP: #1002999)
  * submittodebian: Revert Ubuntu Maintainer mangling, and re-build the source
    package before diffing. (LP: #902233)

  [ Evan Broder ]
  * backportpackage: Add -c, --close flag to include a changelog closer.
  * backportpackage: Switch to ~ubuntu12.04.1-style version numbers
    instead of ~precise1, to make our version numbers more future-proof.
  * backportpackage: Pass -v to debuild with last published version
    number. This matches the way backports have traditionally been
    generated by archive admins.

 -- Stefano Rivera <email address hidden> Mon, 28 May 2012 23:35:52 +0100

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

Other bug subscribers