Page MenuHomeFreeBSD

crossbuild: Make the CHECK_TIME variable work on Linux
ClosedPublic

Authored by arichardson on Feb 1 2018, 2:20 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Aug 21, 1:27 PM
Unknown Object (File)
Mon, Aug 17, 5:24 PM
Unknown Object (File)
Sat, Aug 15, 8:22 PM
Unknown Object (File)
Fri, Aug 14, 9:21 PM
Unknown Object (File)
Fri, Aug 14, 6:09 PM
Unknown Object (File)
Fri, Aug 14, 5:59 AM
Unknown Object (File)
Wed, Aug 12, 7:54 PM
Unknown Object (File)
Tue, Aug 11, 9:28 AM
Subscribers
None

Details

Summary

Linux /usr/bin/find doesn't understand the -mtime -0s flag.
Instead create a temporary file and compare that file's mtime to
sys/sys/param.h to check whether the clock is correct.

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

Makefile
360 โ†—(On Diff #38756)

This is now wrong. This changes from 0s to 0 days. Due to funky roundings, this tests to see if the sys/sys/param.h file is one day or less old (or in the future), whereas on FreeBSD it tests if it is 1 second old or in the future. The difference is important and will lead to false positives.

"touch /tmp/junk && find ${.CURDIR}/sys/sys/param.h -newer /tmp/junk && rm /tmp/junk" would do the same thing and be portable...

arichardson edited the summary of this revision. (Show Details)

Use a temporary file instead

imp added inline comments.
Makefile
355 โ†—(On Diff #38884)

Thanks!

This revision is now accepted and ready to land.Feb 5 2018, 5:12 PM
This revision was automatically updated to reflect the committed changes.