diff mbox series

doc: nft.8 document use of -f option to start nft scripts

Message ID 20171119095150.6697-1-duncan_roe@optusnet.com.au
State Accepted
Delegated to: Florian Westphal
Headers show
Series doc: nft.8 document use of -f option to start nft scripts | expand

Commit Message

Duncan Roe Nov. 19, 2017, 9:51 a.m. UTC
The man page didn't document nft scripts at all before, so putting that with the
-f option seemed as good a place as any. It does work to start scripts
#!/usr/sbin/nft --file but I didn't think this was worth documenting. The wiki
is also updated.

Signed-off-by: Duncan Roe <duncan_roe@optusnet.com.au>
---
 doc/nft.xml | 3 +++
 1 file changed, 3 insertions(+)

Comments

Duncan Roe Nov. 24, 2017, 2:33 a.m. UTC | #1
Hi Pablo,

Is there anything wrong with this patch? Like, does it document an untinded
feature or something?

Cheers ... Duncan.

On Sun, Nov 19, 2017 at 08:51:50PM +1100, Duncan Roe wrote:
> The man page didn't document nft scripts at all before, so putting that with the
> -f option seemed as good a place as any. It does work to start scripts
> #!/usr/sbin/nft --file but I didn't think this was worth documenting. The wiki
> is also updated.
>
> Signed-off-by: Duncan Roe <duncan_roe@optusnet.com.au>
> ---
>  doc/nft.xml | 3 +++
>  1 file changed, 3 insertions(+)
>
> diff --git a/doc/nft.xml b/doc/nft.xml
> index 2dec0aa..432f1c4 100644
> --- a/doc/nft.xml
> +++ b/doc/nft.xml
> @@ -168,6 +168,9 @@ vi:ts=4 sw=4
>  					<para>
>  						Read input from <replaceable>filename</replaceable>.
>  					</para>
> +					<para>
> +						nft scripts must start <command>#!/usr/sbin/nft -f</command>
> +					</para>
>  				</listitem>
>  			</varlistentry>
>  			<varlistentry>
> --
> 2.9.0
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Duncan Roe Nov. 24, 2017, 2:43 a.m. UTC | #2
On Fri, Nov 24, 2017 at 01:33:08PM +1100, Duncan Roe wrote:
...
> Is there anything wrong with this patch? Like, does it document an untinded
> feature or something?
...
Rats! untinded => unintended
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Florian Westphal Nov. 24, 2017, 6:46 a.m. UTC | #3
Duncan Roe <duncan_roe@optusnet.com.au> wrote:
> Hi Pablo,
> 
> Is there anything wrong with this patch? Like, does it document an untinded
> feature or something?

Oh, sorry, its my fault.

I applied this patch but forgot to say so.
--
To unsubscribe from this list: send the line "unsubscribe netfilter-devel" in
the body of a message to majordomo@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
diff mbox series

Patch

diff --git a/doc/nft.xml b/doc/nft.xml
index 2dec0aa..432f1c4 100644
--- a/doc/nft.xml
+++ b/doc/nft.xml
@@ -168,6 +168,9 @@  vi:ts=4 sw=4
 					<para>
 						Read input from <replaceable>filename</replaceable>.
 					</para>
+					<para>
+						nft scripts must start <command>#!/usr/sbin/nft -f</command>
+					</para>
 				</listitem>
 			</varlistentry>
 			<varlistentry>