remove make.code.deps from GRHydro

Issue #1323 closed
Roland Haas created an issue

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)

  1. Erik Schnetter
    • 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

  2. Roland Haas 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).

  3. Log in to comment