From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH] erlang: Add new package.
Date: Fri, 17 Aug 2012 19:33:40 +0200 [thread overview]
Message-ID: <20120817193340.39e7ddb5@skate> (raw)
In-Reply-To: <CAFbHwiTCXTT23fGLnz_w8kr+6NmNzV9kK+=nHYpXOY5NOMt1dg@mail.gmail.com>
Hello Will,
Le Wed, 4 Apr 2012 17:49:07 +0100,
Will Newton <will.newton@gmail.com> a ?crit :
> ---
> package/Config.in | 1 +
> package/erlang/Config.in | 9 +++++++++
> package/erlang/erlang-build-fix.patch | 13 +++++++++++++
> package/erlang/erlang.mk | 26 ++++++++++++++++++++++++++
> 4 files changed, 49 insertions(+), 0 deletions(-)
> create mode 100644 package/erlang/Config.in
> create mode 100644 package/erlang/erlang-build-fix.patch
> create mode 100644 package/erlang/erlang.mk
Thanks for this.
A couple of questions/comments:
*) Could you update the patch to use autotools-package and
host-autotools-package in accordance with the latest package
infrastructure changes?
*) Could you add your Signed-off by line?
*) The complete Erlang stuff gets installed in /usr/lib/erlang, even
the binaries (compiler, interpreter?) in /usr/lib/erlang/bin/. Is this
the normal installation location? If so, then we should at least
probably provide a few symlinks from /usr/bin/.
*) By default the /usr/lib/erlang/ size is 55 MB on ARM. Is this the
minimal size of the standard library?
*) From the configure.in script, I see options like shared-zlib, or
ssl path or things like that, but your erlang package does not depend
on zlib nor OpenSSL. Can you give a few comments about this?
Thanks!
Thomas
--
Thomas Petazzoni, Free Electrons
Kernel, drivers, real-time and embedded Linux
development, consulting, training and support.
http://free-electrons.com
next prev parent reply other threads:[~2012-08-17 17:33 UTC|newest]
Thread overview: 4+ messages / expand[flat|nested] mbox.gz Atom feed top
2012-04-04 16:49 [Buildroot] [PATCH] erlang: Add new package Will Newton
2012-08-17 17:33 ` Thomas Petazzoni [this message]
2012-08-22 13:00 ` Will Newton
2012-08-22 13:41 ` Thomas Petazzoni
Reply instructions:
You may reply publicly to this message via plain-text email
using any one of the following methods:
* Save the following mbox file, import it into your mail client,
and reply-to-all from there: mbox
Avoid top-posting and favor interleaved quoting:
https://en.wikipedia.org/wiki/Posting_style#Interleaved_style
* Reply using the --to, --cc, and --in-reply-to
switches of git-send-email(1):
git send-email \
--in-reply-to=20120817193340.39e7ddb5@skate \
--to=thomas.petazzoni@free-electrons.com \
--cc=buildroot@busybox.net \
/path/to/YOUR_REPLY
https://kernel.org/pub/software/scm/git/docs/git-send-email.html
* If your mail client supports setting the In-Reply-To header
via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line
before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox