There is a double prompt in the IPsec vpn article at the moment.
e.g. # corp-net#
This patch attempts to address that by moving the hostname to the start of the line before using the root.prompt macro.
Fix spelling mistake picked up by igor.
Details
Details
Diff Detail
Diff Detail
- Repository
- rD FreeBSD doc repository - subversion
- Lint
Lint Not Applicable - Unit
Tests Not Applicable
Event Timeline
Comment Actions
Clarify that FreeBSD 11 comes with IPsec enabled by default, a custom kernel is only required on older releases.
Comment Actions
A few comments.
| en_US.ISO8859-1/books/handbook/security/chapter.xml | ||
|---|---|---|
| 374 ↗ | (On Diff #29453) | That is output from the utility. Is that fixed in there and we did not pick it up yet or did they fix it and have not updated this section? |
| 2130 ↗ | (On Diff #29453) | A between &os; and 11 will be good to have. |
| en_US.ISO8859-1/books/handbook/security/chapter.xml | ||
|---|---|---|
| 374 ↗ | (On Diff #29453) | Ah, good catch, I didn't check the context. It was flagged by igor. |
Comment Actions
We had encountered the same problem at BSDCan in another review. Seemed like a low-hanging fruit, but it isn't. ;-)
| head/en_US.ISO8859-1/books/handbook/security/chapter.xml | ||
|---|---|---|
| 2130 | I think we generally say "later" rather than "newer". | |
| 2131 | s/older releases/previous versions/ Rewriting to avoid the if/pause/then structure (and leaving out some markup for clarity): Add these options to the custom kernel configuration file to add IPsec support to previous versions of &os;. | |
| 2132 | Please eliminate "the following" whenever it is used incorrectly, which is almost always. The only time to use it is when it can't be replaced with a simpler "this" or "these". | |
Comment Actions
Hi Warren,
Apologies for the late reply to this, just wanted to follow up to say I've not forgotten about this review, will get the points you raised addressed next week if not this Sunday.
Sevan