dgetlp crashes

Bug #979117 reported by Scott Kitterman
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
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)

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

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

Revision history for this message
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)
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Medium
Benjamin Drung (bdrung)
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.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  
Everyone can see this information.

Other bug subscribers

Remote bug watches

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