[jira] [Commented] (TS-1116) Fix build issues with clang (particularly on OSX)

2012-04-12 Thread Jan-Frode Myklebust (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13252322#comment-13252322
 ] 

Jan-Frode Myklebust commented on TS-1116:
-

Brian, please see patch in TS-1185. I've cherry picked commit 
9e3459a1006ad2e4381d20fe5374a11994dccf88 and 
cea3c71360a066a24b39e36ee116e83ea1db1bc8 into v3.0.4 to make it build on 
fedora-17 which has gcc-4.7. It would be great if you could include these 
patches in the upcoming v3.0.5 release.

 Fix build issues with clang (particularly on OSX)
 -

 Key: TS-1116
 URL: https://issues.apache.org/jira/browse/TS-1116
 Project: Traffic Server
  Issue Type: Improvement
Affects Versions: 3.0.4, 3.0.3
Reporter: Leif Hedstrom
Assignee: Brian Geffon
 Fix For: 3.1.3

 Attachments: TS-1116.diff


 We should get it to compile with clang again, specially since on OSX, clang 
 is the 'future'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1116) Fix build issues with clang (particularly on OSX)

2012-04-12 Thread Brian Geffon (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13252764#comment-13252764
 ] 

Brian Geffon commented on TS-1116:
--

I'll get it added to STATUS so voting can start on it.

 Fix build issues with clang (particularly on OSX)
 -

 Key: TS-1116
 URL: https://issues.apache.org/jira/browse/TS-1116
 Project: Traffic Server
  Issue Type: Improvement
Affects Versions: 3.0.4, 3.0.3
Reporter: Leif Hedstrom
Assignee: Brian Geffon
 Fix For: 3.1.3

 Attachments: TS-1116.diff


 We should get it to compile with clang again, specially since on OSX, clang 
 is the 'future'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1116) Fix build issues with clang (particularly on OSX)

2012-04-12 Thread Brian Geffon (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13252951#comment-13252951
 ] 

Brian Geffon commented on TS-1116:
--

@Jan-Frode Myklebust: all that will be included in 3.0.5 will be the commits 
referenced in the STATUS file which is only 
fe1da80b94d368343174ada8b137a0c4ae1c387e.

@Igor, should there be multiple SHAs in the STATUS file, should I be 
backporting based on your comment above / and or start a new vote?

 Fix build issues with clang (particularly on OSX)
 -

 Key: TS-1116
 URL: https://issues.apache.org/jira/browse/TS-1116
 Project: Traffic Server
  Issue Type: Improvement
Affects Versions: 3.0.4, 3.0.3
Reporter: Leif Hedstrom
Assignee: Brian Geffon
 Fix For: 3.1.3

 Attachments: TS-1116.diff


 We should get it to compile with clang again, specially since on OSX, clang 
 is the 'future'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1116) Fix build issues with clang (particularly on OSX)

2012-04-12 Thread Commented

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13252973#comment-13252973
 ] 

Igor Galić commented on TS-1116:


Brian,

I only added the SHAs which seemed sensible. The one for configure.in to add 
-Werror didn't seem useful for this particular bug.

 Fix build issues with clang (particularly on OSX)
 -

 Key: TS-1116
 URL: https://issues.apache.org/jira/browse/TS-1116
 Project: Traffic Server
  Issue Type: Improvement
Affects Versions: 3.0.4, 3.0.3
Reporter: Leif Hedstrom
Assignee: Brian Geffon
 Fix For: 3.1.3

 Attachments: TS-1116.diff


 We should get it to compile with clang again, specially since on OSX, clang 
 is the 'future'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira




[jira] [Commented] (TS-1116) Fix build issues with clang (particularly on OSX)

2012-02-18 Thread Leif Hedstrom (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanelfocusedCommentId=13211200#comment-13211200
 ] 

Leif Hedstrom commented on TS-1116:
---

I got most of it to compile, with the exception of this in IpMap.cc:

{code}
IpMap.cc:850:11: error: call to function 'operator' that is neither visible in 
the template definition nor found by argument-dependent lookup
if (x  n-_min) n = left(n);
  ^
IpMap.cc:1236:24: note: in instantiation of member function 
'ts::detail::IpMapBasets::detail::Ip6Node::contains' requested here
zret = _m6  _m6-contains(ink_inet_ip6_cast(target), ptr);
   ^
IpMap.cc:1164:13: note: 'operator' should be declared prior to the call site 
or in the global namespace
inline bool operator(sockaddr_in6 const* lhs, sockaddr_in6 const rhs) {
^
IpMap.cc:665:17: error: call to function 'operator==' that is neither visible 
in the template definition nor found by argument-dependent lookup
if (n-_min == min) {
^
IpMap.cc:1260:21: note: in instantiation of member function 
'ts::detail::IpMapBasets::detail::Ip6Node::mark' requested here
this-force6()-mark(ink_inet_ip6_cast(min), ink_inet_ip6_cast(max), data);
^
IpMap.cc:1172:13: note: 'operator==' should be declared prior to the call site 
or in the global namespace
inline bool operator==(sockaddr_in6 const lhs, sockaddr_in6 const* rhs) {
^
IpMap.cc:736:19: error: call to function 'operator=' that is neither visible 
in the template definition nor found by argument-dependent lookup
  if (n-_max = max_plus) { // completely covered, drop span, continue
  ^
IpMap.cc:1188:13: note: 'operator=' should be declared prior to the call site 
or in the global namespace
inline bool operator=(sockaddr_in6 const lhs, sockaddr_in6 const rhs) {
^
IpMap.cc:514:16: error: call to function 'operator' that is neither visible in 
the template definition nor found by argument-dependent lookup
if (target  n-_min) n = left(n);
   ^
IpMap.cc:640:16: note: in instantiation of member function 
'ts::detail::IpMapBasets::detail::Ip6Node::lowerBound' requested here
  N* n = this-lowerBound(min); // current node.
   ^
IpMap.cc:1260:21: note: in instantiation of member function 
'ts::detail::IpMapBasets::detail::Ip6Node::mark' requested here
this-force6()-mark(ink_inet_ip6_cast(min), ink_inet_ip6_cast(max), data);
^
IpMap.cc:1164:13: note: 'operator' should be declared prior to the call site 
or in the global namespace
inline bool operator(sockaddr_in6 const* lhs, sockaddr_in6 const rhs) {
^
4 errors generated.
{code}

I'll commit the other changes (after I test them on Linux), but I'll need 
assistant from some C++ propeller head with the template madness above :).

 Fix build issues with clang (particularly on OSX)
 -

 Key: TS-1116
 URL: https://issues.apache.org/jira/browse/TS-1116
 Project: Traffic Server
  Issue Type: Improvement
Reporter: Leif Hedstrom
Assignee: Leif Hedstrom
 Fix For: 3.1.3


 We should get it to compile with clang again, specially since on OSX, clang 
 is the 'future'.

--
This message is automatically generated by JIRA.
If you think it was sent incorrectly, please contact your JIRA administrators: 
https://issues.apache.org/jira/secure/ContactAdministrators!default.jspa
For more information on JIRA, see: http://www.atlassian.com/software/jira