Refactor inadyn_cmd.c into a conf.c and a getopt_long() in main() about inadyn HOT 10 CLOSED

Maybe we could use lex/yacc for config parsing?

from inadyn.

Yes, I was thinking about stealing and modifying http://github.com/troglobit/mrouted/blob/master/cfparse.y,
which is another project I maintain. Should be a cake walk :)

from inadyn.

Hi,

Anyone working on this?

from inadyn.

Yes, I'm actually working on this now (finally!) -- the current parser is however not based on lex/yacc, but a simple tokenizer. The idea is however to reduce the amount of command-line options and keep what's for the .conf file in the .conf file ...

from inadyn.

For some systems(for example like my OpenELEC media center system) command line options much easier I guess.

Like my XBMC add-on, very easy to parse options from add-on's option menu and execute inadyn directly with that options.
Example: https://github.com/queeup/service.inadyn/blob/frodo/service.py#L36

But at the end you will decide what is it better for your project.

from inadyn.

@queeup Hmm, yeah I guess there will be a few comments from disgruntled users on this. Nothing is set in stone yet and no code is yet available to present as an alternative to the current implementation.

In your case, however, it doesn't look like rocket science to rewrite the service.py plugin to create a tempfile to feed your settings to inadyn ... or am I missing something obvious?

from inadyn.

No you are not missing anything :) Yes thats what I am going to do if you are going to take out things from cli.

Really I am not against anything. Just share my thought. Do it what is it better for inadyn. I will find a way to adapt :)

from inadyn.

@queeup OK, just wondered if you for some reason perhaps didn't have any writeable temporary filesystem ... kinda differs a lot between various embedded systems. Thank you for being so kind!

from inadyn.

Progress: http://troglobit.com/blog/2014/02/23/weekend-hack-json-vs-conf/

iface         = IFNAME
bind          = IFNAME
cache-dir     = DIR

period        = SEC
forced-update = SEC
once          = True
iterations    = NUM
proxy-server  = NAME[:PORT]
wildcard      = True
fake-address  = True

# For custom DDNS providers
checkip-url   = URL
server-name   = URL

# For known DDNS providers
system PROVIDER {
   alias ...
   username ...
   password ...
   ssl = True

}

from inadyn.

Mostly fixed now, as of 3f8c0b6. Still a lot of testing remains and there are a few options introduced late in the 1.x cycle that are still to be ported.

from inadyn.