Re: Documentation of device spec list does not agree with parser code



On 17/01/16 21:04, Thomas Haller wrote:
On Fri, 2016-01-15 at 18:21 +0000, Roger James wrote:
Hi,

As far as I can see the code for parsing device interface-name lists
in 
NetworkManagerUtils.c does not agree with the man page in 
NetworkManager.conf.xml.in. In particular

interface-name:IFNAME

does not support globbing. Only

interface-name:?IFNAME
Hi Roger,

where do you read "interface-name:?IFNAME". AFAIS, that is not
mentioned by upstream manpage. See.

It's either:

 interface-name:=<LITERAL>

or

  interface-name:~<PATTERN>

(whereas, omitting the ~ is allowed:

  interface-name:<PATTERN>

http://cgit.freedesktop.org/NetworkManager/NetworkManager/tree/man/NetworkManager.conf.xml.in?id=7a2a96f8ef364e46f9c99798743c11f1ea02659d#n920



does.

Here is the current code.
I don't see anything wrong. Can you be more specific?


There is also a unit-test: http://cgit.freedesktop.org/NetworkManager/NetworkManager/tree/src/tests/test-general.c?id=7a2a96f8ef364e46f9c99798743c11f1ea02659d#n811


ciao
Thomas
Hi Thomas,

If I follow the link above, the file I arrive at has line 920-921 that looks like this

          <term>interface-name:IFNAME</term>
          <term>interface-name:~IFNAME</term>

That is looks the same as the one in my tree, which has its origin at
origin	git://anongit.freedesktop.org/NetworkManager/NetworkManager.git (fetch)
origin	git://anongit.freedesktop.org/NetworkManager/NetworkManager.git (push)
There is no mention of PATTERN in the file.

Roger


[Date Prev][Date Next]   [Thread Prev][Thread Next]   [Thread Index] [Date Index] [Author Index]