You are viewing a plain text version of this content. The canonical link for it is here.
Posted to issues@trafficserver.apache.org by "Leif Hedstrom (Created) (JIRA)" <ji...@apache.org> on 2012/02/19 05:10:02 UTC

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

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

        

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

Posted by "Brian Geffon (Commented) (JIRA)" <ji...@apache.org>.
    [ 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] [Reopened] (TS-1116) Fix build issues with clang (particularly on OSX)

Posted by "Igor Galić (Reopened JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Igor Galić reopened TS-1116:
----------------------------

      Assignee: Brian Geffon  (was: Igor Galić)

assigning to brian, since he's taking care of 3.0.x now ;)
                
> 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)

Posted by "Brian Geffon (Commented) (JIRA)" <ji...@apache.org>.
    [ 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)

Posted by "Igor Galić (Commented JIRA)" <ji...@apache.org>.
    [ 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)

Posted by "Leif Hedstrom (Commented) (JIRA)" <ji...@apache.org>.
    [ 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<ts::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::IpMapBase<ts::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::IpMapBase<ts::detail::Ip6Node>::lowerBound' requested here
  N* n = this->lowerBound(min); // current node.
               ^
IpMap.cc:1260:21: note: in instantiation of member function 'ts::detail::IpMapBase<ts::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

        

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

Posted by "Brian Geffon (Issue Comment Edited) (JIRA)" <ji...@apache.org>.
    [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:comment-tabpanel&focusedCommentId=13252764#comment-13252764 ] 

Brian Geffon edited comment on TS-1116 at 4/12/12 7:41 PM:
-----------------------------------------------------------

Ok, i'll make the final vote for it and it will be included in 3.0.5.
                
      was (Author: briang):
    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] [Resolved] (TS-1116) Fix build issues with clang (particularly on OSX)

Posted by "Brian Geffon (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Brian Geffon resolved TS-1116.
------------------------------

       Resolution: Fixed
    Fix Version/s: 3.0.5

Backported to 3.0.x in commit 944d5babae425706e6238904ee2df621a68ef010
                
> 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.0.5, 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] [Resolved] (TS-1116) Fix build issues with clang (particularly on OSX)

Posted by "Igor Galić (Resolved JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Igor Galić resolved TS-1116.
----------------------------

    Resolution: Fixed
      Assignee: Igor Galić  (was: Leif Hedstrom)

http://git-wip-us.apache.org/repos/asf?p=trafficserver.git;a=commit;h=23af5c5b56d8d3c4d61ace903908f98ec3b7d12a
http://git-wip-us.apache.org/repos/asf?p=trafficserver.git;a=commit;h=fe1da80b94d368343174ada8b137a0c4ae1c387e
http://git-wip-us.apache.org/repos/asf?p=trafficserver.git;a=commit;h=56c29c0a1dd950eba82039f3f4dcb950a1fdff6b
                
> 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: Igor Galić
>             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] [Updated] (TS-1116) Fix build issues with clang (particularly on OSX)

Posted by "Igor Galić (Updated JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Igor Galić updated TS-1116:
---------------------------

      Affects Version/s: 3.0.4
                         3.0.3
    Backport to Version: 3.0.5
    
> 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: Igor Galić
>             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)

Posted by "Jan-Frode Myklebust (Commented) (JIRA)" <ji...@apache.org>.
    [ 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] [Updated] (TS-1116) Fix build issues with clang (particularly on OSX)

Posted by "Leif Hedstrom (Updated) (JIRA)" <ji...@apache.org>.
     [ https://issues.apache.org/jira/browse/TS-1116?page=com.atlassian.jira.plugin.system.issuetabpanels:all-tabpanel ]

Leif Hedstrom updated TS-1116:
------------------------------

    Attachment: TS-1116.diff

This fixes the build problems with clang. Alan, is this the right solution?
                
> 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
>
>         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