termux-packages icon indicating copy to clipboard operation
termux-packages copied to clipboard

upgpkg(main/nodejs): v19.1.0

Open thunder-coding opened this issue 3 years ago • 6 comments

Will run the test suite locally once CI finishes. Also there's update for libicu, will make sure that there's no regression.

  • [X] Update to v19
  • [X] Make sure no regressions are there
  • [ ] ~~Use Ninja with GYP (in progress)~~
  • [ ] ~~Run tests again to ensure nothing broke~~ Migration to Ninja may be done later

Target merge date: December 4, 2022 (along with nodejs-lts update)

thunder-coding avatar Nov 23 '22 13:11 thunder-coding

How about adding a command named termux-setup-npm, which will apply the patch for cacache? If someone updates npm manually, he/she could run that command and apply the patch. We could remove this custom command until upstream fix this issue.

licy183 avatar Nov 25 '22 10:11 licy183

How about adding a command named termux-setup-npm, which will apply the patch for cacache? If someone updates npm manually, he/she could run that command and apply the patch. We could remove this custom command until upstream fix this issue.

The problem with this approach is that this patch won't gauranteedly work on all versions of npm. Also users can install specific versions themselves. Making it better for users to do the patch work manually instead of using a script do the same (which can also be disastrous in certain cases)

thunder-coding avatar Nov 25 '22 12:11 thunder-coding

The latest version of nodejs-lts is 18.12.1, please update it.

2096779623 avatar Nov 25 '22 12:11 2096779623

The latest version of nodejs-lts is 18.12.1, please update it.

I'll be going as usual, first current version then LTS, also users will need some time to migrate their apps and stuff they use to work with v18

thunder-coding avatar Nov 25 '22 12:11 thunder-coding

Any reason why we dont decouple and provide npm separately?

I do agree it is still better to fix it upstream.

truboxl avatar Nov 25 '22 13:11 truboxl

Any reason why we dont decouple and provide npm separately?

Sure we can do this, :+1: from my side.

Btw, I am planning to switch from make to ninja for building nodejs. Should also help reduce build time by a small extent. https://github.com/minrk/node/commit/5e533cb943eef68b270f55f0cba02c0ec67c699d https://github.com/nodejs/node/blob/main/doc/contributing/building-node-with-ninja.md

thunder-coding avatar Nov 25 '22 14:11 thunder-coding

This PR is now targetting v19.4.0. Will be pulled quickly after testing along with updating nodejs-lts to v18.13.0

thunder-coding avatar Jan 07 '23 11:01 thunder-coding