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/11/04 16:40:01 UTC
[GitHub] [tvm] jtuyls opened a new pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
jtuyls opened a new pull request #9446:
URL: https://github.com/apache/tvm/pull/9446
Pin setuptools version to v58.4.0 to circumvent a breaking change in v58.5, see https://github.com/apache/tvm/issues/9445
cc @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.
To unsubscribe, e-mail: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] jtuyls commented on pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
Posted by GitBox <gi...@apache.org>.
jtuyls commented on pull request #9446:
URL: https://github.com/apache/tvm/pull/9446#issuecomment-961286035
> Thanks for fixing this @jtuyls, I think this is a sensible change but I suggest removing
>
> https://github.com/apache/tvm/blob/f656a2274aaac64489da1ece3ed110e214e549e7/docker/install/ubuntu_install_vela.sh#L23
>
> so we aren't surprised by the version of `setuptools` we get in the eventual image, I presume this is a high enough version of `setuptools` given it was working until a few days ago 😸
Done. And yeah, this is the last version that I could find that doesn't have the issue.
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] leandron merged pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
Posted by GitBox <gi...@apache.org>.
leandron merged pull request #9446:
URL: https://github.com/apache/tvm/pull/9446
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] jtuyls commented on pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
Posted by GitBox <gi...@apache.org>.
jtuyls commented on pull request #9446:
URL: https://github.com/apache/tvm/pull/9446#issuecomment-961286035
> Thanks for fixing this @jtuyls, I think this is a sensible change but I suggest removing
>
> https://github.com/apache/tvm/blob/f656a2274aaac64489da1ece3ed110e214e549e7/docker/install/ubuntu_install_vela.sh#L23
>
> so we aren't surprised by the version of `setuptools` we get in the eventual image, I presume this is a high enough version of `setuptools` given it was working until a few days ago 😸
Done. And yeah, this is the last version that I could find that doesn't have the issue.
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] jtuyls commented on pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
Posted by GitBox <gi...@apache.org>.
jtuyls commented on pull request #9446:
URL: https://github.com/apache/tvm/pull/9446#issuecomment-961286035
> Thanks for fixing this @jtuyls, I think this is a sensible change but I suggest removing
>
> https://github.com/apache/tvm/blob/f656a2274aaac64489da1ece3ed110e214e549e7/docker/install/ubuntu_install_vela.sh#L23
>
> so we aren't surprised by the version of `setuptools` we get in the eventual image, I presume this is a high enough version of `setuptools` given it was working until a few days ago 😸
Done. And yeah, this is the last version that I could find that doesn't have the issue.
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org
[GitHub] [tvm] leandron merged pull request #9446: [CI] Pin setuptools to v58.4.0 in CI to circumvent breaking change in v58.5
Posted by GitBox <gi...@apache.org>.
leandron merged pull request #9446:
URL: https://github.com/apache/tvm/pull/9446
--
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: commits-unsubscribe@tvm.apache.org
For queries about this service, please contact Infrastructure at:
users@infra.apache.org