ocket8888 commented on code in PR #7841: URL: https://github.com/apache/trafficcontrol/pull/7841#discussion_r1379086155
########## traffic_ops/build/traffic_ops.spec: ########## @@ -38,7 +38,7 @@ Requires: openssl-devel, perl, perl-core, perl-DBD-Pg, perl-DBI, perl-Di Requires: libidn-devel, libcurl-devel, libcap Requires: postgresql13 >= 13.2 Requires: perl-JSON, perl-libwww-perl, perl-Test-CPAN-Meta, perl-WWW-Curl, perl-TermReadKey, perl-Crypt-ScryptKDF -Requires: python(abi) +Requires: python(abi), python3 Review Comment: Well it's always required, I'm guessing EL8+ just includes it? You're really not supposed to use the OS's special Python interpreter in any case. Also it looks like [you can get Python 3.8 through "AppStream"](https://www.server-world.info/en/note?os=CentOS_8&p=python&f=4) - whatever that is? Seems like a `dnf` command, so can we just trust the package manager to handle that? -- 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: issues-unsubscr...@trafficcontrol.apache.org For queries about this service, please contact Infrastructure at: us...@infra.apache.org
