microkernel icon indicating copy to clipboard operation
microkernel copied to clipboard

[IPC] F: Added correct syscalls for ipc module

Open SamuelVLC opened this issue 10 months ago • 3 comments

The previous syscalls added incorrectly were removed. The correct ones have been in added. Unit tests yet to be implemented.

SamuelVLC avatar Apr 03 '24 23:04 SamuelVLC

@ppenna there's been an error in one of the checks. can u explain me why?

SamuelVLC avatar Apr 03 '24 23:04 SamuelVLC

@ppenna there's been an error in one of the checks. can u explain me why?

It is failing to build the documentation: https://github.com/nanvix/microkernel/actions/runs/8546724311/job/23417642453

Have you run it locally?

make TARGET=x86 doxygen

ppenna avatar Apr 04 '24 14:04 ppenna

@ppenna there's been an error in one of the checks. can u explain me why?

It is failing to build the documentation: https://github.com/nanvix/microkernel/actions/runs/8546724311/job/23417642453

Have you run it locally?

make TARGET=x86 doxygen

I didn't know because I hadn't gotten this error before 😅

SamuelVLC avatar Apr 04 '24 18:04 SamuelVLC

@ppenna requested changes in our meeting have been added.

SamuelVLC avatar Apr 29 '24 20:04 SamuelVLC

Closing this PR without merge, as I had to rewrite the whole commit history to solve conflicts. Note:

  • Before asking for reviews on PRs, rebase it. You can do that with git rebase -i <base branch>
  • Split your PR into multiple commits, each of which buildable and testable.

ppenna avatar Apr 30 '24 23:04 ppenna