advertise on not woking anymore

[fredsco]

After upgrading rebar the gateway_config commands aren't working..

/home/pi/gateway_config/_build/prod/rel/gateway_config/bin/gateway_config start
 

the commando used to work but now I get an error to use "daemon" instead

Now the following command:

/home/pi/gateway_config/_build/prod/rel/gateway_config/bin/gateway_config advertise on

gives me:

Error: args `advertiseon` not a list.
Example to call `foo:bar(1,2,3)` pass `foo bar "[1, 2, 3]."`

(FYI I really typed advertise on and not advertiseon)

I wasn't able to figure out what the right command is to turn on advertisement over bluetooth. I did see a new commit 2 hours ago, I wasn't able to extract how to run the command from the source code unfortunately.

[madninja]

Would you be willing to try branch madninja/fix_cli in PR #79 and see if that fixes it, please?

[fredsco]

Thanks for your reply @madninja

sudo /home/pi/gateway_config/_build/prod/rel/gateway_config/bin/gateway_config advertise on

Now gives me :

ok
ok

It works. FYI before it used to be one "ok".

But everything seems to be working now, thank you!

[madninja]

In can fix the double ok.. Thanks for testing!

[madninja]

Thanks for your reply @madninja

sudo /home/pi/gateway_config/_build/prod/rel/gateway_config/bin/gateway_config advertise on

Now gives me :

ok
ok

It works. FYI before it used to be one "ok".

But everything seems to be working now, thank you!

one more try after pulling that branch if you could, just to confirm

[fredsco]

@madninja I just tried again after fetch and pull. It works but double "ok" is still there.

[madninja]

@madninja I just tried again after fetch and pull. It works but double "ok" is still there.

Agh sorry.. forgot a couple of spots.. should be fixed on that branch now

[fredsco]

Thank you @madninja I still see the double ok, i removed the gateway_config in _build/prod/rel/gateway_config/bin/gateway_config FYI and did a make && make release again. But two times ok is not a issue!

Do you know when you will merge it to master? I'd really appreciate it as I would like to run some automated scripts!