Plugin fails if Elixir lib dir contains spaces. about rebar_mix HOT 3 OPEN

Thanks. Pls send a PR if you have time.

from rebar_mix.

I had the same issue
Here in rebar_mix/src/rebar_mix_utils.erl, in the function get_lib_dir (line 42-53) we have the following case expression

case rebar_utils:sh("elixir -e \"IO.puts :code.lib_dir(:elixir)\"", [return_on_error]) of
    {ok, ElixirLibs_} ->
        filename:join(re:replace(ElixirLibs_, "\\s+", "", [global,{return,list}]), "../");
    _ ->
        erlang:error(missing_elixir)
end;

The insteresting part is re:replace(ElixirLibs_, "\\s+", "", [global,{return,list}])
It basically takes the path to Elixir and removes all whitespaces from it. And so if our path happens to start with, for instance, C:\Program Files (default when installing Elixir on windows), it becomes C:\ProgramFiles, resulting in an error.
I spent quite some time trying to figure out what the problem was, so I thought it is worth mentioning it

I'm not sure if it is a bug. I assume there is a valid reason why we would remove whitespaces from the path, and so I don't know if we should do something about it or keep it as is. I do think that it should be mentioned in the README, though. (e.g. in the requiremets section, we might include something like "Make sure that the path to your Elixir installation contains no significant whitespaces". I beleive, that could save a lot of debagging time for the future users).

from rebar_mix.

No there's no good reason to remove whitespace from a path, it just makes it wrong. The proper fix is to escape and/or quote the path adequately.

from rebar_mix.