Review for PR 213175
Details
Diff Detail
- Repository
- rD FreeBSD doc repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
3946 ↗ | (On Diff #39220) | Although *make* is the command, logically *make install* is the command-to-which-you-add-DESTDIR; I would move the *install* inside <command>. |
4023 ↗ | (On Diff #39220) |
Scans better for me and includes an example. |
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
3946 ↗ | (On Diff #39220) | No, "make install" is not literally the command; "make installworld", "make installkernel", and "make distribution" are, collectively, "the make install commands". |
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
3946 ↗ | (On Diff #39220) | @ian, does something like this sounds better ? but do not forget to add <option>DESTDIR=<replaceable>${NFSROOTDOR} </replaceable></option> when running the <command>make installkernel</command> and <command>make installworld</command> commands. |
I found a typo...
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
3946 ↗ | (On Diff #39220) | s/NFSROOTDOR/NFSROOTDIR/ ;-) |
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
3946 ↗ | (On Diff #39220) | Yes, that's good. |
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
2522 ↗ | (On Diff #39545) | No objection to the changes, just one minor nit. |
en_US.ISO8859-1/books/handbook/advanced-networking/chapter.xml | ||
---|---|---|
2522 ↗ | (On Diff #39545) | You're completely right, this is an unwanted change. |