Conversation
| displayName: 'Download qemu' | ||
|
|
||
| - script: zig*/zig build test -Drt-test=true -Dzig-path=zig*/zig | ||
| - script: | |
There was a problem hiding this comment.
Surely you would also have a normal build as well, as adding the testing and setting to true would have the OS be in testing mode when released. Then with the build OS with runtime testing enabled, add a command to start the kernel which would trigger the runtime tests
There was a problem hiding this comment.
There is actually a normal build without runtime testing enabled earlier on in the pipeline, check the "Build kernel" step.
SamTebbs33
force-pushed
the
feature/build-options
branch
from
9993342 to
533b2f9
Compare
DrDeano
left a comment
DrDeano
left a comment
There was a problem hiding this comment.
Bam, Just a question for you :)
| const log = @import("log.zig"); | ||
| const serial = @import("serial.zig"); | ||
| const mem = @import("mem.zig"); | ||
| const options = @import("build_options"); |
There was a problem hiding this comment.
This is cool, how can you just pass in this string like that?
There was a problem hiding this comment.
I think that it's a hardcoded thing in the compiler and it just finds the right build options file :)
SamTebbs33
force-pushed
the
feature/build-options
branch
from
f0da252 to
32eb09e
Compare
Add infrastructure for build options
Add infrastructure for build options
2 participants
This patch adds the infrastructure for passing build options from build.zig to the source. At the moment the only option added is
rt_testwhich will be used to run extensive runtime tests (#60).I added the config.zig file to hide the rather ugly build options file import from kmain.zig, much like we did with arch.zig.
Closes #59