You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@tvm.apache.org by GitBox <gi...@apache.org> on 2021/04/26 16:09:06 UTC
[GitHub] [tvm] Mousius opened a new pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Mousius opened a new pull request #7926:
URL: https://github.com/apache/tvm/pull/7926
We started seeing issues with building the TVM Docker images, and they stemmed from the SBT (Scala Build Tool) installation which was using bintray instead of a later SBT Debian URL. Thankfully bintray were just running some brown outs before they turn the service off on May 1st:
https://jfrog.com/blog/into-the-sunset-bintray-jcenter-gocenter-and-chartcenter/
So I took the Scala repo URL from here:
https://www.scala-sbt.org/1.x/docs/Installing-sbt-on-Linux.html
Which is the suggested SBT URL as they may change the backend again in future:
https://github.com/sbt/sbt/issues/6294
Thanks for contributing to TVM! Please refer to guideline https://tvm.apache.org/docs/contribute/ for useful information and tips. After the pull request is submitted, please request code reviews from [Reviewers](https://github.com/apache/incubator-tvm/blob/master/CONTRIBUTORS.md#reviewers) by @ them in the pull request thread.
@leandron
--
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
[GitHub] [tvm] Mousius commented on a change in pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
Mousius commented on a change in pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#discussion_r620461475
##########
File path: docker/install/ubuntu_install_sbt.sh
##########
@@ -25,7 +25,7 @@ set -o pipefail
apt-get update && apt-get install -y apt-transport-https
# Install the necessary dependencies for sbt
-echo "deb https://dl.bintray.com/sbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
+echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
Review comment:
Not a problem, thanks for pointing it out :smile_cat:
--
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
[GitHub] [tvm] eed3si9n commented on a change in pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
eed3si9n commented on a change in pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#discussion_r620451752
##########
File path: docker/install/ubuntu_install_sbt.sh
##########
@@ -25,7 +25,7 @@ set -o pipefail
apt-get update && apt-get install -y apt-transport-https
# Install the necessary dependencies for sbt
-echo "deb https://dl.bintray.com/sbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
+echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
Review comment:
```suggestion
echo "deb https://repo.scala-sbt.org/scalasbt/debian all main" | tee /etc/apt/sources.list.d/sbt.list
echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | tee /etc/apt/sources.list.d/sbt_old.list
```
--
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
[GitHub] [tvm] jcf94 merged pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
jcf94 merged pull request #7926:
URL: https://github.com/apache/tvm/pull/7926
--
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
[GitHub] [tvm] eed3si9n commented on a change in pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
eed3si9n commented on a change in pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#discussion_r620451752
##########
File path: docker/install/ubuntu_install_sbt.sh
##########
@@ -25,7 +25,7 @@ set -o pipefail
apt-get update && apt-get install -y apt-transport-https
# Install the necessary dependencies for sbt
-echo "deb https://dl.bintray.com/sbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
+echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
Review comment:
```suggestion
echo "deb https://repo.scala-sbt.org/scalasbt/debian all main" | sudo tee /etc/apt/sources.list.d/sbt.list
echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | sudo tee /etc/apt/sources.list.d/sbt_old.list
```
--
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
[GitHub] [tvm] eed3si9n commented on a change in pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
eed3si9n commented on a change in pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#discussion_r620452728
##########
File path: docker/install/ubuntu_install_sbt.sh
##########
@@ -25,7 +25,7 @@ set -o pipefail
apt-get update && apt-get install -y apt-transport-https
# Install the necessary dependencies for sbt
-echo "deb https://dl.bintray.com/sbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
+echo "deb https://repo.scala-sbt.org/scalasbt/debian /" | tee -a /etc/apt/sources.list.d/sbt.list
Review comment:
Sorry I haven't had time to update the docs but the new Debian packages are now under `all main`.
--
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
[GitHub] [tvm] jcf94 commented on pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
jcf94 commented on pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#issuecomment-828150342
Now merged. Thanks! @Mousius @leandron @eed3si9n @areusch
--
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
[GitHub] [tvm] leandron commented on pull request #7926: Use new SBT Debian Repo before bintray is shutdown
Posted by GitBox <gi...@apache.org>.
leandron commented on pull request #7926:
URL: https://github.com/apache/tvm/pull/7926#issuecomment-827926842
I guess this is now ready to be merged?
--
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