svn commit: r345238 - head
Rodney W. Grimes
freebsd at gndrsh.dnsmgr.net
Mon Mar 18 21:40:04 UTC 2019
[ Charset UTF-8 unsupported, converting... ]
> On 3/16/19 1:02 PM, Wolfram Schneider wrote:
> > Author: wosch
> > Date: Sat Mar 16 20:02:57 2019
> > New Revision: 345238
> > URL: https://svnweb.freebsd.org/changeset/base/345238
> >
> > Log:
> > `make buildkernel' should display the build time in seconds
> >
> > PR: 224433
> > Approved by: cem
> > Differential Revision: https://reviews.freebsd.org/D13910
> >
> > Modified:
> > head/Makefile.inc1
> >
> > Modified: head/Makefile.inc1
> > ==============================================================================
> > --- head/Makefile.inc1 Sat Mar 16 17:55:22 2019 (r345237)
> > +++ head/Makefile.inc1 Sat Mar 16 20:02:57 2019 (r345238)
> > @@ -1584,6 +1584,11 @@ _cleankernobj_fast_depend_hack: .PHONY
> >
> > ${WMAKE_TGTS:N_worldtmp:Nbuild${libcompat}} ${.ALLTARGETS:M_*:N_worldtmp}: .MAKE .PHONY
> >
> > +# record kernel(s) build time in seconds
> > +.if make(buildkernel)
> > +_BUILDKERNEL_START!= date '+%s'
> > +.endif
> > +
> > #
> > # buildkernel
> > #
> > @@ -1640,7 +1645,12 @@ buildkernel: .MAKE .PHONY
> > @echo "--------------------------------------------------------------"
> > @echo ">>> Kernel build for ${_kernel} completed on `LC_ALL=C date`"
> > @echo "--------------------------------------------------------------"
> > +
> > .endfor
> > + @seconds=$$(($$(date '+%s') - ${_BUILDKERNEL_START})); \
> > + echo -n ">>> Kernel(s) build for${BUILDKERNELS} in $$seconds seconds, "; \
> > + echo "ncpu: $$(sysctl -n hw.ncpu)${.MAKE.JOBS:S/^/, make -j/}"
> > + @echo "--------------------------------------------------------------"
>
> Both this and the previous message for buildworld should use 'built' instead
> of 'build' so that the message is something like "World built in 47 seconds"
> rather than "World build in 47 seconds".
>
> This sentence is also somewhat odd:
>
> "Kernel(s) build for FOO BAR BAZ in 47 seconds"
>
> Even with 'built' fixed, I feel like it should be more like:
>
> "FOO BAR BAZ kernel(s) built in 47 seconds" by moving ${BUILDKERNELS} to the
> start of the message?
"Kernel(s) ${BUILDKERNELS} built in 47 seconds"
reads better for me.
> John Baldwin
--
Rod Grimes rgrimes at freebsd.org
More information about the svn-src-all
mailing list