Page MenuHomeFreeBSD

build(7) instructions for alternate dir should be clearer
ClosedPublic

Authored by sd_beastie.io on Apr 26 2016, 4:38 AM.
Tags
None
Referenced Files
Unknown Object (File)
Dec 22 2023, 9:33 PM
Unknown Object (File)
Dec 4 2023, 8:02 AM
Unknown Object (File)
Dec 1 2023, 6:26 AM
Unknown Object (File)
Nov 4 2023, 6:26 PM
Unknown Object (File)
Oct 3 2023, 6:24 PM
Unknown Object (File)
Sep 29 2023, 6:29 PM
Unknown Object (File)
Aug 20 2023, 5:17 AM
Unknown Object (File)
Aug 13 2023, 6:06 AM
Subscribers

Details

Summary

buildworld and buildkernel should be clearer about how to use alternate directories for building. Would be nice to point out to set MAKEOBJDIRPREFIX like installworld and installkernel does.

Test Plan
  • igor validation
  • man output

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped

Event Timeline

sd_beastie.io retitled this revision from to build(7) instructions for alternate dir should be clearer.
sd_beastie.io updated this object.
sd_beastie.io edited the test plan for this revision. (Show Details)
sd_beastie.io added reviewers: wblock, brd.
sd_beastie.io set the repository for this revision to rS FreeBSD src repository - subversion.

Update .Dd at the top of the file as well.

build.7
198

You need to keep the "The" at the start of the sentence.

202

Man page style has the new sentence start on a new line.

202

It is probably more technically correct to say that MAKEOBJDIRPREFIX is set in make's environment; just setting an unexported shell variable would be local to the shell; setting it on the make command line works for make but is treated slightly differently than inheriting it from the shell environment.
(The same discussion applies to the next chunk.)

285

Start this sentence with "The" as well.
Also, "build location" is not exactly a well-defined concept; it may be better to use more words and say that "the location of the object files generated by this build can be changed from the default..."

sd_beastie.io added inline comments.
build.7
202

I wanted to quickly remind users that the variable can be set to change the build output directory, similar to installworld and installkernel does. This acts as a quick reminder in case people are jumping straight to these sections instead of reading the whole man page. Yes RTFM is a separate discussion :) There is already text in the beginning of the man page and in the environment section that talks more in detail about all the variables.

I did change it to make variable as the rest of the text references. And changed build location to object directory.

285

See reply above.

sd_beastie.io edited edge metadata.
sd_beastie.io marked 2 inline comments as done.

Updated based on review comments.

build.7
200

"directory" is redundant here.

202

.Xr make 1
variable.

288

"directory" is redundant here, too.

290

.Xr make 1
variable.

sd_beastie.io edited edge metadata.
sd_beastie.io marked 4 inline comments as done.

Updated based on wblock's review comments

wblock edited edge metadata.

Looks okay to me.

This revision is now accepted and ready to land.Apr 27 2016, 10:13 PM
This revision was automatically updated to reflect the committed changes.