Skip to content

Commit e18e100

Browse files
committed
configure: Minor grammatical changes to be more clear
1 parent a9dbcf0 commit e18e100

File tree

2 files changed

+25
-21
lines changed

2 files changed

+25
-21
lines changed

configure.ac

Lines changed: 24 additions & 20 deletions
Original file line numberDiff line numberDiff line change
@@ -98,7 +98,7 @@ AC_ARG_ENABLE([hardening],
9898

9999
AC_ARG_ENABLE([ccache],
100100
[AS_HELP_STRING([--enable-ccache],
101-
[enable building with ccache (default is yes if ccache is found)])],
101+
[use ccache for building (default is yes if ccache is found)])],
102102
[use_ccache=$enableval],
103103
[use_ccache=auto])
104104

@@ -494,11 +494,11 @@ fi
494494

495495
BITCOIN_QT_PATH_PROGS([PROTOC], [protoc],$protoc_bin_path)
496496

497-
AC_MSG_CHECKING([if bitcoind should be enabled])
497+
AC_MSG_CHECKING([whether to build bitcoind])
498498
AM_CONDITIONAL([BUILD_BITCOIND], [test x$build_bitcoind = xyes])
499499
AC_MSG_RESULT($build_bitcoind)
500500

501-
AC_MSG_CHECKING([if bitcoin-cli should be enabled])
501+
AC_MSG_CHECKING([whether to build bitcoin-cli])
502502
AM_CONDITIONAL([BUILD_BITCOIN_CLI], [test x$build_bitcoin_cli = xyes])
503503
AC_MSG_RESULT($build_bitcoin_cli)
504504

@@ -508,7 +508,7 @@ BITCOIN_QT_CONFIGURE([$use_pkgconfig], [qt4])
508508

509509
if test x$use_ipv6 = xyes; then
510510
dnl Check for ipv6 build requirements
511-
AC_MSG_CHECKING(for IPV6 build support)
511+
AC_MSG_CHECKING(for operating system IPv6 support)
512512
AC_TRY_LINK([
513513
#if defined(_WINDOWS)
514514
#include <winsock2.h>
@@ -523,18 +523,17 @@ if test x$use_ipv6 = xyes; then
523523
struct sockaddr_in6 addr;
524524
#endif
525525
int temp = socket(AF_INET6, SOCK_STREAM, 0);],
526-
[AC_MSG_RESULT(yes); have_ipv6=yes; AC_DEFINE(HAVE_IPV6, 1, [Define this symbol if you have ipv6 build support])],
526+
[AC_MSG_RESULT(yes); have_ipv6=yes; AC_DEFINE(HAVE_IPV6, 1, [Define this symbol if you have operating system IPv6 support])],
527527
[AC_MSG_RESULT(no)]; have_ipv6=no)
528528
fi
529529
AC_LANG_POP
530530

531531
if test "x$use_ccache" != "xno"; then
532-
AC_MSG_CHECKING(if ccache should be enabled)
532+
AC_MSG_CHECKING(if ccache should be used)
533533
if test x$CCACHE = x; then
534534
if test "x$use_ccache" = "xyes"; then
535535
AC_MSG_ERROR([ccache not found.]);
536536
else
537-
AC_MSG_NOTICE([ccache not found. Falling back to default CC])
538537
use_ccache=no
539538
fi
540539
else
@@ -556,40 +555,40 @@ else
556555
fi
557556

558557
dnl enable ipv6 support
559-
AC_MSG_CHECKING([if ipv6 should be enabled])
558+
AC_MSG_CHECKING([whether to build with support for IPv6])
560559
if test x$have_ipv6 = xno; then
561560
if test x$use_ipv6 = xyes; then
562-
AC_MSG_ERROR("ipv6 requested but cannot be built. use --disable-ipv6")
561+
AC_MSG_ERROR([IPv6 requested, but cannot be built. use --disable-ipv6])
563562
fi
564563
AC_MSG_RESULT(no)
565564
else
566565
if test x$use_ipv6 = xyes; then
567566
AC_MSG_RESULT(yes)
568-
AC_DEFINE([USE_IPV6],[1],[Define if ipv6 support should be compiled in])
567+
AC_DEFINE([USE_IPV6],[1],[Define if IPv6 support should be compiled in])
569568
else
570569
AC_MSG_RESULT(no)
571570
fi
572571
fi
573572

574573
dnl enable upnp support
575-
AC_MSG_CHECKING([if upnp should be enabled])
574+
AC_MSG_CHECKING([whether to build with support for UPnP])
576575
if test x$have_miniupnpc = xno; then
577576
if test x$use_upnp = xyes; then
578-
AC_MSG_ERROR("upnp requested but cannot be built. use --without-miniupnpc")
577+
AC_MSG_ERROR("UPnP requested but cannot be built. use --without-miniupnpc")
579578
fi
580579
AC_MSG_RESULT(no)
581580
else
582581
if test x$use_upnp != xno; then
583582
AC_MSG_RESULT(yes)
584-
AC_MSG_CHECKING([if upnp should be on by default])
583+
AC_MSG_CHECKING([whether to build with UPnP enabled by default])
585584
use_upnp=yes
586585
upnp_setting=0
587586
if test x$use_upnp_default != xno; then
588587
use_upnp_default=yes
589588
upnp_setting=1
590589
fi
591590
AC_MSG_RESULT($use_upnp_default)
592-
AC_DEFINE_UNQUOTED([USE_UPNP],[$upnp_setting],[Define to 1 for upnp runtime support])
591+
AC_DEFINE_UNQUOTED([USE_UPNP],[$upnp_setting],[UPnP support not compiled if undefined, otherwise value (0 or 1) determines default state])
593592
if test x$TARGET_OS = xwindows; then
594593
CPPFLAGS="$CPPFLAGS -DSTATICLIB"
595594
fi
@@ -602,14 +601,14 @@ dnl these are only used when qt is enabled
602601
if test x$bitcoin_enable_qt != xno; then
603602
BUILD_QT=qt
604603
dnl enable dbus support
605-
AC_MSG_CHECKING([if dbus should be enabled])
604+
AC_MSG_CHECKING([whether to build GUI with support for D-Bus])
606605
if test x$bitcoin_enable_qt_dbus != xno; then
607606
AC_DEFINE([USE_DBUS],[1],[Define if dbus support should be compiled in])
608607
fi
609608
AC_MSG_RESULT($bitcoin_enable_qt_dbus)
610609

611610
dnl enable qr support
612-
AC_MSG_CHECKING([if qr should be enabled])
611+
AC_MSG_CHECKING([whether to build GUI with support for QR codes])
613612
if test x$have_qrencode = xno; then
614613
if test x$use_qr == xyes; then
615614
AC_MSG_ERROR("QR support requested but cannot be built. use --without-qrencode")
@@ -629,16 +628,21 @@ if test x$bitcoin_enable_qt != xno; then
629628
AC_MSG_WARN("xgettext is required to update qt translations")
630629
fi
631630

632-
if test x$use_tests = xyes; then
631+
AC_MSG_CHECKING([whether to build test_bitcoin-qt])
632+
if test x$use_tests$bitcoin_enable_qt_test = xyesyes; then
633+
AC_MSG_RESULT([yes])
633634
BUILD_TEST_QT="test"
634-
if test x$bitcoin_enable_qt_test != xyes; then
635-
AC_MSG_ERROR("Qt Test lib not found. Use --disable-tests or --without-gui.")
636-
fi
635+
else
636+
AC_MSG_RESULT([no])
637637
fi
638638
fi
639639

640+
AC_MSG_CHECKING([whether to build test_bitcoin])
640641
if test x$use_tests = xyes; then
642+
AC_MSG_RESULT([yes])
641643
BUILD_TEST="test"
644+
else
645+
AC_MSG_RESULT([no])
642646
fi
643647

644648
if test "x$use_tests$build_bitcoind$use_qt" = "xnonono"; then

src/m4/bitcoin_qt.m4

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -113,7 +113,7 @@ AC_DEFUN([BITCOIN_QT_CONFIGURE],[
113113
114114
115115
dnl enable qt support
116-
AC_MSG_CHECKING(if GUI should be enabled)
116+
AC_MSG_CHECKING(whether to build Bitcoin Core GUI)
117117
BITCOIN_QT_CHECK([
118118
bitcoin_enable_qt=yes
119119
bitcoin_enable_qt_test=yes

0 commit comments

Comments
 (0)