(Mix) Could not compile dependency :chatterbox

link · August 17, 2018, 8:45am

(Mix) Could not compile dependency :chatterbox, “/home/user/.mix/rebar3 bare compile --paths “/home/user/Documents/signalbroker/signal_server/_build/dev/lib/*/ebin”” command failed. You can recompile this dependency with “mix deps.compile chatterbox”, update it with “mix deps.update chatterbox” or clean it with “mix deps.clean chatterbox”

I have tried all the three suggestions, still get the same error.
I use ubuntu 18, 64 bit

NobbZ · August 17, 2018, 8:57am

Probably because they have warnings_as_errors in their rebar default profile… Please checkout the repository in the exact version you want to use in your project, try to build and then file a bug report in their GitHub repository, asking to move the warnings_as_errors setting as well, as it breaks projects that depend on it.

link · August 17, 2018, 11:55am

How do I know which exact version I need in my project?

NobbZ · August 17, 2018, 12:04pm

mix deps | grep chatterbox IIRC.

link · August 17, 2018, 12:58pm

chatterbox (https://github.com/tony612/chatterbox.git) (rebar3)
This is what I get from the command you sent. I guess it gives me which version I have, not which version I should have?

NobbZ · August 17, 2018, 1:23pm

It looks like you have a git dependency, not constraint by any branch/tag/commit, so you get whatever HEAD was on the first deps.get, it should be fine to simply try with HEAD from the repo and create the issue based on that.

NobbZ · August 17, 2018, 1:29pm

PS: they are using gen_fsm which is a not trivial to fix issue, depending on which versions of OTP they want to support.

Easiest for now should be to fork/vendor a version that has the warnings_as_errors removed from the config.

link · August 20, 2018, 9:42am

Thanks, removing the warnings_as_errors fixed the problem.

NobbZ · August 20, 2018, 9:44am

Have you made them aware of the problems they are causing by that setting?

tristan · August 20, 2018, 3:49pm

I’m fine with removing it from the default profile. Please send a PR to chatterbox and I’ll merge it.

Also note that in the latest gen_fsm has been replaced by gen_statem.

(I just realized you are using a fork of chatterbox… Maybe tony612 would be willing to upgrade his fork to the latest upstream also).

link · August 31, 2018, 10:20am

No, how do I do that?

link · August 31, 2018, 10:21am

I don’t know why I got that, I did not change any default configuration.

josevalim · August 31, 2018, 10:30am

What is your Elixir version? Recent Elixir versions are meant to automatically remove warnings_as_errors from dependencies, even rebar ones.

NobbZ · August 31, 2018, 10:47am

Open an issue on the GitHub repository.

NobbZ · August 31, 2018, 10:48am

But rebar won’t, so I still think it should be removed from library projects default profile.

josevalim · August 31, 2018, 11:45am

Mix tells rebar exactly which config and profile to use and we make sure to not pass warnings_as_errors, so IIRC it should work as well.

NobbZ · August 31, 2018, 11:48am

I mean in erlang projects. Completely unrelated to elixir.

link · August 31, 2018, 1:05pm

It is 1.7.2

tristan · September 4, 2018, 2:58pm

I assume you are using the Elixir grpc package? That uses a fork of chatterbox.

I think it still needs an outstanding PR to work with upstream. I have an open PR I’m working on getting into chatterbox that allows it to work with my grpc server/client package (grpcbox), so hopefully we can all consolidate back to the upstream repo in the near future.

link · September 10, 2018, 11:07am

Yes the project supports grpc…