search

gsilvis / silvos

half-baked Silvis OS kernel
MIT License
4 stars 2 forks source link

Add yield() test #17

Closed allanlw closed 7 years ago

allanlw commented 7 years ago

This adds a simple test case to make sure that yield() works.

gsilvis commented 7 years ago

Fix Makefile conflict and squash the two commits, then LGTM

allanlw commented 7 years ago

Rebase'd onto master.

gsilvis commented 7 years ago

The problem with this method of editing the makefile is that no two tests will ever not conflict. It's very tiresome.

In any case, you have to rebase again. This time, can you also squash the two commits into one commit?

allanlw commented 7 years ago

That is only true if they are added at the end.

Regardless, it was not I that came up with this method of adding tests. An easy replacement for this would be to use a wildcard and patsubst to add all the userland programs that have expected.txt in their folders.

gsilvis commented 7 years ago

Oh yes, I am well aware of who created this problem. Thanks for the test!