Migrate from mod.rs to modern module file naming convention

[itstrivial]

Summary

The project currently uses the older mod.rs convention for module files.
Since Rust 1.30, the recommended approach is to use module_name.rs instead.

Current State

There are 14 mod.rs files in the project, e.g.:

• kernel/src/drivers/mod.rs
• kernel/src/memory/mod.rs
• shared/common/src/enums/mod.rs

Proposed Change

Rename according to the modern convention.

Example:

• drivers/mod.rs → drivers.rs
• drivers/keyboard/mod.rs → drivers/keyboard.rs

Reference

https://doc.rust-lang.org/reference/items/modules.html#module-source-filenames

---

I'm ready to work on this if necessary.

[sagi21805]

Hi!
Thanks for the suggestions, will try to migrate after finishing some WIP I have.

[sagi21805]

One question I have, that will remove the folder tree structure? Or I could still have a module folder for drivers for example, with all the files inside, and instead of mod.rs inside it would be called drivers.rs?

[itstrivial]

One question I have, that will remove the folder tree structure? Or I could still have a module folder for drivers for example, with all the files inside, and instead of mod.rs inside it would be called drivers.rs?

You keep the folder tree. For example, if drivers has submodules, you still keep them in a drivers/ directory.

The layout will become something like:

src/
  lib.rs
  drivers.rs          // Defines `mod drivers`.
  drivers/
    keyboard.rs       // Defines `mod keyboard` (`drivers::keyboard`).

As of today, a module can be defined by either (not both):

• drivers/mod.rs
• drivers.rs,

with drivers.rs being an encouraged modern style. It's not critical, I’ve just noticed that throughout the project you lean towards a more modern Rust.

Thanks a lot for LearnixOS!

[sagi21805]

Oh that seems better than the current approach, if you would like to work on it, you can!

Currently there is no CI that tests the build, so just test and run it yourself :)

If you want me to work on it, I will probably do it after some features that are currently in development.

In any case, thanks for the issue and explanation!

I really appreciate it!!

[itstrivial]

Don’t have time right now, unfortunately, but I’ll keep it in mind. 🙂

[sagi21805]

Ok, opening the issue was more then enough!

Currently I am migrating the project to a new structure and this change is included :)