p8: bootstrap 残骸を撤去して #99 を closing#111
Merged
Merged
Conversation
FetchContent 移行 (#109) の仕上げ。 - build_tools/ubuntu.deps から masstree 旧 bootstrap.sh (autoreconf) 用の autoconf 依存を削除 - build_tools/bootstrap_apt.sh ラッパを削除 (ubuntu.deps と冗長) - docs/build_{ja,en}.md を submodule + bootstrap 前提から `cmake -S . -B build && cmake --build build` 完結のフローに更新 - docs/architecture_{ja,en}.md のレイアウト記述を FetchContent 反映 (third_party/ submodule 行を削除し、cmake/ThirdParty.cmake を追記) - README.md / .devcontainer/post-create.sh の bootstrap 案内も同期 Closes #99
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
1 participant
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Summary
build_tools/bootstrap_apt.shラッパ (ubuntu.depsをxargs apt-get installに投げるだけのスクリプト) を削除し、build_tools/ubuntu.depsは手動apt installでそのまま流せる shopping list として残置。docs/build_{ja,en}.mdを「3 本の bootstrap スクリプト + submodule init」前提からcmake -S . -B build && cmake --build build完結のフローに書き換え (CI キャッシュ説明のbootstrap output言及も同期)。docs/architecture_{ja,en}.mdのレイアウト記述・README.md・.devcontainer/post-create.shを FetchContent ベースに合わせて整合。PR #109 (MERGED) で完了済みの 3 ライブラリ FetchContent 移行 (masstree / mimalloc / googletest) の仕上げ。CLAUDE.md ハードルール#5 に従い
_ja/_enを同一 commit で更新。補足:
autoconfは残置当初
ubuntu.depsからautoconfも削除したが (commita848508)、cmake/ThirdParty.cmake(L66-L77) が今も masstree 上流の./bootstrap.sh(= autoreconf) をadd_custom_command経由で起動するためautoconfは 依然として必須。host build が壊れるので follow-up commit (c759ceb) で復活させた。ubuntu.depsは PR #111 前と同内容。フォローアップ提案 (別 issue 化想定)
masstree 上流には commit 済みの
configure(autoreconf 出力) が同梱されており、bootstrap.sh が呼ぶ autoreconf は本来不要。ThirdParty.cmakeの./bootstrap.sh呼び出しを撤去して./configureを直叩きにすれば、ubuntu.deps/ devcontainer Dockerfile の双方からautoconf(および間接依存のautomake/libtool/pkg-config) を落とせる。Issue #99 のスコープを越えるので本 PR には含めず別 issue で対応する。Test plan
docs/build_{ja,en}.md/docs/architecture_{ja,en}.mdが submodule / bootstrap 言及なしの記述に揃っていること (_ja/_enセット)buildジョブがグリーン (devcontainer image 経由).github/workflows/format.ymlはdocs/**/.devcontainer/**をpaths-ignoreにしているので、本 PR ではトリガーされないことを確認ubuntu.depsの内容が PR 前と同じ (autoconf 行を含む) ことCloses #99