Skip to content
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

Fix CI again #304

Merged
merged 2 commits into from
Aug 3, 2022
Merged

Fix CI again #304

merged 2 commits into from
Aug 3, 2022

Conversation

Jean1995
Copy link
Member

@Jean1995 Jean1995 commented Aug 2, 2022

The CI has been broken for some time now. This should fix it now with the following steps:

  • Using the newest version of spdlog in our conanfile. With the old version, there was some problem with finding the fmt package. This caused the gcc builds to fail with the conan error:
    Could NOT find fmt: Found unsuitable version "None", but required is at
    least "5.3.0" (found None)
    
  • Using windows-latest for the CI runner. As desribed in issue Testing on latest Windows #262, we haven't been using the latest windows runner. However, on the old version, weird linker errors started to appear. They are gone with the newest version.
  • Disabling conan caches for the windows-runner. Building PROPOSAL and its dependencies works without a problem. However, reloading the caches fails with conan throwing the error message
    ERROR: Error, the file to download already exists: 'C:\Users\runneradmin\.conan\data\boost\1.78.0\_\_\dl\export   \conan_sources.tgz'. [Remote: conancenter]
    The 'boost/1.78.0' package has 'exports_sources' but sources not found in local cache.
    Probably it was installed from a remote that is no longer available.
    
    Therefore, the cache has been disabled for the windows build for now. This means that it will take an additional ~15 minutes to build the dependencies every time, but the windows tests take over three hours in total, so this isn't too severe in comparison. Still, it would be nice to fix this again, but for now this at least makes the CI run again.

@Jean1995 Jean1995 added the CI label Aug 2, 2022
@Jean1995 Jean1995 requested a review from sudojan August 2, 2022 19:58
@Jean1995 Jean1995 merged commit 6307588 into master Aug 3, 2022
@Jean1995 Jean1995 deleted the disable_conan_caches_on_windows branch August 3, 2022 07:28
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants