Skip to content

replace unix with is_unix to avoid intel fpp collisions - #894

Merged
perazz merged 1 commit into
fortran-lang:mainfrom
perazz:fix_ifort_fpp_unix
Apr 27, 2023
Merged

replace unix with is_unix to avoid intel fpp collisions#894
perazz merged 1 commit into
fortran-lang:mainfrom
perazz:fix_ifort_fpp_unix

Conversation

@perazz

@perazz perazz commented Apr 27, 2023

Copy link
Copy Markdown
Member

Straightforward variable name change to solve #893.

Builds fine now:

root@7ce4928f525f:~/fpm# fpmx build --verbose
 <INFO> BUILD_NAME: build/ifort
 <INFO> COMPILER:  ifort
 <INFO> C COMPILER:  icc
 <INFO> CXX COMPILER: icpc
 <INFO> COMPILER OPTIONS:
 -fpp -warn all -check all -error-limit 1 -O0 -g -assume byterecl -traceback
 <INFO> C COMPILER OPTIONS:
 <INFO> CXX COMPILER OPTIONS:
 <INFO> LINKER OPTIONS:
 <INFO> INCLUDE DIRECTORIES:  []
[100%] Project compiled successfully.
root@7ce4928f525f:~/fpm#

@perazz

perazz commented Apr 27, 2023

Copy link
Copy Markdown
Member Author

Thank you @awvwgk. I will wait for another day or so and merge if there are no further comments.

@arteevraina arteevraina left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

👍

@perazz

perazz commented Apr 27, 2023

Copy link
Copy Markdown
Member Author

Thank you both @awvwgk @arteevraina, I believe I can merge this now.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants