Skip to content
This repository was archived by the owner on Jan 6, 2023. It is now read-only.

packagegroup-iotivity: Add iotivity-node to meta-oic's metapackage#30

Open
rzr wants to merge 3 commits into
ostroproject:masterfrom
TizenTeam:sandbox/pcoval/on/master/review
Open

packagegroup-iotivity: Add iotivity-node to meta-oic's metapackage#30
rzr wants to merge 3 commits into
ostroproject:masterfrom
TizenTeam:sandbox/pcoval/on/master/review

Conversation

@rzr
Copy link
Copy Markdown
Contributor

@rzr rzr commented Feb 14, 2017

Change-Id: I3a8e79427f2766aa426458322ddaaf6d3b4c4676
Signed-off-by: Philippe Coval philippe.coval@osg.samsung.com

@ostroproject-ci
Copy link
Copy Markdown

Can one of the admins verify this patch?

@mythi
Copy link
Copy Markdown
Contributor

mythi commented Feb 15, 2017

is it not possible to use nodejs-runtime packagegroup?

@rzr
Copy link
Copy Markdown
Contributor Author

rzr commented Feb 15, 2017

I think it is but I had in mind to extends that iotivity metapackage to all related projects, well it can be done twice or once , not a big deal.

rzr added 3 commits June 14, 2017 16:34
Observed issue (on GENIVI) is:

  ERROR: ExpansionError during parsing \
  (...)/meta-iot-web/recipes-devtools/nodejs/nodejs_6.11.0.bb
  Traceback (most recent call last):
  bb.data_smart.ExpansionError: \
  Failure expanding variable do_configure, expression was \
  export (...) \
   ./configure --prefix=/usr --without-snapshot --shared-openssl \
               --dest-cpu="${@map_nodejs_arch(d.getVar('TARGET_ARCH'), d)}" \
  (...)
  which triggered exception TypeError: \
  getVar() missing 1 required positional argument: 'expand'

Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
This prevent layer's users to overload recipe
if not using static users/group table (USERADD_UID_TABLES).

Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
Change-Id: I3a8e79427f2766aa426458322ddaaf6d3b4c4676
Signed-off-by: Philippe Coval <philippe.coval@osg.samsung.com>
@rzr rzr force-pushed the sandbox/pcoval/on/master/review branch from f297575 to f5b3647 Compare June 15, 2017 12:36
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants