PR - Issue 50580 - Perl can't be disabled in configure

[389-ds-bot]

Cloned from Pagure Pull-Request: https://pagure.io/389-ds-base/pull-request/50589

• Created at 2019-09-07 16:47:58 by mhonek (@kenoh)
• Merged at 2019-09-11 13:02:56

---

Bug description:
Due to incorrect use of AC_ARG_ENABLE macro arguments' semantics some
calls like ./configure --disable-* and --enable-=no worked
unexpectedly like --enable-=yes.

Fix description:
As this issue affects several ./configure options all were fixed. The
fix uses the fourth argument of the AC_ARG_ENABLE to set the default in
case no option was provided. In case an explicit --disable-* or
--enable-* argument was provided to ./configure the respective $enable_*
variable is implicitly populated by autoconf, hence subsequent
if-else statements take care of additional operations based on the
variable's value.

For the record, some implementations of the options pre-set the default
value before the AC_ARG_ENABLE in case the respective $enable_* variable
has not been set. This is a correct approach, hence left as is.

Resolves: Resolves: #3636

Author: Matus Honek kenoh@redhat.com

Review-By: ????

[389-ds-bot]

Comment from mreynolds (@mreynolds389) at 2019-09-10 21:51:01

LGTM

[389-ds-bot]

Comment from mhonek (@kenoh) at 2019-09-11 13:01:51

rebased onto 00e3331

[389-ds-bot]

Comment from mhonek (@kenoh) at 2019-09-11 13:02:56

Pull-Request has been merged by kenoh

[389-ds-bot]

Patch
50589.patch