Skip to content
Success

Changes

Summary

  1. Turn on dependency checking on Inria CI (commit: 11d0d66) (details)
  2. Silence CI warning for undefined variable (commit: e3dffa8) (details)
  3. Tighten dependencies for C files (commit: b3af8ca) (details)
  4. Restore compatibility with GNU make 3.81 (commit: 617522e) (details)
  5. Don't use -MG (commit: c948b8d) (details)
  6. Remove unneeded $(otherfiles) in runtime/Makefile (commit: 942105f) (details)
  7. Update Changes (commit: 312ec98) (details)
Commit 11d0d662ae637663982e46e7c314c1eeb543417a by david.allsopp
Turn on dependency checking on Inria CI

Only the "old school build" test uses --disable-dependency-generation.
This is also tested on both Travis and AppVeyor, so we have good release
coverage checking of this option.
(commit: 11d0d66)
The file was modifiedtools/ci/inria/main (diff)
The file was modifiedtools/ci/inria/extra-checks (diff)
The file was modifiedtools/ci/inria/bootstrap (diff)
Commit e3dffa8a7f77bb730b006d99e1082f15772b7661 by david.allsopp
Silence CI warning for undefined variable
(commit: e3dffa8)
The file was modifiedMakefile.common (diff)
Commit b3af8ca2a950c3256f6700b388846d26d7439b3a by david.allsopp
Tighten dependencies for C files

The $(wildcard *.h) should only be there with
--disable-dependency-generation, since otherwise gcc -MM will be
determining exactly which header files should be checked.
(commit: b3af8ca)
The file was modifiedMakefile.common (diff)
Commit 617522ec9561873644cb4ce3e9e028a3230b906f by david.allsopp
Restore compatibility with GNU make 3.81

It's the macOS default installed version still. The dependency generation
inadvertently relies on behaviour introduced in GNU make 3.82 a decade ago.

The fix in otherlibs/systhreads/Makefile also corrects missing
NATIVE_CPPFLAGS when generating st_stubs.n.d, so st_stubs.n.o now
correctly depends on caml/stacks.h instead of caml/stack.h
(commit: 617522e)
The file was modifiedruntime/Makefile (diff)
The file was modifiedotherlibs/systhreads/Makefile (diff)
Commit c948b8d051fe0e982ca39674ebbf553d5bee1b7b by david.allsopp
Don't use -MG

It's a subtly broken thing to do, as discovered on the ARM workers on
Inria's CI.
(commit: c948b8d)
The file was modifiedruntime/Makefile (diff)
Commit 942105f49e3f758a7ee0c1cfe003f7415ec64832 by david.allsopp
Remove unneeded $(otherfiles) in runtime/Makefile

No longer required, and in fact causing breakages now that -MG isn't used.
(commit: 942105f)
The file was modifiedruntime/Makefile (diff)
The file was modifiedChanges (diff)