Yes, seems reasonable, Juan. I don't believe it would break anything.<div><br></div><div>D.<br><br><div class="gmail_quote">On Tue, Jun 18, 2013 at 9:41 AM, Tuncer Ayaz <span dir="ltr"><<a href="mailto:tuncer.ayaz@gmail.com" target="_blank">tuncer.ayaz@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="im">On Tue, Jun 18, 2013 at 5:02 PM, Juan Jose Comellas wrote:<br>
> Tuncer, is there a reason why the rebar lib_dirs configuration<br>
> setting only allows relative paths? Right now the function that<br>
> expands these paths looks like this:<br>
<br>
</div>I'm not aware of any reason and I think you're the first to request<br>
support for absolute paths.<br>
<div class="im"><br>
> expand_lib_dirs([], _Root, Acc) -><br>
> Acc;<br>
> expand_lib_dirs([Dir | Rest], Root, Acc) -><br>
> Apps = filelib:wildcard(filename:join([Dir, "*", "ebin"])),<br>
> FqApps = [filename:join([Root, A]) || A <- Apps],<br>
> expand_lib_dirs(Rest, Root, Acc ++ FqApps).<br>
><br>
> Changing it to something like this would resolve this problem:<br>
><br>
> expand_lib_dirs([], _Root, Acc) -><br>
> Acc;<br>
> expand_lib_dirs([Dir | Rest], Root, Acc) -><br>
> Apps = filelib:wildcard(filename:join([Dir, "*", "ebin"])),<br>
> FqApps = case filename:pathtype(Dir) of<br>
> absolute -> Apps;<br>
> relative -> [filename:join([Root, A]) || A <- Apps]<br>
> end,<br>
> expand_lib_dirs(Rest, Root, Acc ++ FqApps).<br>
><br>
> Would this addition be accepted into rebar? I haven't created a pull<br>
> request for this because I don't know what the rationale was for<br>
> making these paths relative.<br>
<br>
</div>This looks reasonable to me, but I can't speak for Dave.<br>
<div class="HOEnZb"><div class="h5">_______________________________________________<br>
erlang-questions mailing list<br>
<a href="mailto:erlang-questions@erlang.org">erlang-questions@erlang.org</a><br>
<a href="http://erlang.org/mailman/listinfo/erlang-questions" target="_blank">http://erlang.org/mailman/listinfo/erlang-questions</a><br>
</div></div></blockquote></div><br></div>