You are viewing a plain text version of this content. The canonical link for it is here.

Posted to notifications@thrift.apache.org by GitBox <gi...@apache.org> on 2022/09/14 15:22:08 UTC

[GitHub] [thrift] Jimexist commented on pull request #2588: THRIFT-5537: Remove python2

Jimexist commented on PR #2588:
URL: https://github.com/apache/thrift/pull/2588#issuecomment-1246931823

   > Thanks for doing this!
   > 
   > Those seem to be new errors related to python:
   > 
   > * https://app.travis-ci.com/github/apache/thrift/jobs/582437473
   > * https://app.travis-ci.com/github/apache/thrift/jobs/582437474
   > * https://app.travis-ci.com/github/apache/thrift/jobs/582437475
   > * https://app.travis-ci.com/github/apache/thrift/jobs/582437476
   
   @fishy turns out cmake was picking up python2 instead of python3 for these test runs. however i cannot get cmake version > 3.10 because by default it's the one shipped in bionic. i'm trying to upgrade to focal in #2528 but before that i guess this PR is a bit stuck unless we want to drop python tests in cmake for the moment


-- 
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: notifications-unsubscribe@thrift.apache.org

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