Date: Fri, 12 Apr 2019 20:26:19 +0000 (UTC) From: Dimitry Andric <dim@FreeBSD.org> To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-branches@freebsd.org Subject: svn commit: r498742 - in branches/2019Q2/www: squid-devel/files squid/files Message-ID: <201904122026.x3CKQJwq076359@repo.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: dim (src committer) Date: Fri Apr 12 20:26:19 2019 New Revision: 498742 URL: https://svnweb.freebsd.org/changeset/ports/498742 Log: MFH: r498377 Fix build of www/squid and www/squid-devel with clang 8 This fixes an error with a defaulted copy constructor: ../../src/security/ServerOptions.h:38:5: error: explicitly defaulted copy constructor is implicitly deleted [-Werror,-Wdefaulted-function-deleted] ServerOptions(const ServerOptions &) = default; ^ ../../src/security/ServerOptions.h:110:29: note: copy constructor of 'ServerOptions' is implicitly deleted because field 'clientCaStack' has a deleted copy constructor X509_NAME_STACK_Pointer clientCaStack; ^ /usr/include/c++/v1/memory:2494:3: note: copy constructor is implicitly deleted because 'unique_ptr<stack_st_X509_NAME, Security::ServerOptions::sk_X509_NAME_free_wrapper>' has a user-declared move constructor unique_ptr(unique_ptr&& __u) noexcept ^ The copy constructor can instead be deleted. Approved by: portmgr (joneum) PR: 236210 Added: branches/2019Q2/www/squid-devel/files/patch-src_security_ServerOptions.h - copied unchanged from r498377, head/www/squid-devel/files/patch-src_security_ServerOptions.h branches/2019Q2/www/squid/files/patch-src_security_ServerOptions.h - copied unchanged from r498377, head/www/squid/files/patch-src_security_ServerOptions.h Modified: Directory Properties: branches/2019Q2/ (props changed) Copied: branches/2019Q2/www/squid-devel/files/patch-src_security_ServerOptions.h (from r498377, head/www/squid-devel/files/patch-src_security_ServerOptions.h) ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ branches/2019Q2/www/squid-devel/files/patch-src_security_ServerOptions.h Fri Apr 12 20:26:19 2019 (r498742, copy of r498377, head/www/squid-devel/files/patch-src_security_ServerOptions.h) @@ -0,0 +1,11 @@ +--- src/security/ServerOptions.h.orig 2018-06-11 16:30:57 UTC ++++ src/security/ServerOptions.h +@@ -29,7 +29,7 @@ class ServerOptions : public PeerOptions (public) + // is more secure to have only a small set of trusted CA. + flags.tlsDefaultCa.defaultTo(false); + } +- ServerOptions(const ServerOptions &) = default; ++ ServerOptions(const ServerOptions &) = delete; + ServerOptions &operator =(const ServerOptions &); + ServerOptions(ServerOptions &&o) { this->operator =(o); } + ServerOptions &operator =(ServerOptions &&o) { this->operator =(o); return *this; } Copied: branches/2019Q2/www/squid/files/patch-src_security_ServerOptions.h (from r498377, head/www/squid/files/patch-src_security_ServerOptions.h) ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ branches/2019Q2/www/squid/files/patch-src_security_ServerOptions.h Fri Apr 12 20:26:19 2019 (r498742, copy of r498377, head/www/squid/files/patch-src_security_ServerOptions.h) @@ -0,0 +1,11 @@ +--- src/security/ServerOptions.h.orig 2019-02-19 02:46:22 UTC ++++ src/security/ServerOptions.h +@@ -35,7 +35,7 @@ class ServerOptions : public PeerOptions (public) + // is more secure to have only a small set of trusted CA. + flags.tlsDefaultCa.defaultTo(false); + } +- ServerOptions(const ServerOptions &) = default; ++ ServerOptions(const ServerOptions &) = delete; + ServerOptions &operator =(const ServerOptions &); + ServerOptions(ServerOptions &&o) { this->operator =(o); } + ServerOptions &operator =(ServerOptions &&o) { this->operator =(o); return *this; }
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201904122026.x3CKQJwq076359>