Skip to content

Update conda version used in Appveyor #627

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

Merged
merged 26 commits into from
Nov 9, 2022

Conversation

martin-frbg
Copy link
Collaborator

@martin-frbg martin-frbg commented Oct 10, 2021

Description
fixes recent CI failures with internal conda errors like "InvalidVersionSpecError: Invalid version spec =2.7" (see also conda/conda issue 10618) and inability to install the "charset-normalizer" package
Checklist
N/A

Copy link
Collaborator

@weslleyspereira weslleyspereira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Hi @martin-frbg, thanks for the PR!

I observed another problem with recent runs of AppVeyor, e.g., https://ci.appveyor.com/project/langou/lapack/builds/41852332
Do you know if this PR also fix that behavior?

Preparing transaction: ...working... done
Verifying transaction: ...working... done
Executing transaction: ...working... failed
ERROR conda.core.link:_execute(502): An error occurred while installing package 'conda-forge::charset-normalizer-2.0.9-pyhd8ed1ab_0'.
CondaError: Cannot link a source that does not exist. C:\Miniconda36-x64\Scripts\conda.exe
Running `conda clean --packages` may resolve your problem.
Attempting to roll back.
Rolling back transaction: ...working... done
CondaError: Cannot link a source that does not exist. C:\Miniconda36-x64\Scripts\conda.exe
Running `conda clean --packages` may resolve your problem.
call "C:\Program Files (x86)\Microsoft Visual Studio 14.0\VC\vcvarsall.bat" amd64
set "LIB=%CONDA_INSTALL_LOCN%\Library\lib;%LIB%"
set "CPATH=%CONDA_INSTALL_LOCN%\Library\include;%CPATH%"
if (-Not (Test-Path .\build)) { mkdir build }
cd build
cmake -G "NMake Makefiles JOM" -DCMAKE_Fortran_COMPILER=flang -DCMAKE_BUILD_TYPE=Release -DBUILD_TESTING=ON ..
CMake Error at CMakeLists.txt:3 (project):
  The CMAKE_Fortran_COMPILER:
    C:/Miniconda36-x64/Library/bin/flang.exe
  is not a full path to an existing compiler tool.
  Tell CMake where to find the compiler by setting either the environment
  variable "FC" or the CMake cache entry CMAKE_Fortran_COMPILER to the full
  path to the compiler, or to the compiler name if it is in the PATH.
-- Configuring incomplete, errors occurred!
See also "C:/projects/lapack/build/CMakeFiles/CMakeOutput.log".
See also "C:/projects/lapack/build/CMakeFiles/CMakeError.log".
Command exited with code 1

@martin-frbg
Copy link
Collaborator Author

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

@martin-frbg martin-frbg reopened this Dec 14, 2021
@weslleyspereira
Copy link
Collaborator

Same problem. Thanks for rerunning it.

@martin-frbg
Copy link
Collaborator Author

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 martin-frbg reopened this Dec 15, 2021
@martin-frbg martin-frbg reopened this Dec 28, 2021
@weslleyspereira
Copy link
Collaborator

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

@martin-frbg
Copy link
Collaborator Author

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 martin-frbg closed this Nov 8, 2022
@martin-frbg martin-frbg reopened this Nov 8, 2022
@martin-frbg
Copy link
Collaborator Author

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.

@weslleyspereira
Copy link
Collaborator

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.

@martin-frbg
Copy link
Collaborator Author

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 martin-frbg changed the title Update conda before enabling conda-forge in Appveyor Update conda version used in Appveyor Nov 9, 2022
Removing comments.

Keep the one commented line that disabled automatic updates of conda itself.
@langou
Copy link
Contributor

langou commented Nov 9, 2022

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

Copy link
Collaborator

@weslleyspereira weslleyspereira left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks for the effort dedicated to this PR!

@martin-frbg
Copy link
Collaborator Author

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

@weslleyspereira
Copy link
Collaborator

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 weslleyspereira merged commit d7614a3 into Reference-LAPACK:master Nov 9, 2022
@julielangou julielangou added this to the LAPACK 3.10.1 milestone Nov 12, 2022
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

Successfully merging this pull request may close these issues.

4 participants