Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!
Bug 724486 - net-libs/nodejs should set USE=+icu
Summary: net-libs/nodejs should set USE=+icu
Status: RESOLVED FIXED
Alias: None
Product: Gentoo Linux
Classification: Unclassified
Component: Current packages (show other bugs)
Hardware: All Linux
: Normal normal
Assignee: Jeroen Roovers (RETIRED)
URL: https://github.com/sindresorhus/camel...
Whiteboard:
Keywords:
Depends on:
Blocks:
 
Reported: 2020-05-21 19:12 UTC by Adrian
Modified: 2020-06-04 12:56 UTC (History)
2 users (show)

See Also:
Package list:
Runtime testing required: ---


Attachments

Note You need to log in before you can comment on or make changes to this bug.
Description Adrian 2020-05-21 19:12:56 UTC
Not having icu support in nodejs is rarely (never?) what you want. Especially on a development system this now breaks the very popular testing framework jest since one of its dependencies uses regex features only available with icu:

https://github.com/sindresorhus/camelcase/issues/63#issuecomment-632275988

Reproducible: Always

Steps to Reproduce:
1. run `node`
2. paste `x = /[_.\- ]+([\p{Alpha}\p{N}_]|$)/gu;` in the REPL
Actual Results:  
syntax error

Expected Results:  
the regex is accepted
Comment 1 Larry the Git Cow gentoo-dev 2020-06-04 12:56:55 UTC
The bug has been closed via the following commit(s):

https://gitweb.gentoo.org/repo/gentoo.git/commit/?id=07c02f4d5360028d6afa48670dd460ad860fdf3b

commit 07c02f4d5360028d6afa48670dd460ad860fdf3b
Author:     Jeroen Roovers <jer@gentoo.org>
AuthorDate: 2020-06-04 12:52:08 +0000
Commit:     Jeroen Roovers <jer@gentoo.org>
CommitDate: 2020-06-04 12:56:51 +0000

    net-libs/nodejs: Set IUSE=+icu
    
    Package-Manager: Portage-2.3.100, Repoman-2.3.22
    Closes: https://bugs.gentoo.org/724486
    Signed-off-by: Jeroen Roovers <jer@gentoo.org>

 net-libs/nodejs/nodejs-14.4.0.ebuild   | 2 +-
 net-libs/nodejs/nodejs-99999999.ebuild | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)