basho / cuttlefish

never lose your childlike sense of wonder baby cuttlefish, promise me?
Apache License 2.0
205 stars 124 forks source link

Added lager output for badly formated advanced.config #59

Closed joedevivo closed 10 years ago

joedevivo commented 10 years ago

As reported by @reiddraper in #49

[{riak_kv,
    [
        {some_option, 2}
        {some_other, 3}
    ]
}].

see the missing ',' on line 3?

Here's the output now:

➜  riak_ee git:(develop) ✗ ./rel/riak/bin/riak chkconfig
09:07:13.459 [info] Application lager started on node nonode@nohost
09:07:13.459 [info] Checking /Users/joe/dev/basho/riak_ee/rel/riak/bin/../etc/app.config exists... false
09:07:13.459 [info] Checking /Users/joe/dev/basho/riak_ee/rel/riak/bin/../etc/vm.args exists... false
09:07:13.459 [info] No app.config or vm.args detected in /Users/joe/dev/basho/riak_ee/rel/riak/bin/../etc, activating cuttlefish
09:07:13.676 [info] Adding Defaults
09:07:13.679 [info] Applying Datatypes
09:07:13.693 [info] Validation
09:07:13.697 [info] Applied 1:1 Mappings
09:07:13.699 [info] Applied Translations
09:07:13.699 [info] /Users/joe/dev/basho/riak_ee/rel/riak/bin/../etc/advanced.config detected, overlaying proplists
09:07:13.700 [error] Error parsing advanced.config: {4,erl_parse,["syntax error before: ","'{'"]}
Error generating config with cuttlefish
reiddraper commented 10 years ago

works great, +1, and thanks