Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

Provide a proper mfem.m4 macro for the build system #83

Closed
koomie opened this issue Dec 10, 2021 · 1 comment
Closed

Provide a proper mfem.m4 macro for the build system #83

koomie opened this issue Dec 10, 2021 · 1 comment
Labels
enhancement New feature or request
Milestone
1.1

Comments

@koomie
Copy link
Contributor

koomie commented Dec 10, 2021

Ideally, interrogate MFEM installation for 3rd party libraries (e.g. hypre) that it was built against.
@koomie koomie added the enhancement New feature or request label Dec 10, 2021
@koomie koomie added this to the 1.1 milestone Dec 10, 2021
koomie added a commit that referenced this issue Dec 16, 2021
@koomie
adding new macro files to support autodetection of MFEM configuration…
17b2238 
…. Trigger

off of default MFEM_DIR env variable or user-supplied --with-mfem=<path>
option. Updated tps build system to use header and library variables set by
the new macro (#83).  Updated configure summary as will to cleanup long paths
for MFEM linkage.
koomie added a commit that referenced this issue Dec 21, 2021
@koomie
Merge pull request #85 from pecos/macro
01d80ff 
introduce usage of a mfem macro for build system (#83)
koomie added a commit that referenced this issue Dec 21, 2021
@koomie
add python requirement for autoconf (#83)
99cf8c2
@koomie
Copy link
Contributor Author

koomie commented Dec 21, 2021

A --with-mfem option has been added to provide the top-level path of the desired MFEM linkage. This macro will interrogate MFEM build collateral at $MFEM_DIR/share/mfem/config.mk to query which 3rd party libraries are needed with the local MFEM build (via a new python script); TPS build system updated accordingly to use MFEM_CXXFLAGS and MFEM_LIBS variables set by the new macro. Note that this addition does require availability of python3 locally.

@koomie koomie closed this as completed Dec 21, 2021
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
enhancement New feature or request
Projects
None yet
Milestone
1.1
Development

No branches or pull requests
1 participant
@koomie