potiuk commented on code in PR #26217:
URL: https://github.com/apache/airflow/pull/26217#discussion_r965716689
##########
Dockerfile:
##########
@@ -92,7 +92,7 @@ fi
function get_dev_apt_deps() {
if [[ "${DEV_APT_DEPS=}" == "" ]]; then
DEV_APT_DEPS="apt-transport-https apt-utils build-essential
ca-certificates dirmngr \
-freetds-bin freetds-dev git gosu krb5-user ldap-utils libffi-dev \
+freetds-bin freetds-dev git gosu graphviz graphviz-dev krb5-user ldap-utils
libffi-dev \
Review Comment:
it's needed to install `eralchemy2`. It has only source packages but it
links to graphviz C libraries and needs to be compiled. One of the reasons why
adding it as depedency for "local" virtualenv might be a pain as it might not
compile properly in all environments - and that's where having our shared CI
image where we have those prerequisites is handy.
--
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]
