Skip to content

Conversation

LecrisUT
Copy link
Collaborator

The generator expression used was only for 3.27.
Reproduced the error in: https://github.com/spglib/spglib/actions/runs/8045683520
Confirming the fix in: https://github.com/spglib/spglib/actions/runs/8045769170

Hot-fix: #437
Closes #444

Copy link

codecov bot commented Feb 26, 2024

Codecov Report

All modified and coverable lines are covered by tests ✅

Project coverage is 83.59%. Comparing base (f1ff014) to head (3bf659e).

Additional details and impacted files
@@           Coverage Diff            @@
##           develop     #447   +/-   ##
========================================
  Coverage    83.59%   83.59%           
========================================
  Files           24       24           
  Lines         8118     8118           
  Branches      1688     1693    +5     
========================================
  Hits          6786     6786           
  Misses        1332     1332           
Flag Coverage Δ
c_api 74.43% <ø> (ø)
fortran_api 56.16% <ø> (ø)
python_api 79.70% <ø> (ø)
unit_tests 13.47% <ø> (ø)

Flags with carried forward coverage won't be shown. Click here to find out more.

☔ View full report in Codecov by Sentry.
📢 Have feedback on the report? Share it here.

It should be used only with CMake >= 3.27

Signed-off-by: Cristian Le <cristian.le@mpsd.mpg.de>
Copy link
Member

@lan496 lan496 left a comment

Choose a reason for hiding this comment

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

I've also checked to configure with cmake<3.27 in my local environment.

@LecrisUT LecrisUT merged commit bcb95c3 into spglib:develop Feb 26, 2024
@LecrisUT LecrisUT deleted the fix/export branch February 29, 2024 10:48
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Status: Done
Development

Successfully merging this pull request may close these issues.

Fail to configure
2 participants