Linux-only feedstock gets macOS lint #2159
Comments
|
The processing of CBC doesn't know that a feedstock is linux-only. It already needs to juggle some selectors to find the right subset per platform. I'm doubtful that it's worth adding extra logic to determine whether we're on a skipped platform, when the solution is very simply to add a selector: c_stdlib_version: # [linux]
- "2.28" # [linux] |
|
This is the logic in case you want to wade into this fray: conda-smithy/conda_smithy/linter/lints.py Lines 840 to 885 in b3ad532 Linux-only recipes are pretty rare nowadays. I don't think it's an undue burden to add the selector (though I agree it'd be nice if the logic were smart enough to ignore skipped platforms; but it's already fragile enough that I don't want to add yet another layer of complexity for such a minimal gain). |
Solution to issue cannot be found in the documentation.
Issue
The linter has flagged a macOS error. Normally this would be reasonable to raise. However this is a Linux only feedstock
The lint is included below. Also here is a reference to the original comment: conda-forge/nccl-feedstock#120 (comment)
Installed packages
Environment info
micromamba info /home/runner/micromamba-bin/micromamba info -r /home/runner/micromamba -n webservices libmamba version : 2.0.2 micromamba version : 2.0.2 curl version : libcurl/8.10.1 OpenSSL/3.3.2 zlib/1.3.1 zstd/1.5.6 libssh2/1.11.0 nghttp2/1.58.0 libarchive version : libarchive 3.7.4 zlib/1.2.13 bz2lib/1.0.8 libzstd/1.5.6 envs directories : /home/runner/micromamba/envs package cache : /home/runner/micromamba/pkgs /home/runner/.mamba/pkgs environment : webservices env location : /home/runner/micromamba/envs/webservices user config files : /home/runner/.mambarc populated config files : /home/runner/work/_temp/setup-micromamba/.condarc virtual packages : __unix=0=0 __linux=6.5.0=0 __glibc=2.35=0 __archspec=1=x86_64_v3 channels : https://conda.anaconda.org/conda-forge/linux-64 https://conda.anaconda.org/conda-forge/noarch base environment : /home/runner/micromamba platform : linux-64The text was updated successfully, but these errors were encountered: