You are viewing a plain text version of this content. The canonical link for it is here.
Posted to github@trafficserver.apache.org by "masaori335 (via GitHub)" <gi...@apache.org> on 2023/08/31 02:11:07 UTC

[GitHub] [trafficserver] masaori335 opened a new pull request, #10319: Fix lua plugin build

masaori335 opened a new pull request, #10319:
URL: https://github.com/apache/trafficserver/pull/10319

   Fix #10318. We start using c++ compiler and the symbol `log` conflicts with `math.h`.
   
   ```
   lua/ts_lua_log.cc:21:24: error: redefinition of 'log' as different kind of symbol
   static TSTextLogObject log;
                          ^
   /opt/brew/opt/llvm@16/bin/../include/c++/v1/math.h:827:1: note: previous definition is here
   log(_A1 __x) _NOEXCEPT {return __builtin_log((double)__x);}
   ^
   ```


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] jpeach commented on pull request #10319: Fix lua plugin build

Posted by "jpeach (via GitHub)" <gi...@apache.org>.
jpeach commented on PR #10319:
URL: https://github.com/apache/trafficserver/pull/10319#issuecomment-1704482092

   Relevant reading for `<math.h>` symbols in C++
   
   https://developers.redhat.com/blog/2016/02/29/why-cstdlib-is-more-complicated-than-you-might-think
   https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89855


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org


[GitHub] [trafficserver] traeak merged pull request #10319: Fix lua plugin build

Posted by "traeak (via GitHub)" <gi...@apache.org>.
traeak merged PR #10319:
URL: https://github.com/apache/trafficserver/pull/10319


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

To unsubscribe, e-mail: github-unsubscribe@trafficserver.apache.org

For queries about this service, please contact Infrastructure at:
users@infra.apache.org