Skip to content

fujitsu-mpi: Add methods, headers() and libs().#17253

Merged
alalazo merged 3 commits intospack:developfrom
t-karatsu:fujitsu-mpi/add_method
Jun 29, 2020
Merged

fujitsu-mpi: Add methods, headers() and libs().#17253
alalazo merged 3 commits intospack:developfrom
t-karatsu:fujitsu-mpi/add_method

Conversation

@t-karatsu
Copy link
Copy Markdown
Contributor

Add this feature to search the Fujitsu compiler libraries and header files.
In particular, mpi.h is stored deep in the include directory, so it needs to be searched recursively with find method.

@t-karatsu t-karatsu changed the title fujitsu-mpi: Add method, headers() and libs(). fujitsu-mpi: Add methods, headers() and libs(). Jun 26, 2020

@property
def headers(self):
hdrs = HeaderList(find(self.prefix.include, 'mpi.h', recursive=True))
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.

Suggested change
hdrs = HeaderList(find(self.prefix.include, 'mpi.h', recursive=True))
hdrs = find_headers('mpi', self.prefix.include, recursive=True)

Copy link
Copy Markdown
Contributor Author

Choose a reason for hiding this comment

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

@adamjstewart Thanks for review. I fixed it.

@alalazo alalazo merged commit 4425e83 into spack:develop Jun 29, 2020
@t-karatsu t-karatsu deleted the fujitsu-mpi/add_method branch June 29, 2020 07:02
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants