Page MenuHomeFreeBSD

Followup to PR231977: Mention that /etc/pf.conf must be created first
ClosedPublic

Authored by bcr on Mar 10 2019, 4:50 PM.
Tags
None
Referenced Files
Unknown Object (File)
Feb 6 2024, 2:35 AM
Unknown Object (File)
Jan 16 2024, 8:25 AM
Unknown Object (File)
Dec 23 2023, 12:06 AM
Unknown Object (File)
Dec 18 2023, 2:35 PM
Unknown Object (File)
Dec 13 2023, 8:19 AM
Unknown Object (File)
Sep 2 2023, 11:50 AM
Unknown Object (File)
Sep 1 2023, 8:02 PM
Unknown Object (File)
Aug 15 2023, 6:14 AM
Subscribers

Details

Summary

This is a followup change to address PR 231977: we need to mention that /etc/pf.conf does not exist by default.

I'm having a bit of trouble coming up with a proper change/wording in /head/libexec/rc/rc.conf. Removing the line

pf_rules="/etc/pf.conf"         # rules definition file for pf

would be wrong, as it still is the default location for the file, even if it does not exist there by default. What would be a good way to say that in the comment? Something like

pf_rules="/etc/pf.conf"         # rules definition file for pf (nonexistent by default)

maybe?

Test Plan
  1. Apply the patch to share/man/man5/pf.conf.5
  2. See how it looks like: man ./share/man/man5/pf.conf.5

Diff Detail

Repository
rS FreeBSD src repository - subversion
Lint
Lint Skipped
Unit
Tests Skipped
Build Status
Buildable 22993

Event Timeline

pf_rules="/etc/pf.conf" # rules definition file for pf (nonexistent by default)

seems fine.

share/man/man5/pf.conf.5
3056 ↗(On Diff #54897)

How about The file has to be created manually as it is not installed with a standard installation.

Update pf.conf.5 with suggestions from @0mp.

This revision is now accepted and ready to land.Mar 12 2019, 10:07 AM
This revision was automatically updated to reflect the committed changes.