Missing include statements for PowerPC in ec_get_cycles.c

ecmwf-ifs / fiat

The Fortran IFS and Arpege Toolkit

Apache License 2.0

7 stars 19 forks source link

Missing include statements for PowerPC in ec_get_cycles.c #1

Closed samhatfield closed 2 years ago

samhatfield commented 2 years ago

When building fiat on Summit, I get errors relating to the FILE and PATH_MAX constants. These are defined in stdio.h and linux/limits.h so #include statements need to be added. This code is only compiled when __powerpc__ is defined, hence why it probably wasn't spotted before.

wdeconinck commented 2 years ago

That is great Sam! And congrats for the first public issue :) Please fork and create a pull-request to "develop" with fix.

samhatfield commented 2 years ago

Done! https://github.com/ecmwf-ifs/fiat/pull/2

samhatfield commented 2 years ago

Closed by https://github.com/ecmwf-ifs/fiat/pull/2.