Skip to content

CMake: Add SDFTD3_INSTALL_FMODDIR - #239

Open
Growl1234 wants to merge 1 commit into
dftd3:mainfrom
Growl1234:cmake
Open

Growl1234 wants to merge 1 commit into
dftd3:mainfrom
Growl1234:cmake

Conversation

@Growl1234

Copy link
Copy Markdown
Contributor

This is mainly for convenience of Fedora packaging as it requires Fortran modules to be installed under %{fmoddir}.

Note: We shouldn't simply adopt what's done for mctc-lib etc:

set(
  "${PROJECT_NAME}-module-dir"
  "${PROJECT_NAME}/modules"
  CACHE STRING
  "Subdirectory to install generated module files to"
)
set(
  module-dir
  "${${PROJECT_NAME}-module-dir}"
)
set(module-dir "${module-dir}" PARENT_SCOPE)

Downstream sets CMAKE_INSTALL_INCLUDEDIR to %{_lib}/gfortran/modules as a workaround, which is not applicable to s-dftd3 as it has public C header. This is also an issue on dftd4 and tblite side; will fix them soon.

@codecov

codecov Bot commented Oct 11, 2026 •

Copy link
Copy Markdown

Codecov Report

✅ All modified and coverable lines are covered by tests.
✅ Project coverage is 65.28%. Comparing base (5bf48d7) to head (89ded45).

Additional details and impacted files
@@            Coverage Diff             @@
##             main     #239      +/-   ##
==========================================
+ Coverage   63.49%   65.28%   +1.78%     
==========================================
  Files          47       40       -7     
  Lines        6821     5919     -902     
  Branches     2422     2422              
==========================================
- Hits         4331     3864     -467     
+ Misses       1182      747     -435     
  Partials     1308     1308              

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

🚀 New features to boost your workflow:
  • ❄️ Test Analytics: Detect flaky tests, report on failures, and find test suite problems.

@Growl1234
Growl1234 force-pushed the cmake branch 3 times, most recently from 2b7b939 to 89ded45 Compare October 12, 2026 16:24

This branch has not been deployed

No deployments
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.

1 participant