Merge pull request #239 from talentdeficit/as_comma_then_space

parse `rebar3 as foo, bar task` correctly

1 files changed, 12 insertions, 8 deletions

diff --git a/src/rebar_prv_as.erl b/src/rebar_prv_as.erl
index 7ac5465..beee00d 100644
--- a/src/rebar_prv_as.erl
+++ b/src/rebar_prv_as.erl
@@ -51,19 +51,23 @@ args_to_profiles_and_tasks(Args) ->
 first_profile([]) -> {[], []};
 first_profile([ProfileList|Rest]) ->
     case re:split(ProfileList, ",", [{return, list}, {parts, 2}]) of
-        %% profile terminated by comma
-        [P, More]       -> profiles([More] ++ Rest, [P]);
-        %% profile not terminated by comma
-        [P]           -> comma_or_end(Rest, [P])
+        %% `foo, bar`
+        [P, ""]   -> profiles(Rest, [P]);
+        %% `foo,bar`
+        [P, More] -> profiles([More] ++ Rest, [P]);
+        %% `foo`
+        [P]       -> comma_or_end(Rest, [P])
     end.
 
 profiles([], Acc) -> {lists:reverse(Acc), rebar_utils:args_to_tasks([])};
 profiles([ProfileList|Rest], Acc) ->
     case re:split(ProfileList, ",", [{return, list}, {parts, 2}]) of
-        %% profile terminated by comma
-        [P, More]  -> profiles([More] ++ Rest, [P|Acc]);
-        %% profile not terminated by comma
-        [P]      -> comma_or_end(Rest, [P|Acc])
+        %% `foo, bar`
+        [P, ""]   -> profiles(Rest, [P|Acc]);
+        %% `foo,bar`
+        [P, More] -> profiles([More] ++ Rest, [P|Acc]);
+        %% `foo`
+        [P]       -> comma_or_end(Rest, [P|Acc])
     end.
 
 %% `, foo...`