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
No Lint Coverage - Unit
No Test Coverage - Build Status
Buildable 9797 Build 10232: arc lint + arc unit
Event Timeline
Comment Actions
Clarify that FreeBSD 11 comes with IPsec enabled by default, a custom kernel is only required on older releases.
| en_US.ISO8859-1/books/handbook/security/chapter.xml | ||
|---|---|---|
| 374 | 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 ↗ | (On Diff #29464) | I think we generally say "later" rather than "newer". |
| 2131 ↗ | (On Diff #29464) | 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 ↗ | (On Diff #29464) | 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