Skip to content

Include t8code package#39391

Merged
bernhardkaindl merged 2 commits intospack:developfrom
svengoldberg:t8code_package
Aug 23, 2023
Merged

Include t8code package#39391
bernhardkaindl merged 2 commits intospack:developfrom
svengoldberg:t8code_package

Conversation

@svengoldberg
Copy link
Copy Markdown
Contributor

This PR creates a new spack package for t8code, a library to manage parallel adaptive meshes with various element types.
Thanks for reviewing in advance!

@spackbot-app
Copy link
Copy Markdown

spackbot-app bot commented Aug 11, 2023

@Davknapp @melven can you review this PR?

This PR modifies the following package(s), for which you are listed as a maintainer:

  • t8code

@melven
Copy link
Copy Markdown
Contributor

melven commented Aug 11, 2023

@Davknapp @melven can you review this PR?

This PR modifies the following package(s), for which you are listed as a maintainer:

* t8code

Hi,

I looked into this together with @svengoldberg and I'm fine with it (of course).
So it's probably more helpful to have a review from some spack developer.

Copy link
Copy Markdown
Contributor

@bernhardkaindl bernhardkaindl left a comment

Choose a reason for hiding this comment

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

Reviewed, looks good. I applied minor updates:

  • Rebased to the current develop
    I'm hoping that this might get the readthedocs CI task working on the 2nd attempt.
  • Use 'spec = self.spec' to abbreviate self.spec as is custom practise
  • Use f-strings as these are fair game now to enjoy.

Tested with:

bin/spack install t8code +netcdf +metis +petsc

I could not test +vtk because vtk failed to build with

The variable SEACASIoss_INCLUDE_DIRS was expected to have been available

but this is an error in vtk, not t8code.

@bernhardkaindl bernhardkaindl enabled auto-merge (squash) August 23, 2023 22:29
@bernhardkaindl bernhardkaindl merged commit a95e061 into spack:develop Aug 23, 2023
mpokorny pushed a commit to mpokorny/spack that referenced this pull request Sep 18, 2023
* t8code: Create new spack package

Co-authored-by: Bernhard Kaindl <[email protected]>
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