Skip to content

CMake: implement ability to export build tree#1149

Merged
ax3l merged 8 commits intoAMReX-Codes:developmentfrom
mic84:mr/cmake-export
Jul 22, 2020
Merged

CMake: implement ability to export build tree#1149
ax3l merged 8 commits intoAMReX-Codes:developmentfrom
mic84:mr/cmake-export

Conversation

@mic84
Copy link
Copy Markdown
Contributor

@mic84 mic84 commented Jul 17, 2020

No description provided.

@mic84 mic84 requested a review from ax3l July 17, 2020 00:58
@mic84
Copy link
Copy Markdown
Contributor Author

mic84 commented Jul 17, 2020

@sayerhs try this out and let me know if it works for you.

@mic84 mic84 marked this pull request as ready for review July 20, 2020 18:12
@mic84 mic84 changed the title [WIP] CMake: implement ability to export build tree CMake: implement ability to export build tree Jul 21, 2020
Copy link
Copy Markdown
Member

@ax3l ax3l left a comment

Choose a reason for hiding this comment

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

Thank you both, great work! ✨

@ax3l ax3l merged commit f87409c into AMReX-Codes:development Jul 22, 2020
@mic84 mic84 deleted the mr/cmake-export branch August 10, 2020 19:02
dwillcox pushed a commit to dwillcox/amrex that referenced this pull request Oct 3, 2020
* CMake: define C++ required flags as a property of target amrex

* CMake: fix handling of typechecker.

Just found out that:
  1) We did not install typechecker
  2) We do not need the typechecker path in the config file anymore

* CMake: enable exporting of build-tree

* CMake: linking against Flags_CXX_REQUIRED not needed anymore

* CMake: MSVC preprocessor flags must the public

* CMake: one more instance of Flags_CXX_REQUIRED to get rid of

* CMake: clean-up and add some comments

* Update Tools/CMake/AMReXInstallHelpers.cmake

Co-authored-by: Axel Huebl <[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