lapack icon indicating copy to clipboard operation
lapack copied to clipboard

Update conda before enabling conda-forge in Appveyor

Open martin-frbg opened this issue 4 years ago • 3 comments

Description fixes recent CI failure "InvalidVersionSpecError: Invalid version spec =2.7" (see also conda/conda issue 10618) Checklist N/A

martin-frbg avatar Oct 10 '21 15:10 martin-frbg

I don't know, but closing and reopening this PR (to rerun the CI) should tell

martin-frbg avatar Dec 14 '21 18:12 martin-frbg

Same problem. Thanks for rerunning it.

weslleyspereira avatar Dec 14 '21 18:12 weslleyspereira

nope, seems to be a more recent problem. (the only reference I can find is from Oct 15, but only a githubmemory link with no accessible primary source)

martin-frbg avatar Dec 14 '21 18:12 martin-frbg

Hi @martin-frbg. Do you have any update on this thread? The current script for Appveyor is not working.

weslleyspereira avatar Nov 07 '22 22:11 weslleyspereira

I had pretty much given up on this - no time and energy to debug conda on top of everything else. The originally proposed fix used to work for a few weeks around when I created the PR... The appveyor.yml in OpenBLAS does not have this problem, but the difference in base images may explain that.

martin-frbg avatar Nov 08 '22 11:11 martin-frbg

Hmm, looks like it primarily needs the updated miniconda that is already preinstalled alongside the one currently used. (Plus dropping the build cache - once, I guess - so that cmake forgets about the old compiler location). The remaining build failure now appears to be a genuine accuracy error in the tests.

martin-frbg avatar Nov 09 '22 11:11 martin-frbg

Hmm, looks like it primarily needs the updated miniconda that is already preinstalled alongside the one currently used. (Plus dropping the build cache - once, I guess - so that cmake forgets about the old compiler location). The remaining build failure now appears to be a genuine accuracy error in the tests.

Great news!!

Should I remove the commented lines? Maybe it is a good idea to let those lines in case we need to test alternatives to work with future updates in conda.

weslleyspereira avatar Nov 09 '22 16:11 weslleyspereira

I guess we may want to keep the one commented line that disabled automatic updates of conda itself. The others are either options I moved into the remaining commands, or commands that did nothing to solve the problem. (And I notice that the title of this PR is outdated now as well)

martin-frbg avatar Nov 09 '22 16:11 martin-frbg

Thanks @martin-frbg for getting appveyor to work again!

langou avatar Nov 09 '22 17:11 langou

We'll probably need a follow-on to reenable the build cache once this is merged (and Appveyor has run once more)

martin-frbg avatar Nov 09 '22 17:11 martin-frbg

We'll probably need a follow-on to reenable the build cache once this is merged (and Appveyor has run once more)

Ok! I will merge it now, and we watch how it behaves. Thanks!

weslleyspereira avatar Nov 09 '22 17:11 weslleyspereira