llnl / llnl/blt

Turn off tools by default

Open
#492 0 comments 0 reactions 0 assignees View on GitHub

Nobody has claimed this yet.

code checks
Dominant language
C++
Stars
296
Forks
66
Avg merge
1h 3m
Merged PRs (30d)
1

Description

e.g. clangformat, clangtidy, doxygen, etc.

This seems to be causing more problems then its solving. Originally it was thought that these were safe to be turned on by default because it would just find whatever was on the system and then the users would not call make style. But for one example, this has caused CMake to hang on a couple systems trying to determine information about the executable.

See: https://github.com/LLNL/axom/pull/568

Having them off by default and the codes can turn them on when in developer mode (or just in general) is safer.

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.

Research direction

Start by tracing the CMake configuration that enables clangformat, clangtidy, doxygen, and similar tools by default. Confirm that these tools are disabled in a normal configuration while remaining available when developer mode or an explicit option enables them; use the behavior described in LLNL/axom#568 as a regression reference.

Written by the indexing model from the issue text.

Assessment

Tech stack
cmake, cpp
Domain
build-system
Issue type
Feature
Difficulty
3/5
Estimated time
1-2 days
Activity status
Stale
Clarity
Mostly clear
Newbie friendliness
48/100

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.