Organize scripts by purpose#1237
Merged
Merged
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
previously 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).
Member
|
@qianxichen233 @vidyalakshmir Thanks for pointing these out! I've opened an issue to track this: Lind-Project/lind-wasm-apps#268. @Shounak-Ghosh Can you resolve merge conflicts and then I think this should be good to merge, then you can work on Lind-Project/lind-wasm-apps#268 as next task :D |
Contributor
I think an ideal model is to have the PR up on apps side as well, then merge both PR (this one and the one from apps side) together. Otherwise there will be a period of time where apps side is broken |
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
previously approved these changes
Jun 16, 2026
qianxichen233
previously approved these changes
Jun 24, 2026
…d reference paths, clean up stale harnesses pycache, and tested that scripts work as before
* Error conditions and constraints added to prevent overflow and silent wrapping * Refactor error handling for page overflow check * code formatted * minor formatting * fix: rustfmt formatting * Test case resolved and removed from skip list * test updated * Test case updated * compile time issue fixed * [Test case] output mismatch fixed --------- Co-authored-by: Harindham Sharma <harindhamsharma@Mac.lan>
fix merge conflict issues
…t/lind-wasm into organize-scripts-dir
Shounak-Ghosh
dismissed stale reviews from qianxichen233 and Yaxuan-w
via
92d6752
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
approved these changes
Jun 25, 2026
qianxichen233
approved these changes
Jun 26, 2026
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
7 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.