[erlang-patches] Add support for dependencies Makefile
Björn Gustavsson
bgustavsson@REDACTED
Tue Dec 15 00:10:52 CET 2009
On Mon, Dec 14, 2009 at 10:00 PM, Jean-Sébastien Pédron
<jean-sebastien.pedron@REDACTED> wrote:
> The GIT branch may be fetched using the following command:
> git fetch git://github.com/dumbbell/otp.git dependencies_makefile
Thanks! It is good to see that you have written both documentation and
a test suite.
I have squashed your branch into one commit, retaining the commit
message from the first commit, because your branch contains 4 commits
and a merge without any good reason. If that is OK for you, you can
base your further work on that version. See
http://wiki.github.com/erlang/otp/branches
for how you can set up local branch based on the branch I have merged
into 'pu'. (If my squash is not OK, please clean up your commit
history yourself and email me the reference to your branch.)
The code style needs some work before we can consider including it in
Erlang/OTP. Here are the major things I saw when glancing at your
code:
We always write two percent characters ("%%"), not only one ("%") for
comment lines not preceded by code.
We don't use spaces in around '=' in function heads. We prefer :
makedep(#compile{code=Code,options=Opts}=St)
In the compiler application, we import functions from the lists
module; we don't write lists:member(...).
In the compiler application, we don't have spaces after commas in
terms (but we do put spaces after commas in function calls).
In the same function, please use either proplists:get_value/2 or
member/2 consistently. (I recommend proplists:get_value/2.)
We write variables without underscores: LineLen, not Line_Len.
Don't use throw from inside a try block in makedep_output/1. As far as
I can see, the only function call that you will need to catch is the
one to io:fwrite/2, so just put that function call in a try block.
Don't use library calls like this
File1 = case lists:prefix("./", File) of
true -> lists:nthtail(2, File);
false -> File
end,
when matching will do
File1 = case File of
"./"++File2 -> File2
_ -> File
end,
(NOT TESTED, but it should work)
--
Björn Gustavsson, Erlang/OTP, Ericsson AB
More information about the erlang-patches
mailing list