Page MenuHomeFreeBSD
Differential D11510

Minor fixes to VPN section Pt2
ClosedPublic
Actions

Authored by sevan on Jul 7 2017, 1:41 AM.
Tags
None
Referenced Files
Unknown Object (File)
Nov 18 2023, 6:05 AM
Unknown Object (File)
Nov 6 2023, 11:38 PM
Unknown Object (File)
Nov 4 2023, 5:49 AM
Unknown Object (File)
Oct 25 2023, 2:02 AM
Unknown Object (File)
Oct 5 2023, 10:37 PM
Unknown Object (File)
Oct 3 2023, 5:45 AM
Unknown Object (File)
Sep 16 2023, 3:30 PM
Unknown Object (File)
Sep 11 2023, 8:35 PM
View All 16 Files
Subscribers
None

Details

Reviewers
wblock
bcr
Commits
rD50456: Reword sentence about IPsec on legacy releases based on feedback from wblock.
Summary

Address points raised in D11143

Diff Detail

Repository
rD FreeBSD doc repository - subversion
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 10362
Build 10774: arc lint + arc unit

Event Timeline

sevan created this revision.Jul 7 2017, 1:41 AM
sevan added a commit: rD50359: Add a note that FreeBSD 11 & newer have IPsec support enabled by default (no….Jul 7 2017, 1:42 AM
sevan added a parent revision: D11143: Minor fixes to VPN section.
sevan removed a commit: rD50359: Add a note that FreeBSD 11 & newer have IPsec support enabled by default (no….
sevan mentioned this in D11143: Minor fixes to VPN section.Jul 7 2017, 1:45 AM
bcr edited edge metadata.Jul 7 2017, 6:53 AM

A small suggestion on the original text...

en_US.ISO8859-1/books/handbook/security/chapter.xml
2130

This
&os;   11
with the spaces before and after the   defeats the purpose of keeping the version number close to the word "FreeBSD" in case there is a line break. I know it was there before your change, but we might as well fix it while we're here: &os; 11

Your actual change looks fine to me.

sevan updated this revision to Diff 30540.Jul 7 2017, 4:01 PM

Drop the spaces between the macros.

Harbormaster completed remote builds in B10371: Diff 30540.Jul 7 2017, 4:01 PM
sevan marked an inline comment as done.Jul 7 2017, 4:01 PM
wblock added inline comments.Jul 7 2017, 4:09 PM
en_US.ISO8859-1/books/handbook/security/chapter.xml
2131

Telling the reader that this is for earlier versions first seems important:

For previous versions of &os;, add these options to a custom kernel configuration file and rebuild the kernel as shown in <xref
sevan updated this revision to Diff 30541.Jul 7 2017, 4:54 PM

Re-edit based on feedback.

Harbormaster completed remote builds in B10372: Diff 30541.Jul 7 2017, 4:54 PM
wblock accepted this revision.Jul 7 2017, 5:09 PM
This revision is now accepted and ready to land.Jul 7 2017, 5:09 PM
Closed by commit rD50456: Reword sentence about IPsec on legacy releases based on feedback from wblock. (authored by sevan). · Explain WhyJul 7 2017, 5:33 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathSize
en_US.ISO8859-1/
books/
handbook/
security/
8 lines

Diff 30519

View Options

en_US.ISO8859-1/books/handbook/security/chapter.xml

Loading...