-
Notifications
You must be signed in to change notification settings - Fork 18
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Putting arch file into a package. #324
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
I like how much cleaner this makes things.
3a8017a
to
d53c10b
Compare
Sorry for the stream of force pushes, I got it working properly. I thought it wasn't possible to have circular references but it actually is :), I went off a on a tangent yesterday haha. Most of the tests are passing. Still needs some work. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
c7cb85d
to
f2b9df3
Compare
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Soz for the late review. I am slow.
Two small things.
This is very nice, (Insert the nice * mouth click* meme)
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
🚀
issue #77
Unfortunately, arch.zig and the kernel are dependant on each other. I couldn't think of a way to have two pacakges directly depend on each other. I made a file pluto.zig that collects and re-exports all the files in the kernel. I made packages out of arch.zig and pluto.zig. kmain.zig then imports those packges.
This works for just running the kernel but getting the tests to work will be a bigger challenge.