Organize scripts by purpose#1237
Open
Shounak-Ghosh wants to merge 1 commit into
Open
Conversation
…d reference paths, clean up stale harnesses pycache, and tested that scripts work as before
Contributor
End-to-End Test ReportTest Previewgrate harnessGrate Test Report
Cases
static harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
wasm harnessTest ReportDeterministic TestsSummary
Test Results by Category
Fail TestsSummary
Test Results by Category
C++ harnessSummary
Cases
|
|||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||||
Yaxuan-w
requested review from
qianxichen233,
rishabhBudhouliya and
vidyalakshmir
Yaxuan-w
approved these changes
Jun 5, 2026
Contributor
|
I think some of the scripts path is used in apps repo (https://github.com/Lind-Project/lind-wasm-apps/), will need a parallel PR on that side to fix path otherwise a few application compilation will be broken |
vidyalakshmir
left a comment
vidyalakshmir
left a comment
Contributor
There was a problem hiding this comment.
Looks fine! Agree what Qianxi said. We need to fix the paths to any of these scripts/binaries (like Lind-run etc) within the apps and also grates repo (if they use it).
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
4 participants
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.
Description
This PR reorganizes the
scripts/directory to group files by their logical purpose. All existing standard commands continue to function exactly as before.Closes #1208
What Changed
bin/,build/,test/,generate/,data/,tools/,config/) and relocated the appropriate scripts, harnesses, and data files into them.../to account for the new depth) across theMakefile, Bash scripts, Python harnesses, and Dockerfiles.__pycache__directories to prevent any caching of old file paths.Verification
The following standard workflows were tested to ensure the file movements did not break existing functionality:
make build(Full build chain: lind-boot + sysroot resolves properly)make test(All harnesses successfully discovered via the new test runner path)make lindfsscripts/bin/lind_compileandscripts/bin/lind_run) correctly discover the repo root.scripts/test/benchrunner.py) executes successfully.