Call for 1.10.1 testing: Delay opening logfiles, macOS 10.12

[Jan Niklas Hasse]

Hi everyone,

we'd like to release 1.10.1 from current master.

1.10.0 introduced the restat tool, which CMake 3.17 uses to update the timestamps in .ninja_log to avoid having to run some rules twice after regenerating. This didn't work on Windows though, as you can't write to a file which is opened for writing by another process there. To clarify what I mean imagine you have the following hierarchy:

a.) ninja

b.) -> regenerate by calling CMake

c.) -> -> CMake calls ninja -t restat

d.) ninja continues the build and rereads the build manifest

Between a.) and d.) the outer ninja process would (unnecessarily) open .ninja_log, but not write anything to it yet. c.) would then fail on Windows.

After https://github.com/ninja-build/ninja/pull/1780 ninja will now open .ninja_log on the first write - so during d.) - or if it has finished. In the future we could even avoid creating .ninja_log if it would be empty, but I didn't want to change too much of the behavior in a minor release (it would also require touching some unit tests).

The other change is that the automatic build on GitHub Actions now builds binaries compatible with macOS 10.12 (previously they required 10.14 IIRC).

For a complete list of changes see: https://github.com/ninja-build/ninja/compare/v1.10.0...00968321d6ace10b6798855cab7620dc6a5620fd

Please test a build of the master branch with your projects and report any problems. Thanks!

[Brad King]

On 8/3/2020 7:08 AM, Jan Niklas Hasse wrote:
> we'd like to release 1.10.1 from current master.
>

> After https://github.com/ninja-build/ninja/pull/1780

I just tested Ninja `master` as of commit 00968321d6 on Windows.
I locally hacked its version number to be 1.10.1. With that, the
problem from https://github.com/ninja-build/ninja/issues/1724 is
fixed. I've posted a merge request to CMake:

https://gitlab.kitware.com/cmake/cmake/-/merge_requests/5095

to remove our workaround for Ninja 1.10.1 and above. I've closed the
CMake merge request temporarily until Ninja 1.10.1 is actually released,
but once it is I'll re-open and merge the CMake change.

Thanks!
-Brad

[jha...@gmail.com]

Thanks! Since there haven't been any objections, I've just released 1.10.1 on GitHub.