Back In Time

Configure scripts aren't portable

Bug #377429 reported by Jonathan Wiltshire
10
This bug affects 2 people
Affects Status Importance Assigned to Milestone
Back In Time
Fix Released
Undecided
Dan

Bug Description

The configure scripts in common/, gnome/ and kde4/ of 0.9.24 don't use bash features - please downgrade the shebang to /bin/sh instead.

Related branches

lp:backintime/1.0
Dan (danleweb)
Changed in backintime:
assignee: nobody → Dan (dan-le-web)
Revision history for this message
Dan (danleweb) wrote :

If I use 'sh', 'echo -e' does not seems to work !

Revision history for this message
Jonathan Wiltshire (jwiltshire) wrote :

The attached patch uses printf instead of echo for output, so it can safely be used with portable shells (tested with /bin/dash particularly, given https://wiki.ubuntu.com/DashAsBinSh).

Does the "SHELL=/bin/bash" declaration ever actually get used, or can it go?

summary: - Configure scripts don't use bash features
+ Configure scripts aren't portable
Bart de Koning (bratdaking)
Changed in backintime:
status: New → Fix Released
Revision history for this message
Jonathan Wiltshire (jwiltshire) wrote :

This bug doesn't seem to actually be fixed, the configure scripts still depend on bash as a shell.

Changed in backintime:
status: Fix Released → New
Germar (germar)
Changed in backintime:
status: New → Fix Committed
Germar (germar)
Changed in backintime:
status: Fix Committed → Fix Released
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.