Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

BUG: Binaries not found when using cache: #1077

Closed
h-vetinari opened this issue Oct 2, 2024 · 2 comments
Closed

BUG: Binaries not found when using cache: #1077

h-vetinari opened this issue Oct 2, 2024 · 2 comments

Comments

@h-vetinari
Copy link

h-vetinari commented Oct 2, 2024

In (a reduced form of) conda-forge/zlib-feedstock#83, I'm running into the build not finding cmake, despite it clearly being there, both in the minimized recipe.yaml,

context:
  version: 1.3.1
  build_num: 2

recipe:
  name: zlib-split
  version: ${{ version }}

source:
  url:
    - http://zlib.net/zlib-${{ version }}.tar.gz
    - https://gnupg.org/ftp/gcrypt/zlib/zlib-${{ version }}.tar.gz
  sha256: 9a93b2b7dfdac77ceba5a558a580e74667dd6fede4585b91eefb60f03b72df23

build:
  number: ${{ build_num }}

cache:
  requirements:
    build:
      - cmake
      - ninja
      - ${{ compiler('c') }}
      - ${{ stdlib('c') }}
  build:
    script:
      # rattler-build figures out the {.sh,.bat} extensions
      - build_global

outputs:
  - package:
      name: libzlib

  - package:
      name: zlib

as well as the logs, e.g.

 │ │ Resolving build environment:
 │ │   Platform: win-64 [__win=0=0, __archspec=1=x86_64_v4]
 │ │   Channels: 
 │ │    - file:///D:/bld/
 │ │    - conda-forge
 │ │   Specs:
 │ │    - cmake    # <---
 │ │    - ninja
 │ │    - vs2019_win-64
 │ │    - vs_win-64
 │ │ [...]
 │ │ 'cmake' is not recognized as an internal or external command,
 │ │ operable program or batch file.

The build_global scripts here effectively just call cmake:

#!/bin/bash
set -ex

mkdir build
cd build

cmake -G Ninja \
    ..
@echo on

cmake -G "NMake Makefiles" ^
      %CMAKE_ARGS% %SRC_DIR%
if errorlevel 1 exit 1
@h-vetinari
Copy link
Author

As of the latest rattler-build (and overall conda-forge CI setup), this problem reproduces identically on all platforms.

@wolfv
Copy link
Member

wolfv commented Oct 2, 2024

The problem is that we are actually missing the installation part due to a recent refactor :)

Here is the fix: #1078

@wolfv wolfv closed this as completed Oct 2, 2024
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

No branches or pull requests

2 participants