Fails to indicate required debuild dependency

Bug #428722 reported by Steven Sheehy
20
This bug affects 3 people
Affects Status Importance Assigned to Milestone
bzr-builder
Fix Released
High
Unassigned

Bug Description

Running bzr dailydeb without debuild installed displays this useless error:

Building the source package
bzr: ERROR: [Errno 2] No such file or directory

The script should either check for debuild being installed before running or at the very least throw a useful error message to indicate to the user that they need to install debuild. I had to look into bzr-builder source to find out why it was failing.

Tags: easy
Changed in bzr-builder:
status: New → Confirmed
Revision history for this message
James Westby (james-w) wrote :

Catching and wrapping this error would be good, and making the packaging depend on
devscripts if it doesn't already.

Thanks,

James

Changed in bzr-builder:
status: Confirmed → Triaged
importance: Undecided → High
tags: added: easy
Revision history for this message
James Westby (james-w) wrote :

Oh, catching and wrapping would be something like

try:
   ...
except OSError, e:
   if e.errno != 2:
     raise
   raise MissingDependency("debuild...

defining MissingDependency near the top of the file if bzr doesn't provide something
we can re-use.

Thanks,

James

Revision history for this message
Robert Collins (lifeless) wrote : Re: [Bug 428722] Re: Fails to indicate required debuild dependency

errors.DependencyNotPresent is probably what you want.

BTW, better to use errno.ENOENT rather than '2' - more portable.

-Rob

James Westby (james-w)
Changed in bzr-builder:
status: Triaged → Fix Committed
James Westby (james-w)
Changed in bzr-builder:
milestone: none → 0.3
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.