Skip to content

mkfs implemented#5

Closed
sudoheckbeluga wants to merge 2 commits into
codenet:p4-trapsfrom
sudoheckbeluga:p6-mkfs
Closed

mkfs implemented#5
sudoheckbeluga wants to merge 2 commits into
codenet:p4-trapsfrom
sudoheckbeluga:p6-mkfs

Conversation

@sudoheckbeluga

Copy link
Copy Markdown

No description provided.

legends1307 added a commit to legends1307/col331rs that referenced this pull request Feb 16, 2026
- Added src/mkfs.rs: pure Rust mkfs tool (based on sudoheckbeluga's
  work in PR codenet#5), adapted with FSSIZE=1000 and LOGSIZE=0 to match
  the C repo's p9-name-layer branch. No more ../col331/ dependency.
- Updated Makefile: mkfs now compiled with rustc from src/mkfs.rs
- Added name layer functions in src/fs.rs: irelse, namecmp,
  dirlookup, skipelem, namex, namei, nameiparent
- Updated welcome() in src/lib.rs to use namei("/welcome.txt")
legends1307 added a commit to legends1307/col331rs that referenced this pull request Feb 16, 2026
- Added src/mkfs.rs: pure Rust mkfs tool (based on sudoheckbeluga's
  work in PR codenet#5), adapted with FSSIZE=1000 and LOGSIZE=0 to match
  the C repo's p9-name-layer branch. No more ../col331/ dependency.
- Updated Makefile: mkfs now compiled with rustc from src/mkfs.rs
- Added name layer functions in src/fs.rs: irelse, namecmp,
  dirlookup, skipelem, namex, namei, nameiparent
- Updated welcome() in src/lib.rs to use namei("/welcome.txt")
@codenet

codenet commented May 1, 2026

Copy link
Copy Markdown
Owner

This was superceded by #17 which is merged.

@codenet codenet closed this May 1, 2026
@sudoheckbeluga

sudoheckbeluga commented May 1, 2026

Copy link
Copy Markdown
Author

Sir #17 has used my implementation only, He also gives credit to #5 in his PR description. So will this be considered for my errata marks ?

@codenet

codenet commented May 1, 2026 via email

Copy link
Copy Markdown
Owner

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants