submittodebian crashes after successful filing due to editor backup files

Bug #899399 reported by Steve Langasek
6
This bug affects 1 person
Affects Status Importance Assigned to Milestone
ubuntu-dev-tools (Ubuntu)
Fix Released
Medium
Stefano Rivera

Bug Description

My editor leaves backup files behind using the ~ convention. This makes submittodebian unhappy; after successfully sending the bug report to Debian, it then tries to clean up the tmpdir and fails:

Traceback (most recent call last):
  File "/usr/bin/submittodebian", line 195, in <module>
    main()
  File "/usr/bin/submittodebian", line 192, in main
    os.rmdir(tmpdir)
OSError: [Errno 39] Directory not empty: '/tmp/tmpnzr4dF'

Perhaps submittodebian should clean that directory harder?

ProblemType: Bug
DistroRelease: Ubuntu 12.04
Package: ubuntu-dev-tools 0.136
ProcVersionSignature: Ubuntu 3.1.0-3.3-generic 3.1.0
Uname: Linux 3.1.0-3-generic x86_64
ApportVersion: 1.90-0ubuntu1
Architecture: amd64
Date: Fri Dec 2 12:54:05 2011
InstallationMedia: Ubuntu 10.04.1 LTS "Lucid Lynx" - Release amd64 (20100816.1)
PackageArchitecture: all
ProcEnviron:
 PATH=(custom, user)
 LANG=en_US.UTF-8
 SHELL=/bin/bash
SourcePackage: ubuntu-dev-tools
UpgradeStatus: Upgraded to precise on 2011-11-08 (24 days ago)

Revision history for this message
Steve Langasek (vorlon) wrote :
Changed in ubuntu-dev-tools (Ubuntu):
importance: Undecided → Low
status: New → Fix Committed
importance: Low → Medium
assignee: nobody → Stefano Rivera (stefanor)
Revision history for this message
Launchpad Janitor (janitor) wrote :

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

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

  [ Stefano Rivera ]
  * mk-sbuild: Make --eatmydata command line flag actually work.
  * Remove dgetlp. No longer needed.
  * Use httplib2 everywhere that we do https. The python stdlib doesn't do
    certificate verification.
  * requestbackport:
    - Check for existing backport bugs first.
    - Check that there are published binaries (i.e. not stuck in bin-NEW).
  * pull-lp-source, requestbackport: Take the latest version from any
    non-backports pocket. Implemented by making lpapicache's getSourcePackage
    smarter.
  * sponsor-patch: Build a source package for lintian to run on, when not
    test-building syncs.
  * sponsor-patch: Check the bug's title, not the task, when determining
    source series for syncs.
  * mk-sbuild, pbuilder-dist, ubuntu-build: Add armhf.
  * pull-debian-source, pull-lp-source: Resolve the source package (via DDE),
    if a binary package was requested (LP: #617349)
  * submittodebian:
    - Do the report boiler-plate checking in a script that wraps an editor, so
      that we only edit the report once, after checking for duplicates.
    - rm the tmpdir with a little more force (shutil.rmtree) (LP: #899399)
  * New Tools: (LP: #876554)
    - ubuntu-upload-permission: Query upload permissions.
    - seeded-in-ubuntu: Query a package's seed status. Whether it is on
      current daily images and/or part of the supported seed.
  * syncpackage: Support sponsorship for native-syncs, now that LP does.

  [ Andreas Moog ]
  * sponsor-patch: Check permission to unsubscribe sponsors-team (LP: #896884)
  * grep-merges: We already require a UTF-8 enabled terminal, so encode
    package and uploader name in UTF-8 (LP: #694388)
  * requestsync: Give user option to retry in case of temporary error
    (LP: #850360)

 -- Stefano Rivera <email address hidden> Fri, 09 Dec 2011 12:59:29 +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.