debpython.pydist.guess_dependency prints to stderr if egg-info not present

Bug #812960 reported by Jonathan Lange
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
python-defaults (Ubuntu)
Fix Released
Low
Unassigned

Bug Description

Call guess_dependency('foo') when foo is not at all present on a system. guess_dependency will try running a command that looks like: "dpkg -S 'foo-?*\\.egg-info' | grep '/python2\\../\\|/pyshared/'". This command runs, prints output to stderr, and that output is displayed in the stderr of whatever program is calling guess_dependency.

Instead, no output should be displayed at all. Functions should give feedback via return values or raised exceptions.

The way to do this is to add stderr=PIPE to the arguments to Popen.

Revision history for this message
Scott Kitterman (kitterman) wrote :

Fix committed to the Debian VCS for python-defaults.

Changed in python-defaults (Ubuntu):
importance: Undecided → Low
status: New → Fix Committed
Revision history for this message
Launchpad Janitor (janitor) wrote :

This bug was fixed in the package python-defaults - 2.7.2-5ubuntu1

---------------
python-defaults (2.7.2-5ubuntu1) oneiric; urgency=low

  * Merge from Debian Experimental. Remaining Ubuntu changes:
    - Package priorities and status.
    - Adjust version numbers in maintainer scripts.
    - Provide Python pkgconfig defaults

python-defaults (2.7.2-5) experimental; urgency=low

  [ Piotr Ożarowski ]
  * dh_python2:
    - generates rtupdate scripts for private directories that compile files
      from given package only, add --compile-all to dh_python2's call if you
      want the old behaviour (i.e. compile also files (plugins?) provided by
      other packages, which do not use a helper/bytecompile them)
    - add --no-dbg-cleaning option (to disable removing files from debug
      packages)
    - fix handling "distname[features]" in debpython.pydist.guess_dependency
      (closes: 636255)
  * pyclean now accepts --package and private dir argument at the same time

  [ Scott Kitterman ]
  * debpyhton/pydist: Incorporate patch from Jonathan Lange to stop printing
    to stderr (LP: #812960)
  * Add Piotr Ożarowski <email address hidden> back to Uploaders
 -- Scott Kitterman <email address hidden> Wed, 10 Aug 2011 22:19:46 -0400

Changed in python-defaults (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.