[GitHub] [trafficserver] maskit opened a new pull request, #10461: Make Cripts buildable with BoringSSL

["maskit (via GitHub)" <gi...@apache.org>]

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

   The type of return value is different on BoringSSL (int vs uint).
   ```
   Crypto.cc:173:28: error: comparison of integers of different signs: 'int' and 'unsigned int' [-Werror,-Wsign-compare]
     TSReleaseAssert(_key_len == EVP_CIPHER_key_length(_cipher)); // Make sure the crypto key was correct size
                     ~~~~~~~~ ^  ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
   ../../include/ts/ts.h:313:38: note: expanded from macro 'TSReleaseAssert'
   #define TSReleaseAssert(EX) ((void)((EX) ? (void)0 : _TSReleaseAssert(#EX, __FILE__, __LINE__)))
                                        ^~
   1 error generated.
   ```


-- 
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

Re: [PR] Make Cripts buildable with BoringSSL [trafficserver]

["maskit (via GitHub)" <gi...@apache.org>]

maskit merged PR #10461:
URL: https://github.com/apache/trafficserver/pull/10461


-- 
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