Page MenuHomeFreeBSD
Differential D34427

rc.conf: Document zfskeys
ClosedPublic
Actions

Authored by 0mp on Mar 3 2022, 7:05 PM.
Tags
None
Referenced Files
Unknown Object (File)
Fri, Apr 5, 3:32 PM
Unknown Object (File)
Fri, Apr 5, 3:32 PM
Unknown Object (File)
Fri, Apr 5, 3:02 PM
Unknown Object (File)
Fri, Apr 5, 2:50 PM
Unknown Object (File)
Sun, Mar 31, 12:24 PM
Unknown Object (File)
Jan 16 2024, 12:51 AM
Unknown Object (File)
Dec 20 2023, 7:19 AM
Unknown Object (File)
Dec 18 2023, 7:39 PM
View All 32 Files
Subscribers
imp
pauamma_gundo.com

Details

Reviewers
allanjude
Group Reviewers
manpages
Commits
rGe8baa0024251: rc.conf: Document zfskeys
rG8719e8a951b7: rc.conf: Document zfskeys
Summary

Fixes: 33ff39796ffe Add zfskeys rc.d script for auto-loading encryption keys
MFC after: 3 days
Sponsored by: Modirum
Sponsored by: Klara Inc

Diff Detail

Repository
rG FreeBSD src repository
Lint
Lint Not Applicable
Unit
Tests Not Applicable

Event Timeline

0mp created this revision.Mar 3 2022, 7:05 PM
Herald added a reviewer: manpages. · View Herald TranscriptMar 3 2022, 7:05 PM
Herald added a subscriber: imp. · View Herald Transcript
0mp requested review of this revision.Mar 3 2022, 7:05 PM
Harbormaster completed remote builds in B44630: Diff 103453.Mar 3 2022, 7:05 PM
allanjude accepted this revision as: allanjude.Mar 3 2022, 7:07 PM
allanjude added inline comments.
share/man/man5/rc.conf.5
4652

I am not sure we need this comma

This revision is now accepted and ready to land.Mar 3 2022, 7:07 PM
0mp marked an inline comment as done.Mar 3 2022, 7:13 PM
0mp added inline comments.
share/man/man5/rc.conf.5
4652

Hmm, it may indeed not be needed. I'll remove it in the final version.

Closed by commit rG8719e8a951b7: rc.conf: Document zfskeys (authored by 0mp). · Explain WhyMar 3 2022, 7:14 PM
This revision was automatically updated to reflect the committed changes.
0mp marked an inline comment as done.
0mp added a commit: rG8719e8a951b7: rc.conf: Document zfskeys.
pauamma_gundo.com added a subscriber: pauamma_gundo.com.Mar 4 2022, 6:06 PM

Too fast for me.

share/man/man5/rc.conf.5
4650

s/the/then/ is the important part.

0mp marked an inline comment as done.Mar 4 2022, 7:40 PM
0mp added inline comments.
share/man/man5/rc.conf.5
4650

Thank you! A fix has been committed.

0mp marked an inline comment as done.Mar 6 2022, 8:22 PM
0mp added a commit: rGe8baa0024251: rc.conf: Document zfskeys.

Revision Contents
Changeset List

PathSize
share/
man/
man5/
26 lines

Diff 103454

View Options

share/man/man5/rc.conf.5

Loading...