netfilter-devel.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
* [PATCH] doc: nft.8 document use of -f option to start nft scripts
@ 2017-11-19  9:51 Duncan Roe
  2017-11-24  2:33 ` Duncan Roe
  0 siblings, 1 reply; 4+ messages in thread
From: Duncan Roe @ 2017-11-19  9:51 UTC (permalink / raw)
  To: netfilter-devel

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


^ permalink raw reply related	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: nft.8 document use of -f option to start nft scripts
  2017-11-19  9:51 [PATCH] doc: nft.8 document use of -f option to start nft scripts Duncan Roe
@ 2017-11-24  2:33 ` Duncan Roe
  2017-11-24  2:43   ` Duncan Roe
  2017-11-24  6:46   ` Florian Westphal
  0 siblings, 2 replies; 4+ messages in thread
From: Duncan Roe @ 2017-11-24  2:33 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: nft.8 document use of -f option to start nft scripts
  2017-11-24  2:33 ` Duncan Roe
@ 2017-11-24  2:43   ` Duncan Roe
  2017-11-24  6:46   ` Florian Westphal
  1 sibling, 0 replies; 4+ messages in thread
From: Duncan Roe @ 2017-11-24  2:43 UTC (permalink / raw)
  To: pablo; +Cc: netfilter-devel

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

^ permalink raw reply	[flat|nested] 4+ messages in thread

* Re: [PATCH] doc: nft.8 document use of -f option to start nft scripts
  2017-11-24  2:33 ` Duncan Roe
  2017-11-24  2:43   ` Duncan Roe
@ 2017-11-24  6:46   ` Florian Westphal
  1 sibling, 0 replies; 4+ messages in thread
From: Florian Westphal @ 2017-11-24  6:46 UTC (permalink / raw)
  To: pablo, netfilter-devel

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.

^ permalink raw reply	[flat|nested] 4+ messages in thread

end of thread, other threads:[~2017-11-24  6:47 UTC | newest]

Thread overview: 4+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2017-11-19  9:51 [PATCH] doc: nft.8 document use of -f option to start nft scripts Duncan Roe
2017-11-24  2:33 ` Duncan Roe
2017-11-24  2:43   ` Duncan Roe
2017-11-24  6:46   ` Florian Westphal

This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).