From d105717fd776352f833b81cacbb311fb41495b24 Mon Sep 17 00:00:00 2001 From: Kubilay Kocak Date: Thu, 9 Aug 2018 06:42:47 +0000 Subject: x11-wm/blackbox: Fix build with Clang 6, Fix TOOLS_ONLY option Clang 6 (on 12-CURRENT) reports the following error during build: EWMH.cc:250:7: error: non-constant-expression cannot be narrowed from type 'long' to 'unsigned long' in initializer list [-Wc++11-narrowing] { static_cast(x), static_cast(y) }; ^~~~~~~~~~~~~~~~~~~~ There was also a bug introduced 4 years ago which removed the MAN1 variable a later INSTALL_MAN macro relied on [1]. This caused a build/install failure when the TOOLS_ONLY option was enabled. This change fixes those two issues. While I'm here level up port compliance: - Add LICENSE_FILE - Convert to OPTIONS helpers - Regenerate patches [1] http://svnweb.freebsd.org/changeset/ports/346174 [2] https://lists.freebsd.org/pipermail/freebsd-ports/2018-August/114039.html PR: 226708 Submitted by: (Clang 6 fix) Approved by: Andrew J. Caines (technically) Approved by: portmgr (implicit, build fixes, framework compliance) Reported by: Erich Dollansky [2] MFH: 2018Q3 --- x11-wm/blackbox/files/patch-src_Screen.cc | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) (limited to 'x11-wm/blackbox/files/patch-src_Screen.cc') diff --git a/x11-wm/blackbox/files/patch-src_Screen.cc b/x11-wm/blackbox/files/patch-src_Screen.cc index 133d759c8367..4bb2df4678be 100644 --- a/x11-wm/blackbox/files/patch-src_Screen.cc +++ b/x11-wm/blackbox/files/patch-src_Screen.cc @@ -1,6 +1,6 @@ ---- src/Screen.cc.orig 2008-11-17 11:30:06.000000000 -0800 -+++ src/Screen.cc 2008-11-17 11:31:37.000000000 -0800 -@@ -1870,11 +1870,12 @@ +--- src/Screen.cc.orig 2005-10-18 08:07:22 UTC ++++ src/Screen.cc +@@ -1870,11 +1870,12 @@ void BScreen::clientMessageEvent(const XClientMessageE if (event->message_type == _blackbox->ewmh().numberOfDesktops()) { unsigned int number = event->data.l[0]; -- cgit v1.2.3