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.

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

WIP: possible node-gyp-native cleanup #465

Draft
wants to merge 3 commits into
base: master
Choose a base branch
from

Conversation

shr-project
Copy link
Contributor

No description provided.

@shr-project shr-project requested review from Herrie82 and Tofee October 27, 2021 13:56
@shr-project
Copy link
Contributor Author

Doesn't work as-is, these were just some experiments I did, before finding smaller work around in #464

hyunjae92-shin pushed a commit to webosose/meta-webosose that referenced this pull request Jun 29, 2022
:Release Notes:
Fixes:
ERROR: node-gyp-packages-native-0.12.2-r0 do_unpack: QA Issue: node-gyp-packages-native: SRC_URI uses PN not BPN [src-uri-bad]

:Detailed Notes:
For other node-gyp-native fixes see:
webOS-ports/meta-webos-ports@812672d
webOS-ports/meta-webos-ports@29f569b
webOS-ports/meta-webos-ports@1c72a66
webOS-ports/meta-webos-ports@7a21203
webOS-ports/meta-webos-ports@3af337e
webOS-ports/meta-webos-ports@0941218
and
webOS-ports/meta-webos-ports#465

:Testing Performed:
Only build tested.

:QA Notes:
No change to image.

:Issues Addressed:
[WRN-9458] NPM registry should be accessed over https now
[WRN-9820] Create GPVB with Yocto 3.5 Kirkstone

Change-Id: If4c819d628fa9cb382ce682f160c1820508384d5
@shr-project shr-project force-pushed the jansa/node-gyp-node-16 branch from 9d9ee0e to 3ed248f Compare July 6, 2022 18:52
@Herrie82
Copy link
Contributor

@shr-project Do we still need this or can we close this one?

@shr-project
Copy link
Contributor Author

@shr-project Do we still need this or can we close this one?

still needed to resolve eventually

@shr-project shr-project force-pushed the master branch 2 times, most recently from c1bcdb8 to ff421f5 Compare November 10, 2023 00:10
@Herrie82
Copy link
Contributor

@shr-project node-sqlite we're not using anymore in general I guess (was mainly used for the https-everywhere which we used in our old QtWebEngine based browser), so that should make the node-gyp stuff a bit easier?

@shr-project shr-project force-pushed the jansa/node-gyp-node-16 branch from 3ed248f to 0b8d1bc Compare November 15, 2023 11:46
@shr-project
Copy link
Contributor Author

I don't really remember what were remaining issues with this but I think the webos-nodejs-webos-* didn't build with this yet.

I've rebased the changes just to see what's left.

I don't remember how this relates to node-sqlite.

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
…e node-gyp recipes

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
…node-gyp-native

Signed-off-by: Martin Jansa <Martin.Jansa@gmail.com>
@shr-project shr-project force-pushed the jansa/node-gyp-node-16 branch from 0b8d1bc to 6591b1e Compare January 20, 2024 13:16
@shr-project shr-project force-pushed the master branch 2 times, most recently from 3dabd23 to 2936d12 Compare May 3, 2024 11:23
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants