- changed status to open
- removed comment
remove make.code.deps from GRHydro
GRHydro contains a file make.code.deps which tries to list the dependencies among its files due to Fortran modules. Naturally the file is out of date. Reading the documentation this should be harmless as make.code.deps should only give additional dependency information. However I just had a case where removing the file (and -cleandeps) helped. The actual fix could thus have been the cleandeps as well.
make.code.deps is not required in this case since Cactus can track module dependency on its own (see http://einsteintoolkit.org/documentation/UsersGuide/UsersGuidech9.html#x13-95000C1.6.2 and http://einsteintoolkit.org/documentation/UsersGuide/UsersGuidech9.html#x13-60000C1.2.5).
This patch removes the file.
Keyword: GRHydro
Comments (4)
-
reporter -
- changed status to open
- removed comment
Approved.
Note: - the variables defined in modules should probably have the SAVE attribute - including cctk_Arguments.h or cctk_Parameters.h is not necessary - the variable coeffs_allocated should probably be removed, as Fortran provides an ALLOCATED intrinsic
-
reporter - changed status to resolved
- removed comment
Applied in rev 506 and 507 of GRHydro (also remove include files and coeffs_allocated and adds SAVE attributes).
-
reporter - edited description
- changed status to closed
- Log in to comment