Repository navigation
fix issues 156, 176 - #179
Conversation
PR Summary by QodoFix cross-shell make help and expand AI documentation pointers
AI Description
Diagram
High-Level Assessment
Files changed (2)
|
pratheesh
left a comment
There was a problem hiding this comment.
Thanks for this, the make help fix is a nice catch. I checked the help recipe under both bash and dash: before the change the output differs (the **** lines glob-expand and the C:\ti\... path loses its backslashes); after the change both shells print identical, correct text. The new docs_ai/README.md pointers to docs/release_notes.md and make help look good and cover #176. (I could not run a real gmake here, so I tested the recipe lines directly in the shells.)
A couple of small asks:
- The PR says it fixes #156, but that issue asks for a docs/ page that people (not only AI agents) can find from README.md / getting_started.md, plus a note that these parts have no SPRUHF8A-style standalone guide and a pointer to SPRUIJ2. Could you either link the TRM/register table from README.md or docs/getting_started.md, or change the description to "part of #156"?
- Please double-check the literature numbers and the "200+ pages" statement against ti.com, since I couldn't verify all of them (I only confirmed AM263x/SPRUJ42 against the issue text).
Otherwise LGTM.
|
@pratheesh thanks for the catch, added a pointer from the top-level readme |
fb3945f to
6768c70
Compare
The makefile echos under 'make/gmake help' worked fine when bash was used, but broke when dash was used on Ubuntu instead. Use quotation marks so that the echos render properly in both scenarios. The Windows path forward slashes were interacting poorly with dash, so I updated them to backslashes for compatibility reasons. Signed-off-by: Nick Saulnier <nsaulnier@ti.com>
Signed-off-by: Nick Saulnier <nsaulnier@ti.com>
Signed-off-by: Nick Saulnier <nsaulnier@ti.com>
6768c70 to
65486f2
Compare
Add pointers to docs_ai/README.md:
TRMs & register docs
release_notes
makefile build instructions
Fixes #156 and #176