search

conda-forge / cupy-feedstock

A conda-smithy repository for cupy.
BSD 3-Clause "New" or "Revised" License
5 stars 23 forks source link

Handle unset CUDA_PATH even when set -u option active #69

Closed calebmarchent closed 4 years ago

calebmarchent commented 4 years ago

Fixes: #68

Test Plan: $ set -u $ conda activate test_cupy_path With fix applied we no longer get CUDA_PATH: unbound variable

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.

jakirkham commented 4 years ago

Yeah we won't get new packages without a build number bump 😉

leofang commented 4 years ago

@conda-forge-admin, please rerender

github-actions[bot] commented 4 years ago

Hi! This is the friendly automated conda-forge-webservice. I tried to rerender for you, but it looks like there was nothing to do.

github-actions[bot] commented 4 years ago

Hi! This is the friendly conda-forge automerge bot!

I considered the following status checks when analyzing this PR:

Thus the PR was passing and merged! Have a great day!

leofang commented 4 years ago

(Looks like the bot doesn't check existing PRs very frequently, so I clicked "rerun" to make the automerge label in effect. Just my little experiment 😁)

jakirkham commented 4 years ago

Yeah it doesn't run on PRs that already passed. We are on our own to merge those 😉