Package: jacal
Version: 1c7-1

        The /usr/bin/jacal script provided by the current jacal package
        fails to set SCHEME_LIBRARY_PATH and JACALDIR correctly:

#! /bin/sh
SCHEME_LIBRARY_PATH=/usr/share/slib
export SCHEME_LIBRARY_PATH
JACALDIR=/build/jacal-PHwb3U/jacal-1c7/debian/jacal/usr/lib/jacal/
VERSION=1c7

        Specifically, SCHEME_LIBRARY_PATH (library-vicinity) is expected
        to be a prefix (i. e., contain a trailing /), not directory name;
        and JACALDIR somehow contains build-time prefix (in place of
        a run-time one.)

        Also, while we’re at it, I think that the script should make it
        possible for the user to override either or both of the variables.
        That is, I’d rather have it start with:

#! /bin/sh
: "${SCHEME_LIBRARY_PATH:=/usr/share/slib/}"
export SCHEME_LIBRARY_PATH
: "${JACALDIR:=/usr/lib/jacal/}"
VERSION=1c7

        (Which is not without its downsides, but is otherwise consistent
        with how, say, shells allow for PATH to be overridden arbitrarily.)

        FTR, the built-time directory name capture is somehow despite
        debian/rules [1] containing:

# These are from upstream's install target, turned into correctness.

debian/jacal/usr/bin/jacal:
        -mkdir -p $$(dirname $@)
        echo '#! /bin/bash'                      > $@
        echo JACALDIR=/usr/lib/jacal/           >> $@
        echo VERSION=$(version)                 >> $@
        cat jacal.sh                            >> $@
        chmod +x $@

[1] http://sources.debian.org/data/main/j/jacal/1c7-1/debian/rules

-- 
FSF associate member #7257      http://am-1.org/

Reply via email to