Remove USE_EXTERNAL_CLI11: it downloads CLI11 2.3.2 while the vendored header is 2.5.0
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 85/100
- issue の種類
- リファクタリング
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- cmake, cpp
- 領域
- build-system
調査の方向性
CMakeLists.txt の USE_EXTERNAL_CLI11 オプションと条件付きブロックを読み、次に cmake/CLI11.cmake と issue 内の 2 つの git grep コマンドで見つかった参照を調べます。オプションとモジュールを削除し、続いて記載されているビルド、ctest、フォーマットチェックのコマンドを実行します。リポジトリに含まれる CLI11 ヘッダーが使われ、オプションまたはモジュールへの参照が残っていなければ完了です。
索引モデルが issue の本文から書いたものです。
説明
ctrace parses its command line with the CLI11 header vendored in external/CLI11.hpp, version 2.5.0. With -DUSE_EXTERNAL_CLI11=ON (CMakeLists.txt:115-120), CMake downloads CLI11 v2.3.2 instead (cmake/CLI11.cmake): an older version that CI never builds, while the release license check reads the vendored header (CMakeLists.txt:209-214). The option gives two possible versions of the same dependency, and nothing needs it.
To do: remove the option and keep the vendored header.
How:
- delete
cmake/CLI11.cmake; - delete the
USE_EXTERNAL_CLI11option and itsifblock inCMakeLists.txt(lines 115 to 120); - run
git grep -n USE_EXTERNAL_CLI11andgit grep -n CLI11.cmaketo check that nothing else refers to them.
The wiki's build page also lists the option; a maintainer will update it.
Check:
cmake --build build --parallel 4
ctest --test-dir build --output-on-failure
./scripts/format-check.sh
Acceptance
cmake/CLI11.cmakeis gone, andgit grep USE_EXTERNAL_CLI11finds nothing.- The README build configures and builds, and
ctestpasses.
From the technical debt register (docs/quality/technical-debt.md, D8).
- 主要言語
- C++
- スター
- 5
- フォーク
- 4
- 平均マージ
- 1時間 9分
- マージ済み PR(30日)
- 81
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CoreTrace/coretrace のほかの issue
-
chore good first issue severity/minor
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
build severity/minor
難易度 2/5 1〜3時間 初心者へのやさしさ 84/100
CoreTrace/coretrace#176 · コメント 1 件 ·
メンテナーはふだん 1 日以内に返信
-
test: add unit tests for ctrace::RunDirectory対応中かも @tecnolgd が 2 日前に担当しました。 オープンgood first issue test
難易度 2/5 1〜3時間 初心者へのやさしさ 88/100
CoreTrace/coretrace#164 · コメント 2 件 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Release a macOS disk image (.dmg) for Apple silicon対応中かも @SizzleUnrlsd が今日担当しました。 オープンbuild enhancement
CoreTrace/coretrace#198 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Builds without a build type are unoptimized: the README's commands and CI's main job対応中かも @SizzleUnrlsd が 1 日前に担当しました。 オープンbuild severity/minor
CoreTrace/coretrace#192 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
CoreTrace/coretrace の issue をすべて見る
似ている issue
-
os:Linux
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 76/100
-
acceptable_buffer_backends: unknown backend names are silently ignored対応中かも @mini-1235 が今日担当しました。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 70/100
ros2/rmw_fastrtps#945 ·
メンテナーはふだん 1 日以内に返信
-
bot-triaged module: correctness (silent) module: mps module: NaNs and Infs triage review
難易度 2/5 半日 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 62/100
メンテナーはふだん 1 日以内に返信