From owner-svn-ports-all@freebsd.org Sun Feb 12 12:02:45 2017 Return-Path: Delivered-To: svn-ports-all@mailman.ysv.freebsd.org Received: from mx1.freebsd.org (mx1.freebsd.org [IPv6:2001:1900:2254:206a::19:1]) by mailman.ysv.freebsd.org (Postfix) with ESMTP id 924F1CD047C; Sun, 12 Feb 2017 12:02:45 +0000 (UTC) (envelope-from tcberner@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 mx1.freebsd.org (Postfix) with ESMTPS id 6CEBF10DF; Sun, 12 Feb 2017 12:02:45 +0000 (UTC) (envelope-from tcberner@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.37]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id v1CC2i37036140; Sun, 12 Feb 2017 12:02:44 GMT (envelope-from tcberner@FreeBSD.org) Received: (from tcberner@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id v1CC2ihL036137; Sun, 12 Feb 2017 12:02:44 GMT (envelope-from tcberner@FreeBSD.org) Message-Id: <201702121202.v1CC2ihL036137@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: tcberner set sender to tcberner@FreeBSD.org using -f From: "Tobias C. Berner" Date: Sun, 12 Feb 2017 12:02:44 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r433938 - in head/net-im: libqtelegram-ae libqtelegram-ae/files telegramqml telegramqml/files X-SVN-Group: ports-head MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit X-BeenThere: svn-ports-all@freebsd.org X-Mailman-Version: 2.1.23 Precedence: list List-Id: SVN commit messages for the ports tree List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , X-List-Received-Date: Sun, 12 Feb 2017 12:02:45 -0000 Author: tcberner Date: Sun Feb 12 12:02:44 2017 New Revision: 433938 URL: https://svnweb.freebsd.org/changeset/ports/433938 Log: Change c++11 behaviour of net-im/libqtelegram-ae and net-im/telegramqml. Instead of explicitely disabling c++11 mode as the code uses narrowing of unsigned ints to ints, simply pass -no-c++11-narrowing if the compiler is clang. This is done in preparation for Qt 5.7.1, which requires c++11 mode. Approved by: rakuco (mentor) Added: head/net-im/libqtelegram-ae/files/ head/net-im/libqtelegram-ae/files/patch-libqtelegram-ae.pro (contents, props changed) head/net-im/telegramqml/files/ head/net-im/telegramqml/files/patch-telegramqml.pro (contents, props changed) Modified: head/net-im/libqtelegram-ae/Makefile head/net-im/telegramqml/Makefile Modified: head/net-im/libqtelegram-ae/Makefile ============================================================================== --- head/net-im/libqtelegram-ae/Makefile Sun Feb 12 12:01:02 2017 (r433937) +++ head/net-im/libqtelegram-ae/Makefile Sun Feb 12 12:02:44 2017 (r433938) @@ -3,6 +3,7 @@ PORTNAME= libqtelegram-ae PORTVERSION= 6.1 +PORTREVISION= 1 DISTVERSIONPREFIX= v DISTVERSIONSUFFIX= -stable CATEGORIES= net-im @@ -22,12 +23,6 @@ USE_QT5= qmake_build buildtools_build co USE_OPENSSL= yes USE_LDCONFIG= yes -# Explicitly disable C++11, as Qt 5.6 defaults to -std=gnu++11 when the -# compiler supports it and the build currently fails. -# ../libqtelegram-aseman-edition-6.1-stable/telegram/types/accountdaysttl.cpp:51:10: error: case value evaluates to 3100684255, which cannot be narrowed to type 'int' [-Wc++11-narrowing] -# case typeAccountDaysTTL: { -QMAKE_ARGS= CONFIG-="c++11" - post-patch: @${REINPLACE_CMD} -e "/OPENSSL_INCLUDE_PATH/d" \ ${WRKSRC}/libqtelegram-ae.pri Added: head/net-im/libqtelegram-ae/files/patch-libqtelegram-ae.pro ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net-im/libqtelegram-ae/files/patch-libqtelegram-ae.pro Sun Feb 12 12:02:44 2017 (r433938) @@ -0,0 +1,18 @@ +C++11 does not allow narrowing of integer types by default -- however multiple +constants defined in the code are too large to fit into a signed int, and rely +on the narrowing behaviour. + +--- libqtelegram-ae.pro.orig 2017-02-10 23:08:39 UTC ++++ libqtelegram-ae.pro +@@ -25,6 +25,11 @@ linux { + } + } + ++# Explictely allow narrowing of integer types ++clang { ++ QMAKE_CXXFLAGS += -Wno-c++11-narrowing ++} ++ + !contains(CONFIG, no_install) { + isEmpty(PREFIX) { + isEmpty(INSTALL_HEADERS_PREFIX): INSTALL_HEADERS_PREFIX = $$[QT_INSTALL_HEADERS] Modified: head/net-im/telegramqml/Makefile ============================================================================== --- head/net-im/telegramqml/Makefile Sun Feb 12 12:01:02 2017 (r433937) +++ head/net-im/telegramqml/Makefile Sun Feb 12 12:02:44 2017 (r433938) @@ -3,6 +3,7 @@ PORTNAME= telegramqml PORTVERSION= 0.9.2 +PORTREVISION= 1 DISTVERSIONPREFIX= v CATEGORIES= net-im @@ -25,12 +26,6 @@ USE_LDCONFIG= yes QMAKE_ARGS= BUILD_MODE+=lib \ LIBQTELEGRAM_INCLUDE_PATH="${LOCALBASE}/include/libqtelegram-ae" -# Explicitly disable C++11, as Qt 5.6 defaults to -std=gnu++11 when the -# compiler supports it and the build currently fails. -# ../TelegramQML-0.9.2/telegramqml.cpp:4794:10: error: case value evaluates to 2573335900, which cannot be narrowed to type 'int' [-Wc++11-narrowing] -# case Update::typeUpdateReadHistoryInbox: -QMAKE_ARGS+= CONFIG-="c++11" - post-patch: @${REINPLACE_CMD} -e "/isEmpty(OPENSSL_INCLUDE_PATH)/d" \ ${WRKSRC}/telegramqml.pri Added: head/net-im/telegramqml/files/patch-telegramqml.pro ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net-im/telegramqml/files/patch-telegramqml.pro Sun Feb 12 12:02:44 2017 (r433938) @@ -0,0 +1,18 @@ +C++11 does not allow narrowing of integer types by default -- however multiple +constants defined in the code are too large to fit into a signed int, and rely +on the narrowing behaviour. + +--- telegramqml.pro.orig 2017-02-11 13:02:31 UTC ++++ telegramqml.pro +@@ -31,6 +31,11 @@ linux { + } + } + ++# Explictely allow narrowing of integer types ++clang { ++ QMAKE_CXXFLAGS += -Wno-c++11-narrowing ++} ++ + contains(BUILD_MODE,lib) { + isEmpty(PREFIX) { + isEmpty(INSTALL_HEADERS_PREFIX): INSTALL_HEADERS_PREFIX = $$[QT_INSTALL_HEADERS]