From owner-svn-ports-head@freebsd.org Tue Jun 19 21:08:34 2018 Return-Path: Delivered-To: svn-ports-head@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2610:1c1:1:606c::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id ADD92100ECF1; Tue, 19 Jun 2018 21:08:34 +0000 (UTC) (envelope-from adridg@FreeBSD.org) Received: from mxrelay.nyi.freebsd.org (mxrelay.nyi.freebsd.org [IPv6:2610:1c1:1:606c::19:3]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client CN "mxrelay.nyi.freebsd.org", Issuer "Let's Encrypt Authority X3" (verified OK)) by mx1.freebsd.org (Postfix) with ESMTPS id 64215760E0; Tue, 19 Jun 2018 21:08:34 +0000 (UTC) (envelope-from adridg@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2610:1c1:1:6068::e6a:0]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (Client did not present a certificate) by mxrelay.nyi.freebsd.org (Postfix) with ESMTPS id 455DA18E89; Tue, 19 Jun 2018 21:08:34 +0000 (UTC) (envelope-from adridg@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id w5JL8YuR026254; Tue, 19 Jun 2018 21:08:34 GMT (envelope-from adridg@FreeBSD.org) Received: (from adridg@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id w5JL8XbS026253; Tue, 19 Jun 2018 21:08:33 GMT (envelope-from adridg@FreeBSD.org) Message-Id: <201806192108.w5JL8XbS026253@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: adridg set sender to adridg@FreeBSD.org using -f From: Adriaan de Groot Date: Tue, 19 Jun 2018 21:08:33 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r472823 - head/games/spring/files X-SVN-Group: ports-head X-SVN-Commit-Author: adridg X-SVN-Commit-Paths: head/games/spring/files X-SVN-Commit-Revision: 472823 X-SVN-Commit-Repository: ports MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-head@freebsd.org X-Mailman-Version: 2.1.26 Precedence: list List-Id: SVN commit messages for the ports tree for head List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Tue, 19 Jun 2018 21:08:34 -0000 Author: adridg Date: Tue Jun 19 21:08:33 2018 New Revision: 472823 URL: https://svnweb.freebsd.org/changeset/ports/472823 Log: Pre-emptively fix games/spring before CMake 3.12 arrives. file(GLOB .. FOLLOW_SYMLINKS ..) was never documented to work; in 3.12-rc1 this has become an error. That in itself is considered a regression in CMake [1], but the use is wrong anyway, so patch it away. The change has been accepted upstream [2] already. Not bumping PORTREVISION because no code actually changes, it just removes a cmake-time no-op. [1] https://gitlab.kitware.com/cmake/cmake/issues/18097 [2] https://springrts.com/mantis/view.php?id=6005 PR: 229101 Reviewed by: tcberner Differential Revision: https://reviews.freebsd.org/D15878 Added: head/games/spring/files/patch-rts-build-cmake-UtilJava (contents, props changed) Modified: head/games/spring/files/patch-rts-build-cmake-Util Modified: head/games/spring/files/patch-rts-build-cmake-Util ============================================================================== --- head/games/spring/files/patch-rts-build-cmake-Util Tue Jun 19 20:45:54 2018 (r472822) +++ head/games/spring/files/patch-rts-build-cmake-Util Tue Jun 19 21:08:33 2018 (r472823) @@ -1,3 +1,5 @@ +Modified for CMake 3.12 to drop FOLLOW_SYMLINKS in file(GLOB ...). + Obtained via: https://springrts.com/mantis/view.php?id=4679 From 9e0db5f602407de4e7875ca85761b41782c1bb9c Mon Sep 17 00:00:00 2001 @@ -10,13 +12,15 @@ I needed this when building on archlinux with cmake 3. ... ---- rts/build/cmake/Util.cmake +--- rts/build/cmake/Util.cmake.orig 2014-10-07 20:09:51 UTC +++ rts/build/cmake/Util.cmake -@@ -166,10 +166,11 @@ EndFunction (MakeGlobal) +@@ -165,11 +165,12 @@ EndFunction (MakeGlobal) + # Find all CMakeLists.txt files in sub-directories Macro (GetListOfSubModules list_var) - File(GLOB ${list_var} RELATIVE "${CMAKE_CURRENT_SOURCE_DIR}" FOLLOW_SYMLINKS "${CMAKE_CURRENT_SOURCE_DIR}/*/CMakeLists.txt") +- File(GLOB ${list_var} RELATIVE "${CMAKE_CURRENT_SOURCE_DIR}" FOLLOW_SYMLINKS "${CMAKE_CURRENT_SOURCE_DIR}/*/CMakeLists.txt") - ++ File(GLOB ${list_var} RELATIVE "${CMAKE_CURRENT_SOURCE_DIR}" "${CMAKE_CURRENT_SOURCE_DIR}/*/CMakeLists.txt") # Strip away the "/CMakeLists.txt" parts, so we end up with just a list of dirs, # for example: AAI;RAI;KAIK - String(REPLACE "//CMakeLists.txt" "" ${list_var} "${${list_var}}") Added: head/games/spring/files/patch-rts-build-cmake-UtilJava ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/games/spring/files/patch-rts-build-cmake-UtilJava Tue Jun 19 21:08:33 2018 (r472823) @@ -0,0 +1,16 @@ +CMake 3.12 errors on FOLLOW_SYMLINKS (which was never supported for +file(GLOB ...) anyway, but was ignored). + +See also https://springrts.com/mantis/view.php?id=6005 + +--- ./rts/build/cmake/UtilJava.cmake.bak 2014-10-07 22:09:51 UTC ++++ ./rts/build/cmake/UtilJava.cmake +@@ -45,7 +45,7 @@ + # Returns the name of the first sub-dir (in alphabetical descending order) + # under dir. + macro (GetFirstSubDirName name_var dir) +- file(GLOB dirContent RELATIVE "${dir}" FOLLOW_SYMLINKS "${dir}/*") ++ file(GLOB dirContent RELATIVE "${dir}" "${dir}/*") + foreach (dirPart ${dirContent}) + if (IS_DIRECTORY "${dir}/${dirPart}") + set(${name_var} ${dirPart})