Opened 7 years ago

Closed 5 years ago

#238 closed enhancement (wontfix)

bld.cfg preprocessor non necessary lines

Reported by: ajornet Owned by: jgipsl
Priority: trivial Milestone:
Component: Driver files Version: trunc
Keywords: Cc:

Description

The following lines are not necessary to make sure the Fortran pre-processor works. Because:

  • It prevents the use of Compiler predefined Macros.
  • It is more convenient to let the compilers do their job instead of FCM (it was not meant to replace compiler's task).
# Pre-process code before analysing dependencies
bld::pp              1  # Regarding documentation 1 is not a valid value (true)
infile_ext::f90 fpp::fpp9x::source

Once, deactivated it is necessary to make sure to change all Fortran files to uppercase when needed. The Fortran standard says the Compiler has to apply the Preprocessor when the file extension is F90 (instead of f90).

Check here for more info: http://metomi.github.io/fcm/doc/user_guide/build.html

Change History (2)

comment:1 Changed 5 years ago by jgipsl

  • Milestone orchidee_1_9_6 deleted
  • Owner changed from somebody to jgipsl
  • Status changed from new to accepted

I think these lines were added so that we could keep the original file names with extension *f90. Nearly all orchidee modules have suffix .f90. Note, it will be difficult to handle different versions and merge.

To be discussed in group.

comment:2 Changed 5 years ago by jgipsl

  • Resolution set to wontfix
  • Status changed from accepted to closed

ORCHIDEE Day 2017 : this will not be done. It would be a too big difficulty to handle merge with other version due to change in suffix F90.

Note: See TracTickets for help on using tickets.