HouzuoGuo/laitos

Allow spaces in the command line

kjcole opened this issue · 4 comments

Small feature request:

sudo ./laitos -gomaxprocs 16 -config sample-config.json.txt -daemons dnsd,httpd,insecurehttpd,maintenance,plainsocket,serialport,simpleipsvcd,smtpd,snmpd,sockd,telegram

is rather long, but breaking it up with \ newlines and spaces broke it. Making the command more "whitespace-tolerant" would be nice.

Thank you!

If you need a hand with setting laitos up - feel free to leave a message, I'm more than glad to help!

Hello! I just double checked how the daemons is parsed in main.go, and noticed that it in fact accepts any delimiter character between daemon names:

laitos/main.go

Line 235 in e40ce7e

daemonNames := regexp.MustCompile(`\w+`).FindAllString(daemonList, -1)

For example, this command line uses a line break in front of telegram daemon name, and uses a mixture of space and comma as delimiter character, the command was able to successfully launch the sample configuration file:

sudo ./laitos -gomaxprocs 16 -config sample-config.json.txt -daemons 'dnsd,httpd,insecurehttpd,maintenance,plainsocket,serialport,simpleipsvcd,smtpd,snmpd,sockd \
telegram’

How did yours look like?

Seems solved for now.