Am Donnerstag, den 25.01.2007, 18:26 -0800 schrieb
[EMAIL PROTECTED]:
> Package: liferea
> Version: 1.0.27-1
> Severity: minor
> 
> *** Please type your report below this line ***
> 
> liferea's wrapper startup script depends upon the shell passing in a
> full path for $0, which is not always the case.
> 
> For example, when running under valgrind:
> 
> [EMAIL PROTECTED]:~> valgrind liferea
> ==25483== Memcheck, a memory error detector.
> [...]
> /usr/bin/liferea: line 18: /home/jrodman/liferea-bin: No such file or 
> directory
> /usr/bin/liferea: line 18: exec: /home/jrodman/liferea-bin: cannot execute: 
> No such file or directory
> [...]
> 
> Or when trying to see, for example, how the wrapper works:
> 
> [EMAIL PROTECTED]:~> sh -x liferea
> ++ dirname liferea
> + dist_bin=.
> + params=
> + LD_LIBRARY_PATH=:
> + export LD_LIBRARY_PATH
> + '[' -z '' ']'
> ++ dbus-launch
> + eval 
> DBUS_SESSION_BUS_ADDRESS=unix:abstract=/tmp/dbus-rGNlw7uW3u,guid=affb28b60b54567c91286f0045b964cd
>  DBUS_SESSION_BUS_PID=25514
> ++ 
> DBUS_SESSION_BUS_ADDRESS=unix:abstract=/tmp/dbus-rGNlw7uW3u,guid=affb28b60b54567c91286f0045b964cd
> ++ DBUS_SESSION_BUS_PID=25514
> + export DBUS_SESSION_BUS_ADDRESS
> + exec ./liferea-bin
> /usr/bin/liferea: line 18: /home/jrodman/liferea-bin: No such file or 
> directory
> /usr/bin/liferea: line 18: exec: /home/jrodman/liferea-bin: cannot execute: 
> No such file or directory
> 
> If computing dist_bin is truly required, consider defaulting it to
> /usr/bin, in the case that $(basename $0) == $0
> 
> eg.
> 
> --- liferea.orig        2007-01-25 18:20:27.000000000 -0800
> +++ liferea     2007-01-25 18:23:22.000000000 -0800
> @@ -4,7 +4,13 @@
>  # This script should be used to start Liferea
>  # to ensure a running DBUS session.
> 
> -dist_bin=`dirname $0`
> +# default
> +dist_bin=/usr/bin
> +
> +# autodetect alternate location when $0 provides information
> +if [ "`basename $0`" != "$0" ]; then
> +       dist_bin=`dirname $0`
> +fi
>  params="$@"
> 
>  LD_LIBRARY_PATH=:$LD_LIBRARY_PATH
> 

I added your solution upstream. I only exchanged /usr/bin with the
configure variable for the installation prefix. Thanks for the patch,
will be released with 1.2.5.

Lars




-- 
To UNSUBSCRIBE, email to [EMAIL PROTECTED]
with a subject of "unsubscribe". Trouble? Contact [EMAIL PROTECTED]

Reply via email to