Add a command line option to verify configuration and exit

[thedrow]

It would be useful for our CI process to be able to verify the configuration of smcrouted.
Would you accept a PR that adds a command line option to smcrouted that says only verify configuration and exit?

[troglobit]

Sure, I've done something similar for troglobit/inadyn@fd3ef93, so that would be great. How deep are you planning to go, verify interface names and group addresses, or just syntax?

[thedrow]

I thought about simply calling conf_parse(), so whatever it does is fine with me.
We can add more functionality later on right?

[thedrow]

Is -C acceptable as a command line option?

[troglobit]

Sounds reasonable, we can add more functionality later.

Hmm, -C is OK I guess, but what do you think about this?

    ...
#ifdef ENABLE_DOTCONF
               "  -f FILE         Set configuration file, default uses ident NAME: %s\n"
               "  -F FILE         Check configuration file syntax, -l LVL to increase verbosity\n"
#endif
    ...

[thedrow]

Also acceptable.
conf_parse() is not part of the header right now. Are you planning to change the signature or can we make it "public"?

[troglobit]

Go ahead and change what you need. Personally I might've refactored conf_read() a bit instead, e.g. moved the access() check and error handling to conf_parse(), but it's up to you :-)

[troglobit]

Any progress on this?

[thedrow]

I haven't had the chance to reach this yet. You'll see a PR once I will.
I hope it will be soon.

[troglobit]

OK, just checking :)

[troglobit]

@thedrow Hi again. I just added a very simple .conf verifyer for the upcoming v2.5 release. I'll see what I can do about making it a bit more strict; error (return 1) on malformed directives and warn on unknown directives.