Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 29 Jul 2023 19:01:04 GMT
From:      =?utf-8?Q?Stefan=20E=C3=9Fer?= <se@FreeBSD.org>
To:        src-committers@FreeBSD.org, dev-commits-src-all@FreeBSD.org, dev-commits-src-main@FreeBSD.org
Subject:   git: 382af2f8d7bf - main - usr.bin/gh-bc: fix Makefile for WITHOUT_NLS_CATALOGS case
Message-ID:  <202307291901.36TJ14M9037457@gitrepo.freebsd.org>

next in thread | raw e-mail | index | archive | help
The branch main has been updated by se:

URL: https://cgit.FreeBSD.org/src/commit/?id=382af2f8d7bf576ff583543737e1134701ccc63d

commit 382af2f8d7bf576ff583543737e1134701ccc63d
Author:     Stefan Eßer <se@FreeBSD.org>
AuthorDate: 2023-07-29 18:52:53 +0000
Commit:     Stefan Eßer <se@FreeBSD.org>
CommitDate: 2023-07-29 18:57:32 +0000

    usr.bin/gh-bc: fix Makefile for WITHOUT_NLS_CATALOGS case
    
    Some macro definitions had been moved into a Makefile section
    that depends on MK_NLS_CATALOGS != "no", leading to LTO and the
    installation of tests being disabled in the WITHOUT_NLS_CATALOGS
    case.
    
    Reported by:    Yuri <yuri@aetern.org>
---
 usr.bin/gh-bc/Makefile | 16 ++++++++--------
 1 file changed, 8 insertions(+), 8 deletions(-)

diff --git a/usr.bin/gh-bc/Makefile b/usr.bin/gh-bc/Makefile
index 6554d626af89..8e5e90715ff0 100644
--- a/usr.bin/gh-bc/Makefile
+++ b/usr.bin/gh-bc/Makefile
@@ -61,6 +61,14 @@ CFLAGS+=	-DNDEBUG
 CFLAGS+=	-DNLSPATH=/usr/share/nls/%L/%N.cat
 CFLAGS+=	-I${BCDIR}/include
 
+# prevent floating point incompatibilities caused by -flto on some architectures
+.if ${MACHINE_ARCH} != riscv64
+CFLAGS+=	-flto
+.endif
+
+HAS_TESTS=	yes
+SUBDIR.${MK_TESTS}+= tests
+
 .if ${MK_NLS_CATALOGS} == "no"
 CFLAGS+=	-DBC_ENABLE_NLS=0
 MAN_SRC_BC=	bc/N.1
@@ -70,14 +78,6 @@ CFLAGS+=	-DBC_ENABLE_NLS=1
 MAN_SRC_BC=	bc/A.1
 MAN_SRC_DC=	dc/A.1
 
-# prevent floating point incompatibilities caused by -flto on some architectures
-.if ${MACHINE_ARCH} != riscv64
-CFLAGS+=	-flto
-.endif
-
-HAS_TESTS=	yes
-SUBDIR.${MK_TESTS}+= tests
-
 .for catalog in ${CATALOGS}
 NLS+=           ${catalog:C/.*://}
 NLSSRCFILES_${catalog:C/.*://}= ${catalog:C/.*://}.msg



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?202307291901.36TJ14M9037457>