Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Jump to bottom

How to deal with npm packages that are only built on host side #297

Open
hellow554 opened this issue Dec 7, 2020 · 2 comments
Open

How to deal with npm packages that are only built on host side #297

hellow554 opened this issue Dec 7, 2020 · 2 comments

Comments

@hellow554
Copy link

I want to develop an application using typescript. This app should only be compiled on the hostside, therefore I don't need nodejs on the target.

If I understand the npm class correctly i will have nodejs as an runtime dependency. That's why I don't want to use inherit npm.

So, what is an good approach to solve my problem? Should I use depend on nodejs-native and implement do_compile and do_install myself? There's a lot of work put in the bbclass, e.g. picking the right target arch, using the correct npm flags as well as having a DEV flag to allow development builds.

I certanly don't want to implement (copy) them myself. Would it be a good approach to have a seperate npm-native class maybe?

What is your opinion about that?
@kraj
Copy link
Contributor

kraj commented Dec 7, 2020

not sure about npm-native class, perhaps a good idea if we are going to have many host packages to recompile otherwise option 2 is the way to go.

@hellow554
Copy link
Author

So you think it's not worth investing in a npm-native class? I certainly don't have enough experience to write such class for the public ;)

kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Mar 30, 2024
@alperak @kraj
python3-validators: upgrade 0.22.0 > 0.24.0 and enable ptest
df763fc 
License-Update: Rename LICENSE to LICENSE.txt and update copyright year

* Ptest and library example tested on qemux86-64 and qemuarm64

* Add ptest into PTESTS_FAST_META_PYTHON

qemux86-64:

Testsuite summary
TOTAL: 599
PASS: 599
SKIP: 0
XFAIL: 0
FAIL: 0
XPASS: 0
ERROR: 0
DURATION: 19
END: /usr/lib/python3-validators/ptest
2024-03-30T05:07
STOP: ptest-runner
TOTAL: 1 FAIL: 0

qemuarm64:

Testsuite summary
TOTAL: 599
PASS: 599
SKIP: 0
XFAIL: 0
FAIL: 0
XPASS: 0
ERROR: 0
DURATION: 18
END: /usr/lib/python3-validators/ptest
2024-03-30T05:10
STOP: ptest-runner
TOTAL: 1 FAIL: 0

Changelog:

0.24.0
===========
Features

    feat: conditionally raises ValidationError; bump version by @yozachar in openembedded#343

Maintenance

    patch: domain & url modules by @yozachar in openembedded#339
    fix: domain name not confirming to rfc_2782 by @yozachar in openembedded#341
    maint: update dev dependencies; adds favicon to docs by @yozachar in openembedded#342

0.23.2
===========
Maintenance

    maint: rectifies changelog by @yozachar in openembedded#336
    fix: packaging as well as rST & md document generation by @yozachar in openembedded#337

0.23.1
===========
Maintenance

    maint: fix between & length validators by @yozachar in openembedded#334
    fix: manual nav reference for mkdocs; bumps version by @yozachar in openembedded#335

0.23.0
===========
Features

    feat: add french i18n validation by @imperosol in openembedded#308

Maintenance

    fix: Valid URLs failing validation - query and fragment parts by @danherbriley in openembedded#297
    fix: bug in between module by @yozachar in openembedded#301
    chore: update dependencies, improve packaging by @yozachar in openembedded#304
    Fix fragment check by @darkdragon-001 in openembedded#305
    build(deps): bump urllib3 from 2.0.6 to 2.0.7 in /package by @dependabot in openembedded#310
    fix: allow pct-encoded entities in fragments by @conitrade-as in openembedded#317
    chore: update dev dependencies by @yozachar in openembedded#318
    build(deps): bump gitpython from 3.1.37 to 3.1.41 in /package by @dependabot in openembedded#321
    build(deps): bump jinja2 from 3.1.2 to 3.1.3 in /package by @dependabot in openembedded#322
    chore: monthly updates for Jan'24 by @yozachar in openembedded#324
    maint: adds versiond docs; update copyright year by @yozachar in openembedded#329
    chore: update dev dependencies by @yozachar in openembedded#330
    build(deps): bump gitpython from 3.1.37 to 3.1.41 in /package by @dependabot in openembedded#331
    build(deps): bump jinja2 from 3.1.2 to 3.1.3 in /package by @dependabot in openembedded#332
    build(deps): bump urllib3 from 2.0.6 to 2.0.7 in /package by @dependabot in openembedded#319

Signed-off-by: alperak <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
halstead pushed a commit that referenced this issue Mar 31, 2024
@alperak @kraj
python3-validators: upgrade 0.22.0 > 0.24.0 and enable ptest
e6673cf 
License-Update: Rename LICENSE to LICENSE.txt and update copyright year

* Ptest and library example tested on qemux86-64 and qemuarm64

* Add ptest into PTESTS_FAST_META_PYTHON

qemux86-64:

Testsuite summary
TOTAL: 599
PASS: 599
SKIP: 0
XFAIL: 0
FAIL: 0
XPASS: 0
ERROR: 0
DURATION: 19
END: /usr/lib/python3-validators/ptest
2024-03-30T05:07
STOP: ptest-runner
TOTAL: 1 FAIL: 0

qemuarm64:

Testsuite summary
TOTAL: 599
PASS: 599
SKIP: 0
XFAIL: 0
FAIL: 0
XPASS: 0
ERROR: 0
DURATION: 18
END: /usr/lib/python3-validators/ptest
2024-03-30T05:10
STOP: ptest-runner
TOTAL: 1 FAIL: 0

Changelog:

0.24.0
===========
Features

    feat: conditionally raises ValidationError; bump version by @yozachar in #343

Maintenance

    patch: domain & url modules by @yozachar in #339
    fix: domain name not confirming to rfc_2782 by @yozachar in #341
    maint: update dev dependencies; adds favicon to docs by @yozachar in #342

0.23.2
===========
Maintenance

    maint: rectifies changelog by @yozachar in #336
    fix: packaging as well as rST & md document generation by @yozachar in #337

0.23.1
===========
Maintenance

    maint: fix between & length validators by @yozachar in #334
    fix: manual nav reference for mkdocs; bumps version by @yozachar in #335

0.23.0
===========
Features

    feat: add french i18n validation by @imperosol in #308

Maintenance

    fix: Valid URLs failing validation - query and fragment parts by @danherbriley in #297
    fix: bug in between module by @yozachar in #301
    chore: update dependencies, improve packaging by @yozachar in #304
    Fix fragment check by @darkdragon-001 in #305
    build(deps): bump urllib3 from 2.0.6 to 2.0.7 in /package by @dependabot in #310
    fix: allow pct-encoded entities in fragments by @conitrade-as in #317
    chore: update dev dependencies by @yozachar in #318
    build(deps): bump gitpython from 3.1.37 to 3.1.41 in /package by @dependabot in #321
    build(deps): bump jinja2 from 3.1.2 to 3.1.3 in /package by @dependabot in #322
    chore: monthly updates for Jan'24 by @yozachar in #324
    maint: adds versiond docs; update copyright year by @yozachar in #329
    chore: update dev dependencies by @yozachar in #330
    build(deps): bump gitpython from 3.1.37 to 3.1.41 in /package by @dependabot in #331
    build(deps): bump jinja2 from 3.1.2 to 3.1.3 in /package by @dependabot in #332
    build(deps): bump urllib3 from 2.0.6 to 2.0.7 in /package by @dependabot in #319

Signed-off-by: alperak <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Aug 19, 2024
@quaresmajose @kraj
composefs: upgrade 1.0.4 -> 1.0.5
0d28cf5 
- switch to meson build system

Changelog [1]:

$ git --no-pager log --oneline v1.0.4..v1.0.5
098d985 (tag: v1.0.5) Merge pull request openembedded#312 from cgwalters/release
0fae8a8 Release 1.0.5
9bdc377 Merge pull request openembedded#310 from cgwalters/symlink-limit
b06ea9b Verify payload length early on
3beb011 Merge pull request openembedded#308 from cgwalters/minor-rust-lints
a695ad4 Merge pull request openembedded#309 from giuseppe/libcomposefs-drop-ro
0a0d50f libcomposefs: drop using "ro" option
b35dc83 dumpfile: Fix clippy lint
a919ab8 rust: Drop unused cfg
3a87868 Merge pull request openembedded#307 from giuseppe/not-override-errno
bb94687 Merge pull request openembedded#306 from giuseppe/mkcomposefs-report-close-errors
76b4da5 mkcompose: report close() errors
502bc27 libcomposefs: do not override errno
2d2c09d Merge pull request openembedded#304 from jluebbe/meson
0260856 meson.build: Allow use as a meson subproject
5f512ac Merge pull request openembedded#303 from cgwalters/mount-source
91a3047 mount: Consistently use `composefs` as source
45e6179 Merge pull request openembedded#302 from cgwalters/info-term
f64022f info: Add missing terminator for `longopts`
d81cb87 Merge pull request openembedded#300 from jluebbe/meson
04c8a8e ci: Add GitHub Actions workflow which uses meson
64709f6 ci: Install meson package
b6f2b75 Add meson as an alternative to autotools
942cd73 Merge pull request openembedded#299 from giuseppe/composefs-info-set-locale
5072fd7 composefs-info: specify locale to isgraph/isprint
4f19d00 Merge pull request openembedded#298 from cgwalters/rust-misc
43a1d28 rust: Add README.md files, link to crates.io
a8bc747 rust: Clarify that "composefs-core" is really "composefs"
483356c Merge pull request openembedded#297 from cgwalters/rust-include
02213c0 composefs-core: Drop incorrect `include`
dce84ff Merge pull request openembedded#295 from cgwalters/rust-descr
3d2af03 Merge pull request openembedded#296 from Kiskae/upstream-tmpdir-fix
6928fcf test: remove hardcoded /tmp
e21be07 rust: Add mandatory `description` field, other misc changes
28dc491 Merge pull request openembedded#292 from cgwalters/rust-ci-dual
59c5a27 Merge pull request openembedded#293 from cgwalters/rust-minor
689ccc6 rust: Some doc and misc tweaks
426b65d ci: Test Rust code against current C code
32b3f6e Merge pull request openembedded#291 from Kiskae/patch-1
f7465b3 use correct `composefs-info` binary

[1] containers/composefs@v1.0.4...v1.0.5

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
kraj pushed a commit to YoeDistro/meta-openembedded that referenced this issue Aug 19, 2024
@quaresmajose @kraj
composefs: upgrade 1.0.4 -> 1.0.5
bb62f52 
- switch to meson build system

Changelog [1]:

$ git --no-pager log --oneline v1.0.4..v1.0.5
098d985 (tag: v1.0.5) Merge pull request openembedded#312 from cgwalters/release
0fae8a8 Release 1.0.5
9bdc377 Merge pull request openembedded#310 from cgwalters/symlink-limit
b06ea9b Verify payload length early on
3beb011 Merge pull request openembedded#308 from cgwalters/minor-rust-lints
a695ad4 Merge pull request openembedded#309 from giuseppe/libcomposefs-drop-ro
0a0d50f libcomposefs: drop using "ro" option
b35dc83 dumpfile: Fix clippy lint
a919ab8 rust: Drop unused cfg
3a87868 Merge pull request openembedded#307 from giuseppe/not-override-errno
bb94687 Merge pull request openembedded#306 from giuseppe/mkcomposefs-report-close-errors
76b4da5 mkcompose: report close() errors
502bc27 libcomposefs: do not override errno
2d2c09d Merge pull request openembedded#304 from jluebbe/meson
0260856 meson.build: Allow use as a meson subproject
5f512ac Merge pull request openembedded#303 from cgwalters/mount-source
91a3047 mount: Consistently use `composefs` as source
45e6179 Merge pull request openembedded#302 from cgwalters/info-term
f64022f info: Add missing terminator for `longopts`
d81cb87 Merge pull request openembedded#300 from jluebbe/meson
04c8a8e ci: Add GitHub Actions workflow which uses meson
64709f6 ci: Install meson package
b6f2b75 Add meson as an alternative to autotools
942cd73 Merge pull request openembedded#299 from giuseppe/composefs-info-set-locale
5072fd7 composefs-info: specify locale to isgraph/isprint
4f19d00 Merge pull request openembedded#298 from cgwalters/rust-misc
43a1d28 rust: Add README.md files, link to crates.io
a8bc747 rust: Clarify that "composefs-core" is really "composefs"
483356c Merge pull request openembedded#297 from cgwalters/rust-include
02213c0 composefs-core: Drop incorrect `include`
dce84ff Merge pull request openembedded#295 from cgwalters/rust-descr
3d2af03 Merge pull request openembedded#296 from Kiskae/upstream-tmpdir-fix
6928fcf test: remove hardcoded /tmp
e21be07 rust: Add mandatory `description` field, other misc changes
28dc491 Merge pull request openembedded#292 from cgwalters/rust-ci-dual
59c5a27 Merge pull request openembedded#293 from cgwalters/rust-minor
689ccc6 rust: Some doc and misc tweaks
426b65d ci: Test Rust code against current C code
32b3f6e Merge pull request openembedded#291 from Kiskae/patch-1
f7465b3 use correct `composefs-info` binary

[1] containers/composefs@v1.0.4...v1.0.5

Signed-off-by: Jose Quaresma <[email protected]>
Signed-off-by: Khem Raj <[email protected]>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees
No one assigned
Labels
None yet
Projects
None yet
Milestone
No milestone
Development

No branches or pull requests
2 participants
@kraj @hellow554