coretrace-log is downloaded twice: declared as coretrace-logger here and coretrace_logger by the compiler
メンテナーはふだん 1 日以内に返信
まだ誰も着手していません。
評価
- 難易度
- 2/5
- 見積もり時間
- 1〜3時間
- 初心者へのやさしさ
- 84/100
- issue の種類
- バグ
- 明瞭さ
- 明確に書かれている
- 活発さ
- 活発
- 技術スタック
- cmake, cpp
- 領域
- build-system
調査の方向性
まず cmake/logger/coretraceLog.cmake:18 を確認し、次に固定されたコンパイラのコミットで CMakeLists.txt:100 と CMakeLists.txt:151-163 の FetchContent 宣言を比較します。logger の宣言名を coretrace_logger に変更します。クリーンなビルドで _deps/*logger*-src が1つだけあることを確認し、Linux と macOS でビルドと ctest を実行します。
索引モデルが issue の本文から書いたものです。
説明
cmake/logger/coretraceLog.cmake:18 declares coretrace-log as coretrace-logger (v1.0.0). coretrace-compiler, which the analyzers fetch, declares the same repository as coretrace_logger, and calls FetchContent_Populate before checking whether the target already exists (its CMakeLists.txt:151-163, at the pinned commit 866fa76). A fresh build therefore downloads the logger twice, into build/_deps/coretrace-logger-src and build/_deps/coretrace_logger-src.
Both copies are the same commit today (v1.0.0 is 624688a), but nothing keeps them equal. Two declarations of this dependency already broke the build once (#37, #40).
Fix
Declare it as coretrace_logger, the name the stack analyzer and coretrace-compiler use. FetchContent keeps the first declaration of a name, and coretrace declares it first (CMakeLists.txt:100). FetchContent_GetProperties(coretrace_logger) in the compiler then finds it populated and downloads nothing.
Acceptance
- A fresh build directory holds a single
_deps/*logger*-src. - The build and
ctestpass on Linux and macOS.
From the technical debt register (docs/quality/technical-debt.md, D9).
- 主要言語
- C++
- スター
- 5
- フォーク
- 4
- 平均マージ
- 1時間 57分
- マージ済み PR（30日）
- 88
環境構築
- Dockerfile・Docker Compose ファイルなし
- プルリクエストのテンプレートなし
- コントリビューションガイドを読む
はじめの一歩
- issue を最後まで読み、次にプロジェクトのコントリビューションガイドを読みます。
- 着手することを issue にコメントします — 二人が同じ作業をするのを防げます。
- リポジトリをフォークし、ブランチを切って変更します。
- issue 番号を参照したプルリクエストを送ります。
CoreTrace/coretrace のほかの issue
-
chore good first issue severity/minor
難易度 1/5 1時間未満 初心者へのやさしさ 90/100
メンテナーはふだん 1 日以内に返信
-
build good first issue severity/minor
難易度 2/5 1〜3時間 初心者へのやさしさ 85/100
メンテナーはふだん 1 日以内に返信
-
Inputs: a file name starting with @ may be read as a response file by the compiler対応中かも @SizzleUnrlsd が今日担当しました。 オープンbug severity/minor
CoreTrace/coretrace#203 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
Builds without a build type are unoptimized: the README's commands and CI's main job対応中かも @SizzleUnrlsd が 2 日前に担当しました。 オープンbuild severity/minor
CoreTrace/coretrace#192 · 担当者 1 名 ·
メンテナーはふだん 1 日以内に返信
-
bug severity/minor
難易度 3/5 1〜2日 初心者へのやさしさ 50/100
メンテナーはふだん 1 日以内に返信
CoreTrace/coretrace の issue をすべて見る
似ている issue
-
Mac mini M6 (16 GB): Q2_0 streaming works - measured 11.7->27.2 tok/s + arena fix for sub-32 GB Macsオープン
難易度 2/5 1〜3時間 初心者へのやさしさ 68/100
メンテナーはふだん 1 日以内に返信
-
champion good first issue remake source/qq
難易度 2/5 1〜3時間 初心者へのやさしさ 65/100
metasequoiaime/msime-windows#1025 ·
メンテナーはふだん 1 日以内に返信
-
backlog bug
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
EchoTools/nevr-runtime#653 ·
メンテナーはふだん 1 日以内に返信
-
`--fail-require-skip` never fires: run_tests.py searches stdout, but the unittest binary writes the skip summary to stderr対応中かも このイシューにリンクされたプルリクエストがオープン中、またはマージ済みです。 オープン
難易度 2/5 1〜3時間 初心者へのやさしさ 72/100
メンテナーはふだん 1 日以内に返信
-
難易度 2/5 1〜3時間 初心者へのやさしさ 75/100
InfiniTimeOrg/InfiniTime#2492 ·