From 5545c6c018125471f90b044353510fd40207005c Mon Sep 17 00:00:00 2001 From: Mike Alfare <13974384+mikealfare@users.noreply.github.com> Date: Thu, 29 Feb 2024 22:29:20 -0500 Subject: [PATCH] Add MacOS dependencies for `lxml` for `dbt-redshift` (#379) * add mac os dependencies for lxml for dbt-redshift * add mac os dependencies for lxml for dbt-redshift * add comment to retain context --- .github/formula-template.j2 | 11 +++++++++++ 1 file changed, 11 insertions(+) diff --git a/.github/formula-template.j2 b/.github/formula-template.j2 index bfea54b3..4c10f097 100644 --- a/.github/formula-template.j2 +++ b/.github/formula-template.j2 @@ -16,6 +16,17 @@ class {{ classname }} < Formula {%- endif %} depends_on "python@3.9" +{# + Addresses failed runs like this one: + https://github.com/dbt-labs/homebrew-dbt/actions/runs/8088432639/job/22102463432#step:17:88 + We need to indicate that `lxml` depends on the os dependencies `libxml2` and `libxslt`. + This needs to occur before we try to import the resource (e.g. at the top of the file). +#} +{%- if package in ('dbt-redshift') %} + uses_from_macos "libxml2" + uses_from_macos "libxslt" +{%- endif %} + {{ resources }} def install