Gentoo Websites Logo
Go to: Gentoo Home Documentation Forums Lists Bugs Planet Store Wiki Get Gentoo!

Bug 724486

Summary: net-libs/nodejs should set USE=+icu
Product: Gentoo Linux Reporter: Adrian <adrian>
Component: Current packagesAssignee: Jeroen Roovers (RETIRED) <jer>
Status: RESOLVED FIXED    
Severity: normal CC: adrian, jstein
Priority: Normal    
Version: unspecified   
Hardware: All   
OS: Linux   
URL: https://github.com/sindresorhus/camelcase/issues/63#issuecomment-632275988
Whiteboard:
Package list:
Runtime testing required: ---

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(-)