From 822d77804f5378acf559747e961f8178eafd9f3a Mon Sep 17 00:00:00 2001
From: Ben Frederickson <ben@benfrederickson.com>
Date: Mon, 16 Dec 2024 10:00:46 -0800
Subject: [PATCH] hack around breathe dependency issue

Breathe doesn't fully support sphinx v8 - it raises some warnings
in sphinx since its passing a `str` to Sphinx functions that
expect a `Path`. This still works for now (will be fully removed
in sphinx v9), but is deprecated - which is why breathe
indicates it only support sphinx<=v7.2 in conda.

However, nvidia-sphinx-theme only works with sphinx v8 afaict,
since it is passing a Path to `sphinx.util.fileutil.copy_asset_file`
- which breaks on sphinx v7 since it's expecting a `str`.

Hack around this dependency issue by installing breathe from pip,
which isn't as strict about supported sphinx versions.
---
 conda/environments/all_cuda-118_arch-aarch64.yaml | 2 +-
 conda/environments/all_cuda-118_arch-x86_64.yaml  | 2 +-
 conda/environments/all_cuda-125_arch-aarch64.yaml | 2 +-
 conda/environments/all_cuda-125_arch-x86_64.yaml  | 2 +-
 dependencies.yaml                                 | 2 +-
 5 files changed, 5 insertions(+), 5 deletions(-)

diff --git a/conda/environments/all_cuda-118_arch-aarch64.yaml b/conda/environments/all_cuda-118_arch-aarch64.yaml
index 01853da84..a6d98ea3b 100644
--- a/conda/environments/all_cuda-118_arch-aarch64.yaml
+++ b/conda/environments/all_cuda-118_arch-aarch64.yaml
@@ -7,7 +7,6 @@ channels:
 - conda-forge
 - nvidia
 dependencies:
-- breathe>=4.35.0
 - c-compiler
 - clang
 - clang-tools=16.0.6
@@ -57,5 +56,6 @@ dependencies:
 - sphinx>=8.0.0
 - sysroot_linux-aarch64==2.17
 - pip:
+  - breathe>=4.35.0
   - nvidia-sphinx-theme
 name: all_cuda-118_arch-aarch64
diff --git a/conda/environments/all_cuda-118_arch-x86_64.yaml b/conda/environments/all_cuda-118_arch-x86_64.yaml
index a1ad68d7f..1063e4d6c 100644
--- a/conda/environments/all_cuda-118_arch-x86_64.yaml
+++ b/conda/environments/all_cuda-118_arch-x86_64.yaml
@@ -7,7 +7,6 @@ channels:
 - conda-forge
 - nvidia
 dependencies:
-- breathe>=4.35.0
 - c-compiler
 - clang
 - clang-tools=16.0.6
@@ -57,5 +56,6 @@ dependencies:
 - sphinx>=8.0.0
 - sysroot_linux-64==2.17
 - pip:
+  - breathe>=4.35.0
   - nvidia-sphinx-theme
 name: all_cuda-118_arch-x86_64
diff --git a/conda/environments/all_cuda-125_arch-aarch64.yaml b/conda/environments/all_cuda-125_arch-aarch64.yaml
index ee0213fff..ee7b37695 100644
--- a/conda/environments/all_cuda-125_arch-aarch64.yaml
+++ b/conda/environments/all_cuda-125_arch-aarch64.yaml
@@ -7,7 +7,6 @@ channels:
 - conda-forge
 - nvidia
 dependencies:
-- breathe>=4.35.0
 - c-compiler
 - clang
 - clang-tools=16.0.6
@@ -53,5 +52,6 @@ dependencies:
 - sphinx>=8.0.0
 - sysroot_linux-aarch64==2.17
 - pip:
+  - breathe>=4.35.0
   - nvidia-sphinx-theme
 name: all_cuda-125_arch-aarch64
diff --git a/conda/environments/all_cuda-125_arch-x86_64.yaml b/conda/environments/all_cuda-125_arch-x86_64.yaml
index d93dcaf7a..7c8e1fd99 100644
--- a/conda/environments/all_cuda-125_arch-x86_64.yaml
+++ b/conda/environments/all_cuda-125_arch-x86_64.yaml
@@ -7,7 +7,6 @@ channels:
 - conda-forge
 - nvidia
 dependencies:
-- breathe>=4.35.0
 - c-compiler
 - clang
 - clang-tools=16.0.6
@@ -53,5 +52,6 @@ dependencies:
 - sphinx>=8.0.0
 - sysroot_linux-64==2.17
 - pip:
+  - breathe>=4.35.0
   - nvidia-sphinx-theme
 name: all_cuda-125_arch-x86_64
diff --git a/dependencies.yaml b/dependencies.yaml
index a11e59e31..a73fe7b8f 100644
--- a/dependencies.yaml
+++ b/dependencies.yaml
@@ -394,7 +394,6 @@ dependencies:
     common:
       - output_types: [conda]
         packages:
-          - breathe>=4.35.0
           - doxygen>=1.8.20
           - graphviz
           - ipython
@@ -405,6 +404,7 @@ dependencies:
           - sphinx-markdown-tables
           - pip:
             - nvidia-sphinx-theme
+            - breathe>=4.35.0
   rust:
     common:
       - output_types: [conda]