docs: add CLAUDE.md with build, test and architecture guidance - #189
Merged
Conversation
Documents the native and cross build commands as CI runs them, the CMake knobs that matter (the hard-reset of CMAKE_C_FLAGS, BUILD_SHARED_LIBS as the way to drop -static, IPCHW_VENDORS), the two hardware-free tests, the release tagging scheme, and the detection flow: getchipname() through the UART-base and /proc/cpuinfo tables, the HAL as global function pointers, chip_generation as the dispatch key, sensor probing over possible_i2c_addrs, and the STANDALONE_LIBRARY split between ipctool and libipchw.
Code Review by Qodo🐞 Bugs (0) 📘 Rule violations (0) 📎 Requirement gaps (0)
Great, no issues found!Qodo reviewed your code and found no material issues that require reviewTip of the day💡 Did you know, you can tweak Display preferences with a live preview to see your comment before it ships |
PR Summary by QodoAdd repository build, test, and architecture guidance
AI Description
Diagram
High-Level Assessment
Files changed (1)
|
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
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
Adds a
CLAUDE.mdat the repository root so coding agents (and new contributors) get the repo's build, test and architecture context without rediscovering it.pr-build-check.yml/release.ymlrun them, the UPX-pack requirement for legacy kernels, and the CMake knobs that matter. Two gotchas are called out:CMAKE_C_FLAGSis hard-reset at the top ofCMakeLists.txtso command-line flags are lost, andBUILD_SHARED_LIBS=ONis what drops-staticfor a dynamic/ASAN build.cYAML_test,tools/test_pipeline.sh, the three-architecture PR build gate, the rollinglatestvsv*release scheme, and the dispatch-only lab test workflow.getchipname()through the UART-base and/proc/cpuinfotables, the HAL as per-vendor installed global function pointers,chip_generationas the switch key every subcommand dispatches on, sensor probing overpossible_i2c_addrs, the board detector table, theSTANDALONE_LIBRARYsplit betweenipctoolandlibipchw, and what adding a new SoC or vendor touches.Verification
ipctool,ipcinfo,cYAML_test,libipchw.a../build/cYAML_testpasses all six cases.tools/test_pipeline.shpasses end to end.Documentation only; no code changes.