search

dftlibs / xcfun

XCFun: A library of exchange-correlation functionals with arbitrary-order derivatives
https://dftlibs.org/xcfun/
Mozilla Public License 2.0
58 stars 32 forks source link

Update CMake infrastructure and remove explicit Makefiles #34

Closed bast closed 6 years ago

bast commented 6 years ago

I suggest to update CMake infrastructure to http://autocmake.readthedocs.io and to remove the explicit Makefiles to avoid code repetition and editing configuration in two places when refactoring code.

Before I embark on this, are there any opposing views on this?

robertodr commented 6 years ago

No objections from me!

On Dec 18, 2017 04:48, "Radovan Bast" notifications@github.com wrote:

I suggest to update CMake infrastructure to http://autocmake.readthedocs. io and to remove the explicit Makefiles to avoid code repetition and editing configuration in two places when refactoring code.

Before I embark on this, are there any opposing views on this?

— You are receiving this because you are subscribed to this thread. Reply to this email directly, view it on GitHub https://github.com/dftlibs/xcfun/issues/34, or mute the thread https://github.com/notifications/unsubscribe-auth/ADiXEd_oho_1Tt8bEw-ZX3PpL_fJnPFjks5tBjTVgaJpZM4RFNUY .

bast commented 6 years ago

OK - self-assigning. Will open a PR soon and it can still be stopped then but IMO two build frameworks in parallel are not long-term maintainable.