Checkinstall cannot create a package if the current directory contain space in name

Bug #335855 reported by Ansus
14
This bug affects 2 people
Affects Status Importance Assigned to Milestone
CheckInstall
Invalid
Medium
checkinstall (Ubuntu)
Triaged
Low
Unassigned

Bug Description

If the name of current directory, in which the software to be installed is located, contains spaces, checkinstall cannot build a package.

Revision history for this message
Andreas Noteng (andreas-noteng) wrote :

Well, package dirs should be equal packagename-version and should not contain spaces. But I can confirm this bug in Lucid.
Did a test with mediatomb:
Downloaded sources using apt-get source mediatomb
installed build-dep using apt-get build-dep mediatomb
renamed the 'mediatomb-version' folder to 'mediatomb version'
autoreconf -i + ./configure
checkinstall
failed with the following error:
Building file list...OK
/usr/bin/checkinstall: line 1772: [: /home/andreas/bzr/mediatomb: binary operator expected
/usr/bin/checkinstall: line 1875: cd: /home/andreas/bzr/mediatomb: No such file or directory

Changed in checkinstall (Ubuntu):
status: New → Confirmed
Revision history for this message
Andreas Noteng (andreas-noteng) wrote :

Can confirm the presence of this issue in the latest upstream version (1.6.2).

Revision history for this message
Ansus (neptunia) wrote :

This is especially annoying for russian users because the usual name for desktop directory is "Рабочий стол" - literally "Working table". So any software that cannot work with directories, located on the desktop produces inconvinience.

Another example is alien utility which completely cannot work with non-English names in path, so one even cannot run it for software stored in "Загрузки" (standard Gnome name for download directory in Russian locale), neither from desktop, nor fron home directory of a user with Cyrillic name.

Changed in checkinstall:
status: New → Confirmed
Philip Muškovac (yofel)
Changed in checkinstall (Ubuntu):
status: Confirmed → Triaged
Revision history for this message
Philip Muškovac (yofel) wrote :

Marking Triaged/Low at request from Andreas.

Changed in checkinstall (Ubuntu):
importance: Undecided → Low
Changed in checkinstall:
status: Confirmed → New
Changed in checkinstall:
importance: Undecided → Unknown
status: New → Unknown
Revision history for this message
Andreas Noteng (andreas-noteng) wrote :

Could you please test if there's been improvement in the latest package checkinstall-1.6.2-3ubuntu1

Changed in checkinstall:
importance: Unknown → Medium
status: Unknown → Confirmed
Changed in checkinstall:
status: Confirmed → Invalid
Revision history for this message
Chris Balcum (sherl0k) wrote :

Someone has already found a fix for this and forked it in their own git repo:

https://github.com/ruxkor/checkinstall/commit/9f2cef7810cfec70c37a79b024989bb479cada7e

Can someone please merge this upstream? It's as simple as just adding quotes around variables (which should be done anyway)

Revision history for this message
Andreas Noteng (andreas-noteng) wrote :

This has been fixed upstream, and a patched package wil be uploaded to Debian as soon as I finish some other issues.

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.