From owner-svn-ports-head@freebsd.org Fri Sep 11 22:51:12 2015 Return-Path: Delivered-To: svn-ports-head@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 CBEA2A012B8; Fri, 11 Sep 2015 22:51:12 +0000 (UTC) (envelope-from hrs@FreeBSD.org) Received: from repo.freebsd.org (repo.freebsd.org [IPv6:2001:1900:2254:2068::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 ADCE71A89; Fri, 11 Sep 2015 22:51:12 +0000 (UTC) (envelope-from hrs@FreeBSD.org) Received: from repo.freebsd.org ([127.0.1.70]) by repo.freebsd.org (8.15.2/8.15.2) with ESMTP id t8BMpCNU088011; Fri, 11 Sep 2015 22:51:12 GMT (envelope-from hrs@FreeBSD.org) Received: (from hrs@localhost) by repo.freebsd.org (8.15.2/8.15.2/Submit) id t8BMpAbK088000; Fri, 11 Sep 2015 22:51:10 GMT (envelope-from hrs@FreeBSD.org) Message-Id: <201509112251.t8BMpAbK088000@repo.freebsd.org> X-Authentication-Warning: repo.freebsd.org: hrs set sender to hrs@FreeBSD.org using -f From: Hiroki Sato Date: Fri, 11 Sep 2015 22:51:10 +0000 (UTC) To: ports-committers@freebsd.org, svn-ports-all@freebsd.org, svn-ports-head@freebsd.org Subject: svn commit: r396720 - in head/net/xorp: . 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-head@freebsd.org X-Mailman-Version: 2.1.20 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: Fri, 11 Sep 2015 22:51:12 -0000 Author: hrs Date: Fri Sep 11 22:51:10 2015 New Revision: 396720 URL: https://svnweb.freebsd.org/changeset/ports/396720 Log: Fix compile error on 9.x and 11.x. Spotted by: amdmi3 PR: 203011 Added: head/net/xorp/files/patch-bgp__route_table_policy.cc (contents, props changed) head/net/xorp/files/patch-libxipc-xrl_parser.cc (contents, props changed) head/net/xorp/files/patch-libxorp-ref_trie.hh (contents, props changed) head/net/xorp/files/patch-rip-port.cc (contents, props changed) Modified: head/net/xorp/Makefile head/net/xorp/files/patch-SConstruct head/net/xorp/files/patch-bgp__path_attribute.cc Modified: head/net/xorp/Makefile ============================================================================== --- head/net/xorp/Makefile Fri Sep 11 22:28:34 2015 (r396719) +++ head/net/xorp/Makefile Fri Sep 11 22:51:10 2015 (r396720) @@ -53,9 +53,6 @@ TCP_RPC_MAKE_ARGS_OFF= transport=local post-patch: ${REINPLACE_CMD} 's,%%LOCALBASE%%,${LOCALBASE},' ${WRKSRC}/SConstruct -.if ${COMPILER_TYPE} == "clang" - ${REINPLACE_CMD} 's,-Werror,& -Wno-unused-function,' ${WRKSRC}/SConstruct -.endif post-install: ${INSTALL_DATA} ${WRKSRC}/rtrmgr/config/multicast4.boot \ Modified: head/net/xorp/files/patch-SConstruct ============================================================================== --- head/net/xorp/files/patch-SConstruct Fri Sep 11 22:28:34 2015 (r396719) +++ head/net/xorp/files/patch-SConstruct Fri Sep 11 22:51:10 2015 (r396720) @@ -25,20 +25,11 @@ variables = vars) prefix = env['prefix'] -@@ -858,13 +849,13 @@ if ((build != "i386-pc-mingw32") and - host_cpu == "i386" or - host_cpu == "x86_64")): - env.AppendUnique(CFLAGS = [ -- '-Werror', -+ '-Werror -Wno-unused-function', - ]) - env.AppendUnique(CXXFLAGS = [ -- '-Werror', -+ '-Werror -Wno-unused-function', - ]) - else: -- print "WARNING: Detected funky platform, will not enable -Werror compile option: ", host_cpu -+ print "WARNING: Detected funky platform, will not enable -Werror -Wno-unused-function compile option: ", host_cpu - +@@ -880,6 +871,7 @@ env.AppendUnique(CFLAGS = [ + '-Wcast-align', + '-Wstrict-prototypes', + '-Wnested-externs', ++ '-Wno-unused-function', + '-pipe', + ]) - # NOTE: gcc specific flags. Modified: head/net/xorp/files/patch-bgp__path_attribute.cc ============================================================================== --- head/net/xorp/files/patch-bgp__path_attribute.cc Fri Sep 11 22:28:34 2015 (r396719) +++ head/net/xorp/files/patch-bgp__path_attribute.cc Fri Sep 11 22:51:10 2015 (r396720) @@ -1,6 +1,25 @@ --- bgp/path_attribute.cc.orig 2012-01-11 17:56:10 UTC +++ bgp/path_attribute.cc -@@ -830,8 +830,11 @@ PathAttribute * +@@ -45,18 +45,6 @@ + + template AttributeManager* PAListRef::_att_mgr = 0; + +-#ifdef DEBUG_LOGGING +-inline static void +-dump_bytes(const uint8_t* d, uint8_t l) +-{ +- for (uint16_t i = 0; i < l; i++) { +- debug_msg("%3u 0x%02x\n", i, d[i]); +- } +-} +-#else +-inline static void dump_bytes(const uint8_t*, uint8_t) {} +-#endif /* DEBUG_LOGGING */ +- + /* + * Flags values crib: + * +@@ -830,8 +818,11 @@ PathAttribute * ClusterListAttribute::clone() const { ClusterListAttribute *ca = new ClusterListAttribute(); Added: head/net/xorp/files/patch-bgp__route_table_policy.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/xorp/files/patch-bgp__route_table_policy.cc Fri Sep 11 22:51:10 2015 (r396720) @@ -0,0 +1,11 @@ +--- bgp/route_table_policy.cc.orig 2015-09-12 04:01:52.945569000 +0900 ++++ bgp/route_table_policy.cc 2015-09-12 04:06:26.219083000 +0900 +@@ -69,7 +69,7 @@ + bool no_modify) const + { + if (!_enable_filtering) { +- return &rtmsg; ++ return true; + } + + _varrw->attach_route(rtmsg, no_modify); Added: head/net/xorp/files/patch-libxipc-xrl_parser.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/xorp/files/patch-libxipc-xrl_parser.cc Fri Sep 11 22:51:10 2015 (r396720) @@ -0,0 +1,72 @@ +--- libxipc/xrl_parser.cc.orig 2012-01-11 17:56:10 UTC ++++ libxipc/xrl_parser.cc +@@ -105,18 +105,6 @@ XrlParseError::pretty_print(const size_t + + _reason + string("\n") + snapshot + string("\n") + indicator; + } + +-// ---------------------------------------------------------------------------- +-// Utility routines for Xrl parsing +- +-static inline void +-advance_to_either(const string& input, string::const_iterator& sci, +- const char* choices) { +- while (sci != input.end()) { +- if (strchr(choices,*sci)) break; +- sci++; +- } +-} +- + static inline void + advance_to_char(const string& input, string::const_iterator& sci, char c) + { +@@ -132,16 +120,6 @@ isxrlplain(int c) + return (xorp_isalnum(c) || c == '_' || c == '-'); + } + +-static inline size_t +-skip_xrl_plain_chars(const string& input, string::const_iterator& sci) +-{ +- string::const_iterator start = sci; +- for ( ; sci != input.end() && isxrlplain(*sci); sci++) { +- // XXX: Empty body +- } +- return sci - start; +-} +- + static inline char + c_escape_to_char(const string& input, + string::const_iterator sci) { +@@ -238,12 +216,6 @@ skip_past_blanks(const string& s, string + } + + static inline void +-skip_one_char(const string&, string::const_iterator* sci) +-{ +- sci++; +-} +- +-static inline void + skip_cplusplus_comments(const string& s, string::const_iterator& sci) + { + assert(*sci == '/'); +@@ -367,20 +339,6 @@ get_unquoted_value(const string& input, + token = string(sci_start, sci); + } + +-static inline string::const_iterator +-uninterrupted_token_end(const string& input, +- string::const_iterator& sci) +-{ +- string::const_iterator end = sci; +- +- while (end != input.end() && +- ( !xorp_isspace(*end) && isascii(*end) && !xorp_iscntrl(*end) )) { +- end++; +- } +- +- return end; +-} +- + static inline void + get_protocol_target_and_command(const string& input, + string::const_iterator& sci, Added: head/net/xorp/files/patch-libxorp-ref_trie.hh ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/xorp/files/patch-libxorp-ref_trie.hh Fri Sep 11 22:51:10 2015 (r396720) @@ -0,0 +1,25 @@ +--- libxorp/ref_trie.hh.orig 2015-09-12 03:45:51.752442000 +0900 ++++ libxorp/ref_trie.hh 2015-09-12 03:48:56.187304000 +0900 +@@ -1336,10 +1336,6 @@ + #ifdef DEBUG_LOGGING + debug_msg_indent(indent); + +- if (this == NULL) { +- debug_msg("%sNULL\n", msg); +- return; +- } + debug_msg("%skey: %s %s\n", + msg, _k.str().c_str(), _p ? "PL" : "[]"); + debug_msg(" U: %s\n", _up ? _up->_k.str().c_str() : "NULL"); +@@ -1356,10 +1352,7 @@ + RefTrieNode::str() const + { + string s; +- if (this == NULL) { +- s = "NULL"; +- return s; +- } ++ + s = c_format("key: %s ", _k.str().c_str()); + if (_p) + s += "PL"; Added: head/net/xorp/files/patch-rip-port.cc ============================================================================== --- /dev/null 00:00:00 1970 (empty, because file is newly added) +++ head/net/xorp/files/patch-rip-port.cc Fri Sep 11 22:51:10 2015 (r396720) @@ -0,0 +1,19 @@ +--- rip/port.cc.orig 2012-01-11 17:56:10 UTC ++++ rip/port.cc +@@ -44,16 +44,6 @@ + #include "output_updates.hh" + + // ---------------------------------------------------------------------------- +-// Utilities +- +-inline static uint32_t +-range_random(uint32_t lo, uint32_t hi) +-{ +- if (hi < lo) swap(hi, lo); +- return lo + ( xorp_random() % (hi - lo) ); +-} +- +-// ---------------------------------------------------------------------------- + // Address Family specific Port methods + + #ifdef INSTANTIATE_IPV4