[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-tabpanel&focusedCommentId=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-04-12 Thread Brian Geffon (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=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 Brian Geffon (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=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 Jan-Frode Myklebust (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=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-02-18 Thread Leif Hedstrom (Commented) (JIRA)

[ 
https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=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::IpMapBase::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::IpMapBase::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::IpMapBase::lowerBound' requested here
  N* n = this->lowerBound(min); // current node.
   ^
IpMap.cc:1260:21: note: in instantiation of member function 
'ts::detail::IpMapBase::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