checkinstall fails due to "which dpkg" multiple results

Bug #4423 reported by Mark Lord
8
Affects Status Importance Assigned to Milestone
checkinstall (Debian)
New
Unknown
checkinstall (Ubuntu)
Fix Released
Medium
MOTU

Bug Description

checkinstall fails to run if the user has their own version/alias for "which".

To fix this, all calls to "which" in checkinstall should use the full pathname prefix, eg. /bin/which

Cheers

Changed in checkinstall:
assignee: nobody → motu
Revision history for this message
Emmet Hikory (persia) wrote : Patch for which -> /usr/bin/which

/usr/bin/which is the which provided by debianutils 2.15.2, and is the which used by checkinstall patched with the attached patch.

Emmet Hikory (persia)
Changed in checkinstall:
status: Unconfirmed → Confirmed
Revision history for this message
Steve Kowalik (stevenk) wrote :

Patch applied and uploaded.

Changed in checkinstall:
status: Confirmed → Fix Released
Revision history for this message
Albin Tonnerre (lutin) wrote :

checkinstall (1.6.1-4ubuntu1) gutsy; urgency=low

  * Merge from Debian unstable. Remaining Ubuntu changes:
    - Use /usr/bin/which instead of which. (LP: #4423)
    - add make to Depends, checkinstall doesn't work without it. (LP: #3142)
    - Modify Maintainer value to match Debian-Maintainer-Field Spec
  * Debian version fixes LP: #115107

 -- Albin Tonnerre <email address hidden> Wed, 22 Aug 2007 11:59:23 +0200

Revision history for this message
Albin Tonnerre (lutin) wrote :

checkinstall (1.6.1-5ubuntu1) hardy; urgency=low

  * Merge from Debian unstable. Remaining Ubuntu changes:
    - Use /usr/bin/which instead of which. (LP: #4423)
    - add make to Depends, checkinstall doesn't work without it. (LP: #3142)
    - Modify Maintainer value to match Debian-Maintainer-Field Spec
  * Debian version fixes LP: #160475

checkinstall (1.6.1-5) unstable; urgency=low

  * Fix typo: striping -> stripping.
   (Closes: #447776)
  * Fix FTBFS on kfreebsd-amd64.
   (Closes: #451111)
  * Move manpage to section 8 and note that checkinstall must be run as root.
   (Closes: #448533)

 -- Albin Tonnerre <email address hidden> Sun, 18 Nov 2007 14:05:51 +0100

Changed in checkinstall:
status: Unknown → New
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.