Use conda ar rather than system one

AnacondaRecipes · May 16, 2024 · c4c66c7 · c4c66c7
1 parent 3269d16
commit c4c66c7
Show file tree

Hide file tree

Showing 2 changed files with 18 additions and 0 deletions.
diff --git a/recipe/0002-use-conda-ar-not-system.patch b/recipe/0002-use-conda-ar-not-system.patch
@@ -0,0 +1,16 @@
+This fixes an issue where the build fails on linux-aarch64, because a system ar
+can't be found. Presumably it's using the system ar on other platforms, too,
+which isn't ideal.
+Index: nccl/src/Makefile
+===================================================================
+--- nccl.orig/src/Makefile	2024-05-16 16:58:16.522514975 -0500
++++ nccl/src/Makefile	2024-05-16 17:47:45.959416166 -0500
+@@ -83,7 +83,7 @@
+ $(LIBDIR)/$(STATICLIBTARGET): $(LIBOBJ) $(DEVMANIFEST)
+ 	@printf "Archiving  %-35s > %s\n" $(STATICLIBTARGET) $@
+ 	mkdir -p $(LIBDIR)
+-	ar cr $@ $(LIBOBJ) $$(cat $(DEVMANIFEST))
++	$(AR) cr $@ $(LIBOBJ) $$(cat $(DEVMANIFEST))
+
+ $(PKGDIR)/nccl.pc : nccl.pc.in
+ 	mkdir -p $(PKGDIR)
diff --git a/recipe/meta.yaml b/recipe/meta.yaml
@@ -11,6 +11,7 @@ source:
   patches:
     # This isn't needed; setting CUDA_HOME sets NVCC correctly
     #- 0001-Allow-custom-NVCC-path.patch
+    - 0002-use-conda-ar-not-system.patch
 
 build:
   number: 0
@@ -28,6 +29,7 @@ requirements:
     - {{ compiler("cxx") }}
     - {{ compiler("cuda") }}
     - make
+    - patch
   host:
     - cuda-version ={{ cuda_compiler_version }}          # [(cuda_compiler_version or "").startswith("11")]
   run: