From: Thomas Petazzoni <thomas.petazzoni@free-electrons.com>
To: buildroot@busybox.net
Subject: [Buildroot] [PATCH v3 1/1] jsen: new package
Date: Thu, 25 Feb 2016 13:51:37 +0100 [thread overview]
Message-ID: <20160225135137.09136142@free-electrons.com> (raw)
In-Reply-To: <1456396337.29027.16.camel@embedded.rocks>
Hello,
On Thu, 25 Feb 2016 11:32:17 +0100, J?rg Krause wrote:
> > Atul, are you using jsen in the context of NodeJS ? If not, in which
> > context are you using it ?
>
> Using BR2_PACKAGE_NODEJS_MODULES_ADDITIONAL should be the only way to
> install Node.js modules. Note that npm, the Node.js package manager
> takes care of all the package dependencies. In my opinion, this is the
> only sane way of installing Node.js modules for now.
Thanks for raising your voice on this patch, it's definitely
interesting to get some feedback. Since I'm not at all a NodeJS guy,
I'm interested in having more details about:
- Why you think the only sane way is to use
BR2_PACKAGE_NODEJS_MODULES_ADDITIONAL as opposed to having proper
packages for the different modules, like we do for the Perl,
Python and Lua packages ?
- What solution you propose to properly integrate this with the
download and legal infrastructure of Buildroot. Right now, having
"npm install" directly download and install stuff means that the
download and legal-info infrastructure of Buildroot is completely
worked-around. Due to this, "make source" will not download all the
source, "make legal-info" will not list all the licenses, caching in
BR2_DL_DIR doesn't work, BR2_PRIMARY_SITE doesn't work, etc.
> About legal infrastructure: for all installed Node.js packages the
> package.json file could be searched for the license information.
Yes, but that's not integrated in the legal-info output, and doesn't
solve the download infrastructure integration problem.
At best, what this package.json file could help with, is help creating
a script (much like scancpan) that automatically creates a Buildroot
package for a given npm module.
Best regards,
Thomas
--
Thomas Petazzoni, CTO, Free Electrons
Embedded Linux, Kernel and Android engineering
http://free-electrons.com
next prev parent reply other threads:[~2016-02-25 12:51 UTC|newest]
Thread overview: 10+ messages / expand[flat|nested] mbox.gz Atom feed top
2016-02-23 13:24 [Buildroot] [PATCH v3 1/1] jsen: new package Atul Singh
2016-02-23 20:44 ` Arnout Vandecappelle
2016-02-23 21:04 ` Thomas Petazzoni
2016-02-25 10:32 ` Jörg Krause
2016-02-25 12:51 ` Thomas Petazzoni [this message]
2016-02-25 14:01 ` Jörg Krause
2016-02-25 16:20 ` Thomas Petazzoni
2016-03-28 14:01 ` Jörg Krause
2016-03-28 14:07 ` Thomas Petazzoni
2016-04-25 15:45 ` Matthew Weber
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=20160225135137.09136142@free-electrons.com \
--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