conda-forge / faunus-feedstock

A conda-smithy repository for faunus.
BSD 3-Clause "New" or "Revised" License
1 stars 4 forks source link

Upgrade to version 2.3.0 #8

Closed mlund closed 4 years ago

mlund commented 4 years ago

Checklist

conda-forge-linter commented 4 years ago

Hi! This is the friendly automated conda-forge-linting service.

I just wanted to let you know that I linted all conda-recipes in your PR (recipe) and found it was in an excellent condition.

mlund commented 4 years ago

@conda-forge-admin, please rerender

conda-forge-linter commented 4 years ago

Hi! This is the friendly automated conda-forge-webservice.

I tried to re-render for you but ran into some issues, please ping conda-forge/core for further assistance. You can also try re-rendering locally.

mlund commented 4 years ago

I requested re-rendering, but it failed. Also, re-rendering with conda smithy -c auto fails due to

conda_build_config.yaml is ambiguous because it does not fully implement all zipped keys, or specifies a subspace that is not fully implemented

It's been almost a year since last update. Do I need to update to match the newer build system? ping @conda-forge/core

mbargull commented 4 years ago

@conda-forge-admin, please rerender

mbargull commented 4 years ago

The error was due to the zipped_keys from https://github.com/conda-forge/conda-forge-pinning-feedstock/blob/f6e8a2fb2eb19a98b26474efc1fb89a6bf6bcc17/recipe/conda_build_config.yaml#L132-L135 for Windows. Instead of https://github.com/conda-forge/faunus-feedstock/pull/8/commits/4ac89b019742ff4e022df6c1db3acc6451f14ee4 we could've also changed the conda_build_config.yaml to

python:    # [not win]
    - 3.6  # [not win]
    - 3.7  # [not win]

but I opted for the skip: True # [win or py<36] line so that this can get Python 3.8 without further changes, when needed. (This is assuming it's compatible with Python 3.8.)