git: fbe5a96591ff - main - graphics/openshadinglanguage: Remove unneeded definition of NDEBUG

Go to: [ bottom of page ] [ top of archives ] [ this month ]

From: Daniel Engberg <diizzy_at_FreeBSD.org>
Date: Thu, 03 Nov 2022 21:53:02 UTC

The branch main has been updated by diizzy:

URL: https://cgit.FreeBSD.org/ports/commit/?id=fbe5a96591ff8d31d0fd6ac4cb784ba19c5363fd

commit fbe5a96591ff8d31d0fd6ac4cb784ba19c5363fd
Author:     Daniel Engberg <diizzy@FreeBSD.org>
AuthorDate: 2022-11-03 07:03:53 +0000
Commit:     Daniel Engberg <diizzy@FreeBSD.org>
CommitDate: 2022-11-03 21:51:57 +0000

    graphics/openshadinglanguage: Remove unneeded definition of NDEBUG
    
    As of 3c5189a0fd4c82addfb659024bf28c5006bf88f7 there's no need to
    define NDEBUG by hand
    
    Reviewed by:    Shane Ambler <FreeBSD@Shaneware.biz> (private email)
---
 graphics/openshadinglanguage/Makefile | 2 --
 1 file changed, 2 deletions(-)

diff --git a/graphics/openshadinglanguage/Makefile b/graphics/openshadinglanguage/Makefile
index 27791dfe0d3f..317839e0b924 100644
--- a/graphics/openshadinglanguage/Makefile
+++ b/graphics/openshadinglanguage/Makefile
@@ -33,8 +33,6 @@ CMAKE_ON=	ENABLERTTI OSL_BUILD_TESTS USE_LLVM_BITCODE USE_LIBCPLUSPLUS
 CMAKE_OFF=	USE_BOOST_WAVE
 CMAKE_ARGS+=	-DLLVM_CONFIG:STRING="${LOCALBASE}/bin/llvm-config${LLVM_VER}" \
 		-DCMAKE_INSTALL_DOCDIR:STRING=${DOCSDIR} -DCMAKE_CXX_STANDARD=14
-# This flag fixes linking with clang 9.0 ??
-CXXFLAGS+=	-DNDEBUG
 
 OPTIONS_DEFINE=		OSLTOY
 OPTIONS_SUB=		yes