comparison contrib/packaging/builddeb @ 40106:930bce0741de

packaging: cleanup() did not read the value of $CLEANUP

When the original author put CLEANUP in a conditional statement he was probably
willing to use it to control the "if". This change tries to restore that
behaviour: the "rm" clause is triggered if and only if CLEANUP is defined and
not empty.

40105:d3780c8f3180

DISTID=`(lsb_release -is 2> /dev/null | tr '[:upper:]' '[:lower:]') || echo debian`
CODENAME=`lsb_release -cs 2> /dev/null || echo unknown`
DEBFLAGS=-b

cleanup() {
    if [ '$CLEANUP' ]; then
        rm -r "$PWD/debian";
    fi
}

while [ "$1" ]; do

40106:930bce0741de

DISTID=`(lsb_release -is 2> /dev/null | tr '[:upper:]' '[:lower:]') || echo debian`
CODENAME=`lsb_release -cs 2> /dev/null || echo unknown`
DEBFLAGS=-b

cleanup() {
    if [ "$CLEANUP" ]; then
        rm -r "$PWD/debian";
    fi
}

while [ "$1" ]; do