[GitHub] [tvm] mherkazandjian commented on a change in pull request #8239: use the latest tag instead of the master branch of papi in the ci

[GitBox <gi...@apache.org>]

mherkazandjian commented on a change in pull request #8239:
URL: https://github.com/apache/tvm/pull/8239#discussion_r649584353



##########
File path: docker/install/ubuntu_install_papi.sh
##########
@@ -26,7 +26,7 @@ apt-get update --fix-missing
 apt-get install -y linux-tools-common linux-tools-generic
 
 cd /
-git clone https://bitbucket.org/icl/papi.git
+git clone -b stable-6.0 --single-branch https://bitbucket.org/icl/papi.git

Review comment:
       i see. well, this is quite inconvenient at the moment. I think this error can be fixed by solving the papi compile issue
   in the papi repo, that is another solution, but i am not sure how long that will take them to fix.
   we could alternatively checkout the commit you refernced above instead of the stable tag (as a temporary fix)?
   that  would get the docker ci_cpu image build to succeed at least.
   




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

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