On Wed, Nov 27, 2013 at 01:27:16PM -0800, Sathyanarayanan wrote:
> > +        }
> > +        if (first_invalid[0] != '\0') {
> > +            log_error("invalid experiment timeout: ");
> > +            censorscope_options_destroy(options);
> > +            return -1;
> > +        }
> > +    }
> > +    else {
> > +        return -1;
> > +    }
> > +
> > +    return 0;
> > +}
> > +
> > +int parse_config_file(censorscope_options_t *options) {
> > +    int error = ini_parse(".censorscope", config_file_handler, options);
> 
> I vote for checking in `~/.censorscope` or `/etc/censorscope/censorscope.conf` and adding an option to the command line arguments (`--config`). Thoughts?

I don't think Lua will let you do '~', so
/etc/censorscope/censorscope.conf it is.
I would still add a build flag to change the default.

Adding a command line option sounds good, but
might be annoying to implement because command
line parsing happens after you parse the config file.


> ---
> Reply to this email directly or view it on GitHub:
> https://github.com/projectbismark/censorscope/pull/54/files#r7970207