Skip to content

fixed version-printout and flags print-out

Nick R. Papior requested to merge npapior/siesta:263-backport into rel-5.0

fixed version-printout and flags print-out

The print-out of version, libraries and flags has been updated.

The Src/SIESTA_cmake_vgen.sh script has been renamed to Tools/version_generate.sh

The SIESTA prefix is superfluous, and the placement is now better.

The build for the various version implementations are now much simpler, and should not require any copying of files (which can be error-prone). Utilites who wants to write out a version and associated flags/ppflags can now follow the quite simple recipe in the TS/TBtrans utility.

It boils down to a couple of things:

  1. translating the FLAGS into a string
  2. extracting the FPPFLAGS for the executable (not all gathering)
  3. translate the FPPFLAGS into a string
  4. configure_file the Src/version-info-template.inc into the current utility's binary-dir
  5. add the binary-dir to the include directories

The above will only use the siesta-used files for everything. So no code duplication etc. One could argue that it should be done in a library. However, the different utilities may use different compile definitions, hence different capabilities. For now, I think this is the simplest, yet very effective way to propagate how its shown to all utilities.

All uncessesary files have been deleted.

The Docs/{release.sh,Makefile} are updated to respect this change as well.

As for the building, now Siesta prints out the error messages a bit more stricter, with some more clear indication of how it interprets the versions.

This fixes #353 (closed), #355 (closed).

Signed-off-by: Nick Papior nickpapior@gmail.com (cherry picked from commit 6ba44264)

Merge request reports