areusch commented on code in PR #11644: URL: https://github.com/apache/tvm/pull/11644#discussion_r893859797
########## docker/install/ubuntu_install_papi.sh: ########## @@ -23,11 +23,15 @@ set -o pipefail apt-get update --fix-missing # deps -apt-get install -y linux-tools-common linux-tools-generic +apt-get install -y linux-tools-common linux-tools-generic kmod cd / git clone https://bitbucket.org/icl/papi.git -cd papi/src +# Pulling the latest version of this has broken the images before. Checkout the commit below for now. +cd papi +git checkout e3c263b6fb578695085d0ab093e4fd1ee5aeac23 Review Comment: did we want to pull a specific version rather than a sha1 that looks random to a new user? it would simplify install instructions. can you also update the docs in docs/install/from_source.rst to include this version constraint? -- 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: [email protected] For queries about this service, please contact Infrastructure at: [email protected]
