[{"id":1758712,"web_url":"http://patchwork.ozlabs.org/comment/1758712/","msgid":"<20170828165853.GB21786@salvia>","list_archive_url":null,"date":"2017-08-28T16:58:53","subject":"Re: [nft PATCH 1/2] nft.8: Fix and enhance synopsis section","submitter":{"id":1315,"url":"http://patchwork.ozlabs.org/api/people/1315/","name":"Pablo Neira Ayuso","email":"pablo@netfilter.org"},"content":"On Thu, Aug 17, 2017 at 03:38:59PM +0200, Phil Sutter wrote:\n> This patch addresses shortcomings in the main synopsis section\n> illustrating possible invocations of nft command:\n> \n> - Fix font styles to correctly put options into bold font and meta\n> characters (brackets, pipes) into normal font.\n> \n> - Add missing options to synopsis line.\n> \n> - Use curly braces where either one of the alternatives is required.\n> \n> - Remove choice=\"opt\" attribute since that is the default anyway.\n> \n> - Note that --includepath option is allowed to be given multiple times.\n\nI'm going to apply this.\n\nI'm undecided with 2/2, I don't remember to have seen such\nindirections in other manpages?\n\nThanks.\n--\nTo unsubscribe from this list: send the line \"unsubscribe netfilter-devel\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netfilter-devel-owner@vger.kernel.org;\n\treceiver=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xgyfZ3Nc3z9s7m\n\tfor ;\n\tTue, 29 Aug 2017 02:59:18 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751532AbdH1Q7S (ORCPT );\n\tMon, 28 Aug 2017 12:59:18 -0400","from ganesha.gnumonks.org ([213.95.27.120]:38388 \"EHLO\n\tganesha.gnumonks.org\" rhost-flags-OK-OK-OK-OK) by vger.kernel.org\n\twith ESMTP id S1751309AbdH1Q7R (ORCPT\n\t);\n\tMon, 28 Aug 2017 12:59:17 -0400","from 129.166.216.87.static.jazztel.es ([87.216.166.129]\n\thelo=gnumonks.org) by ganesha.gnumonks.org with esmtpsa\n\t(TLS1.2:ECDHE_RSA_AES_128_GCM_SHA256:128) (Exim 4.84_2)\n\t(envelope-from )\n\tid 1dmNNG-0004Hx-DS; Mon, 28 Aug 2017 18:58:56 +0200"],"Date":"Mon, 28 Aug 2017 18:58:53 +0200","From":"Pablo Neira Ayuso ","To":"Phil Sutter ","Cc":"netfilter-devel@vger.kernel.org","Subject":"Re: [nft PATCH 1/2] nft.8: Fix and enhance synopsis section","Message-ID":"<20170828165853.GB21786@salvia>","References":"<20170817133900.21946-1-phil@nwl.cc>\n\t<20170817133900.21946-2-phil@nwl.cc>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170817133900.21946-2-phil@nwl.cc>","User-Agent":"Mutt/1.5.23 (2014-03-12)","X-Spam-Score":"-2.9 (--)","Sender":"netfilter-devel-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netfilter-devel@vger.kernel.org"}},{"id":1758857,"web_url":"http://patchwork.ozlabs.org/comment/1758857/","msgid":"<20170828212511.GF19224@orbyte.nwl.cc>","list_archive_url":null,"date":"2017-08-28T21:25:11","subject":"Re: [nft PATCH 1/2] nft.8: Fix and enhance synopsis section","submitter":{"id":4285,"url":"http://patchwork.ozlabs.org/api/people/4285/","name":"Phil Sutter","email":"phil@nwl.cc"},"content":"Hi Pablo,\n\nOn Mon, Aug 28, 2017 at 06:58:53PM +0200, Pablo Neira Ayuso wrote:\n> On Thu, Aug 17, 2017 at 03:38:59PM +0200, Phil Sutter wrote:\n> > This patch addresses shortcomings in the main synopsis section\n> > illustrating possible invocations of nft command:\n> > \n> > - Fix font styles to correctly put options into bold font and meta\n> > characters (brackets, pipes) into normal font.\n> > \n> > - Add missing options to synopsis line.\n> > \n> > - Use curly braces where either one of the alternatives is required.\n> > \n> > - Remove choice=\"opt\" attribute since that is the default anyway.\n> > \n> > - Note that --includepath option is allowed to be given multiple times.\n> \n> I'm going to apply this.\n\nThanks!\n\n> I'm undecided with 2/2,\n\nFrom my side, this is according to (the hidden) plan. :)\n\n> I don't remember to have seen such indirections in other manpages?\n\nFrom iproute2 (as said, I'm biased), I'm used to stuff like this:\n\t\n| nft [ OPTIONS ] ...\n|\n| OPTIONS := [ OPTIONS ] OPTION\n| OPTION := [ -a | -e | -n ]\n\nMaybe for others it's more of an obstacle to parse this recursive style,\nit really seems intuitive to me at least. And it's a quite efficient way\nto define the syntax in a very precise manner.\n\nOf course it's nice to have some layer of abstraction to allow for\nformatting into other markups, like HTML for example. So I'll have a\nlook at alternatives which will keep that flexibility yet allow for the\nstyle I prefer (unless you consider it not much more readable than what\ndocbook offers).\n\nThanks, Phil\n\n--\nTo unsubscribe from this list: send the line \"unsubscribe netfilter-devel\" in\nthe body of a message to majordomo@vger.kernel.org\nMore majordomo info at http://vger.kernel.org/majordomo-info.html","headers":{"Return-Path":"","X-Original-To":"incoming@patchwork.ozlabs.org","Delivered-To":"patchwork-incoming@bilbo.ozlabs.org","Authentication-Results":"ozlabs.org;\n\tspf=none (mailfrom) smtp.mailfrom=vger.kernel.org\n\t(client-ip=209.132.180.67; helo=vger.kernel.org;\n\tenvelope-from=netfilter-devel-owner@vger.kernel.org;\n\treceiver=)","Received":["from vger.kernel.org (vger.kernel.org [209.132.180.67])\n\tby ozlabs.org (Postfix) with ESMTP id 3xh54t6szfz9s72\n\tfor ;\n\tTue, 29 Aug 2017 07:49:02 +1000 (AEST)","(majordomo@vger.kernel.org) by vger.kernel.org via listexpand\n\tid S1751470AbdH1VZN (ORCPT );\n\tMon, 28 Aug 2017 17:25:13 -0400","from orbyte.nwl.cc ([151.80.46.58]:60373 \"EHLO mail.nwl.cc\"\n\trhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP\n\tid S1751218AbdH1VZM (ORCPT );\n\tMon, 28 Aug 2017 17:25:12 -0400","from mail.nwl.cc (orbyte.nwl.cc [127.0.0.1])\n\tby mail.nwl.cc (Postfix) with ESMTP id A0BD7644CF;\n\tMon, 28 Aug 2017 23:25:11 +0200 (CEST)","by mail.nwl.cc (Postfix, from userid 1000)\n\tid 8C05465A5E; Mon, 28 Aug 2017 23:25:11 +0200 (CEST)"],"Date":"Mon, 28 Aug 2017 23:25:11 +0200","From":"Phil Sutter ","To":"Pablo Neira Ayuso ","Cc":"netfilter-devel@vger.kernel.org","Subject":"Re: [nft PATCH 1/2] nft.8: Fix and enhance synopsis section","Message-ID":"<20170828212511.GF19224@orbyte.nwl.cc>","Mail-Followup-To":"Phil Sutter ,\n\tPablo Neira Ayuso ,\n\tnetfilter-devel@vger.kernel.org","References":"<20170817133900.21946-1-phil@nwl.cc>\n\t<20170817133900.21946-2-phil@nwl.cc>\n\t<20170828165853.GB21786@salvia>","MIME-Version":"1.0","Content-Type":"text/plain; charset=us-ascii","Content-Disposition":"inline","In-Reply-To":"<20170828165853.GB21786@salvia>","User-Agent":"Mutt/1.7.2 (2016-11-26)","X-Virus-Scanned":"ClamAV using ClamSMTP","Sender":"netfilter-devel-owner@vger.kernel.org","Precedence":"bulk","List-ID":"","X-Mailing-List":"netfilter-devel@vger.kernel.org"}}]