ziglang / zig-bootstrap

take off every zig
368 stars 86 forks source link

Add copy-pastable build instructions #159

Open matklad opened 1 year ago

matklad commented 1 year ago

The main value here is to put CMAKE_GENERATOR=Ninja front-and-center, so that users realize that extra steps are needed for a parallel build.

It is documented quite clearly already, but, from my very personal experience:

daurnimator commented 1 year ago

This makes it look like ninja is a requirement rather than an option.

people don't read the docs and spend hours hammering one poor core

Perhaps we could put CMAKE_BUILD_PARALLEL_LEVEL=$(nproc) instead?

matklad commented 1 year ago

I think ninja is a better default (simpler, faster, consistent across platforms), so, given that we already need CMake, nudging towards ninja seems like the right call.

LinuxUserGD commented 1 year ago

Maybe also add Windows Build Instructions to TL;DR?