Remove outdated note about USE_RCORDER
Add suggestion to use devel/rclint to check rc scripts Scripts ending .sh.in were renamed a while ago, so make the comment regarding those less prominent Approved by: gjb (mentor)
This commit is contained in:
parent
f668ca929c
commit
1e7cfe7765
Notes:
svn2git
2020-12-08 03:00:23 +00:00
svn path=/head/; revision=43363
1 changed files with 5 additions and 15 deletions
|
@ -9905,15 +9905,6 @@ post-patch:
|
|||
<link linkend="using-sub-files">the relevant
|
||||
section</link>.</para>
|
||||
|
||||
<para>Prior to &os; 6.1-RELEASE, integration with
|
||||
&man.rcorder.8; is available by using
|
||||
<varname>USE_RCORDER</varname> instead of
|
||||
<varname>USE_RC_SUBR</varname>. However, use of this method
|
||||
is not necessary unless the port has an option to install
|
||||
itself in the base, or the service needs to run prior to the
|
||||
<filename>FILESYSTEMS</filename> <filename>rc.d</filename>
|
||||
script in the base.</para>
|
||||
|
||||
<para>As of &os; 6.1-RELEASE, local
|
||||
<filename>rc.d</filename> scripts (including those installed
|
||||
by ports) are included in the overall &man.rcorder.8; of the
|
||||
|
@ -9985,11 +9976,6 @@ run_rc_command "$1"</programlisting>
|
|||
<literal>_enable</literal> variable is not optional,
|
||||
and should use the ":" for the default.</para>
|
||||
|
||||
<note>
|
||||
<para>No new scripts should be added with the
|
||||
<filename>.sh</filename> suffix.</para>
|
||||
</note>
|
||||
|
||||
<sect2>
|
||||
<title>Pre-Commit Checklist</title>
|
||||
|
||||
|
@ -9998,12 +9984,16 @@ run_rc_command "$1"</programlisting>
|
|||
before committing one, please consult the following
|
||||
checklist to be sure that it is ready.</para>
|
||||
|
||||
<para>The <package role="port">devel/rclint</package>
|
||||
port can check for most of these, but it is not a
|
||||
substitute for proper review.</para>
|
||||
|
||||
<procedure>
|
||||
<step>
|
||||
<para>If this is a new file, does it have
|
||||
<filename>.sh</filename> in the file name? If so that
|
||||
should be changed to just <filename>file.in</filename>
|
||||
since new <filename>rc.d</filename> files may not end
|
||||
since <filename>rc.d</filename> files may not end
|
||||
with that extension.</para>
|
||||
</step>
|
||||
|
||||
|
|
Loading…
Reference in a new issue