llnl / llnl/smith

Turn back on Slic macros when Axom built release

Open
#592 2 comments 0 reactions 1 assignee View on GitHub

@white238 is already working on this.

Since Oct 24, 2022.

CS TPL
Dominant language
C++
Stars
245
Forks
36
Avg merge
8d 23h
Merged PRs (30d)
4

Description

If Axom is built not in `Debug` or `RelWithDebInfo`, Axom will not add `AXOM_DEBUG` to the core target and we will lose some of our Slic macros. This should be guarded against/forced on our side, until Axom works around the following issue:

https://github.com/LLNL/axom/issues/713

Contributor guide

Open the contributing guide

First steps

  1. Read the whole issue, then the project's contributing guide.
  2. Comment on the issue to say you are picking it up — it saves two people doing the same work.
  3. Fork the repository and make your change on a branch.
  4. Open a pull request that references the issue number.

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.