Page MenuHomeFreeBSD

Some minor fixes to the makefiles chapter.
ClosedPublic

Authored by mat on Apr 8 2015, 12:00 PM.
Tags
None
Referenced Files
Unknown Object (File)
Wed, Apr 24, 11:05 PM
Unknown Object (File)
Mar 15 2024, 7:15 AM
Unknown Object (File)
Mar 15 2024, 6:38 AM
Unknown Object (File)
Dec 25 2023, 10:49 PM
Unknown Object (File)
Dec 19 2023, 11:53 PM
Unknown Object (File)
Nov 16 2023, 11:45 AM
Unknown Object (File)
Nov 11 2023, 6:43 AM
Unknown Object (File)
Nov 9 2023, 6:44 AM
Subscribers

Details

Reviewers
gjb
wblock

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage

Event Timeline

mat retitled this revision from to Some minor fixes to the makefiles chapter..
mat updated this object.
mat edited the test plan for this revision. (Show Details)
mat added reviewers: wblock, gjb.
mat added a subscriber: Doc Committers.
en_US.ISO8859-1/books/porters-handbook/makefiles/chapter.xml
4388

"This is intentional" is probably a better way to say that. The sentence can be split there:

This is intentional. A

4390

s/change/changes/

4492

s/can/can be/

4605

"being overridden" means the same as just "overridden":

These operators work until overridden, or...

4647–4648

Hmm. Should be "installed". "&man.install.1;ed" might look okay.

en_US.ISO8859-1/books/porters-handbook/makefiles/chapter.xml
4647–4648

Mmmm, no, I started with "These are <command>install</command> with ..." and thought that using &man.install.1; was better.

The sentence is maybe badly written though. The idea is that the previous INSTALL_* macros are the install command with flags, maybe the original was better and should not be touched. Or maybe "the &man.install.1; command" or something.

en_US.ISO8859-1/books/porters-handbook/makefiles/chapter.xml
4647–4648

How about:

These variables are set to the &man.install.1; command with the appropriate flags for each situation.

4652

s/render/renders/

4661–4662

s/forced to/absolutely required/

4661–4662

Passive->Active

s/will install/installs/

4662–4664

s/strip/strips/

4671

s/will strip/strips/

Remove a reference that makes no sense.

wblock edited edge metadata.

Please remember to build-test before commit. Thanks!

This revision is now accepted and ready to land.Apr 9 2015, 2:44 PM