config: Correctly match valueless parameters. Fixes #693

[bramton]

The current RE does not match valueless parameters properly. For instance, if the jail config file has a single line with allow.mount; the field in the AWK program will have the value allow.mount;. This is currently not matched due to the trailing ;. This PR allows for zero or one trailing ; in the field.

This resolves #693 and #702 (comment)

[tschettervictor]

What does the "config" command look like before and after this patch?

[bramton]

The behaviour of the config command is not changed. To give an example what this PR fixes consider the following fictive jail.conf file:

woot {
  host.hostname = woot;
  allow.mount;
  ip6 = enable;
}

Now if we run:

bastille config woot set allow.mount
bastille config woot set allow.mount

The resulting jail.conf now looks like this:

Before this PR

woot {
  host.hostname = woot;
  allow.mount;
  ip6 = enable;
  allow.mount;
  allow.mount;
}

After this PR

woot {
  host.hostname = woot;
  allow.mount;
  ip6 = enable;
}

[tschettervictor]

Ah. I was thinking of the "get" command, which worked ok for me. But you're referring to "set".

I see.
Let me try this.

[tschettervictor]

Seems to work, along with setting properties that include values.

[tschettervictor]

@yaazkal @bmac2 Can you also test?

[bmac2]

worked as advertised. Merging

[louwe]

Thanks for fixing this. May I inquire if this will make it into the next release and when we should expect the version to be released to LATEST package repos?

[bmac2]

@louwe you can get it now installing from github. Hopefully we will have a new release out soon. Got a couple of more PRs to get done before next release.