You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Chromium OS dev
Hi, I am trying to add a new package. The CL is crosreview.com/781599.
npm is required for building a package.
I found third_party/chromiumos-overlay/net-libs/iojs. Looks like it
installs npm. Is it correct?
If yes, I think I can use npm to build shaka-player if I add
net-libs/iojs to the dependency of autotest-deps-shaka.ebuild.
I actually do so. However, I got errors when building iojs. The error
log is attached.
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Hirokazu Honda, Chromium OS dev
currently nodejs/npm are not available inside the sdk. there was one project that was using node on the board itself, but i don't think that was ever released, so you'll find random references like iojs in the tree.
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Mike Frysinger, Chromium OS dev
Thank you for the information.
So it is required to build iojs so that a package is built with npm.
As I said, building iojs is failed currently.
Should I file this bug?
-Hiro
Mike Frysinger
unread,
Dec 8, 2017, 2:20:12 PM12/8/17
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Hirokazu Honda, chromium-os-dev
no one uses iojs, so I don't think a bug would help if your expectation is to have someone fix it. if you need the package, then you'd probably be on the hook to fix it. if you want to post a cl to delete it, seems like it would be reviewed quickly.
-mike
Hirokazu Honda
unread,
Dec 25, 2017, 10:32:46 PM12/25/17
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Mike Frysinger, chromium-os-dev
I just uploaded the CL to update iojs version.
https://chromium-review.googlesource.com/c/chromiumos/overlays/chromiumos-overlay/+/844051 I confirmed iojs was built successfully. With the iojs, I tried to
build shaka-player, but I got an error which there is no npm in the
build environment.
> >>> Configuring source in /build/cyan/tmp/portage/chromeos-base/autotest-deps-shaka-2.2.6/work/shaka-player-2.2.6 ...
> /build/cyan/tmp/portage/chromeos-base/autotest-deps-shaka-2.2.6/temp/environment: line 1089: npm: command not found
npm is installed /build/cyan/usr/bin/npm by iojs ebuild.
Best,
-Hiro
Mike Frysinger
unread,
Jan 4, 2018, 3:19:04 AM1/4/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Hirokazu Honda, chromium-os-dev
/build/xxx/ is the sysroot. you can only compile/link against files in there. npm is a tool that executes in the sdk build system. we cannot (by design) execute programs out of the sysroot. build time tools have to be part of the SDK itself. see the virtual/target-chromium-os-sdk package.
-mike
Hirokazu Honda
unread,
Jan 9, 2018, 4:14:23 AM1/9/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Mike Frysinger, chromium-os-dev
Ah, I misunderstand what you said.
Adding npm to sdk is needed in order to use npm when building the
package in sysroot.
Concretely, net-libs/nodejs is added to RDEPEND in
virtual/target-chromium-os-sdk.
I don't know this is allowed though.
Apparently, although iojs is not related to this issue, I am willingly
going to update it as I said above.
Mike Frysinger
unread,
Jan 9, 2018, 4:10:47 PM1/9/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Hirokazu Honda, chromium-os-dev
looks like nodejs is built from source rather than a binpkg. that's what we'd want.
whether we can bundle it in the SDK by default comes down largely to the question of "how much disk space does this new package (and any new deps) add to the system?".
-mike
Hirokazu Honda
unread,
Jan 9, 2018, 10:33:19 PM1/9/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Mike Frysinger, chromium-os-dev
I measured the size before and after nodejs is added to ChromiumOS SDK
in the following way.
$ cros_sdk --delete && cros_sdk --enter
$ du -shb chroot
6518874919 chroot
$ # add net-libs/nodejs to ChromiumOS SDK
$ cros_sdk --delete && cros_sdk --enter
$ du -shb chroot
6558798401 chroot
I am not sure it is a right way though.
The growth is about 39Megabytes.
What do you think?
Mike Frysinger
unread,
Jan 9, 2018, 11:13:05 PM1/9/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message
to Hirokazu Honda, chromium-os-dev
if nodejs doesn't pull in anything new, you can just run `qsize nodejs` to get details
-mike
Hirokazu Honda
unread,
Jan 9, 2018, 11:30:03 PM1/9/18
Reply to author
Sign in to reply to author
Forward
Sign in to forward
Delete
You do not have permission to delete messages in this group
Copy link
Report message
Sign in to report message
Show original message
Either email addresses are anonymous for this group or you need the view member email addresses permission to view the original message