Bug 256042

Summary: net-im/matterircd Wrong statup options in rc file
Product: Ports & Packages Reporter: Julien Dessaux <julien.dessaux>
Component: Individual Port(s)Assignee: freebsd-ports-bugs (Nobody) <ports-bugs>
Status: New ---    
Severity: Affects Only Me CC: norrland
Priority: --- Flags: norrland: maintainer-feedback+
Version: Latest   
Hardware: Any   
OS: Any   
Attachments:
Description Flags
proposed patch none

Description Julien Dessaux 2021-05-20 22:02:52 UTC
Created attachment 225137 [details]
proposed patch

Hi,

The options in the rc file are invalid therefore a fresh installation of matterircd cannot start. It will fail with a daemon log that looks like :
May 20 22:48:41 matterircd matterircd[40090]: unknown shorthand flag: 'c' in -conf=/usr/local/etc/matterircd.toml
May 20 22:48:41 matterircd matterircd[40090]: Usage of /usr/local/bin/matterircd:
May 20 22:48:41 matterircd matterircd[40090]:       --bind string      interface:port to bind to, or a path to bind to a Unix socket. (default "127.0.0.1:6667")
May 20 22:48:41 matterircd matterircd[40090]:       --conf string      config file (default "matterircd.toml")
May 20 22:48:41 matterircd matterircd[40090]:       --debug            enable debug logging
May 20 22:48:41 matterircd matterircd[40090]:       --tlsbind string   interface:port to bind to. (e.g 127.0.0.1:6697)
May 20 22:48:41 matterircd matterircd[40090]:       --tlsdir string    directory to look for key.pem and cert.pem. (default ".")
May 20 22:48:41 matterircd matterircd[40090]:       --version          show version

Adding a second dash to change -conf= into --conf= fixes the issue.

Patch is attached.
Comment 1 norrland 2021-05-21 09:52:51 UTC
Looks good, thanks for catching this.