HomeClusterLabs Projects

Remove bashism from configure script

Description

Remove bashism from configure script

This was the real problem behind 384d168: Debian experimental now
sports a dash with LINENO support, so configure does not fall back to
using bash instead, choking on such bash-only constructs. Unfortunately
this didn't bail out cleanly, just unexpectedly set link_all_deplibs to
no, and the error message

./configure: 13158: test: yes: unexpected operator

stayed unnoticed in the logs. Actually, link_all_deplibs=no is the
default in Debian, reducing overlinking and causing confusion overall,
see https://debbugs.gnu.org/db/13/13920.html for example.

I think being explicit about used interfaces has its merit, so now that
Corosync has it, it might be advantageous to disable link_all_deplibs
by default across the board (after this patch re-enables it as a side
effect).

Signed-off-by: Ferenc Wágner <wferi@debian.org>
Reviewed-by: Jan Friesse <jfriesse@redhat.com>

Details

Provenance
wferiAuthored on Dec 4 2021, 9:36 AM
jfriesseCommitted on Jan 3 2022, 5:01 AM
Parents
rC8b638e989c39: totemudpu: Don't block local socketpair
Branches
Unknown
Tags
Unknown

Event Timeline