crossbuild: Make the CHECK_TIME variable work on Linux
ClosedPublic
Actions

Authored by arichardson on Feb 1 2018, 2:20 PM.

Details

Reviewers

emaste
jhb
imp
bdrewery

Commits

rS328935: crossbuild: Make the CHECK_TIME variable work on Linux

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

arichardson created this revision.Feb 1 2018, 2:20 PM

Harbormaster completed remote builds in B14727: Diff 38756.Feb 1 2018, 2:20 PM

arichardson added a reviewer: bdrewery.Feb 4 2018, 8:39 PM

imp added inline comments.Feb 4 2018, 8:49 PM

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...

Use a temporary file instead

imp accepted this revision.Feb 5 2018, 5:12 PM

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

jhb accepted this revision.Feb 5 2018, 6:13 PM

Closed by commit rS328935: crossbuild: Make the CHECK_TIME variable work on Linux (authored by arichardson). · Explain WhyFeb 6 2018, 3:42 PM

This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Path

Size

head/

Makefile

2 lines

Diff 38944

View Options

crossbuild: Make the CHECK_TIME variable work on LinuxClosedPublicActions