Open Bug 1712872 Opened 3 years ago Updated 1 years ago

[meta] Add ICU4X support to irregexp

Categories

(Core :: JavaScript Engine, task, P3)

task

Tracking

()

People

(Reporter: iain, Assigned: iain)

References

(Depends on 1 open bug, Blocks 1 open bug)

Details

(Keywords: meta)

Attachments

(5 obsolete files)

For her Outreachy internship, Ashwini will be adding support for using ICU4X instead of ICU4C in irregexp. This is a meta-bug for organizing that work.

Severity: -- → S4
Priority: -- → P1
Depends on: 1715824

Patch 1: Option for icu4x enabled

Depends on D119216

Licence check skipped.
We'll need to request license review for the license used in icu4x and
add it to the allowlist for mach vendor rust. For now, let's just skip
this check so we can run experiments.

This patch is not meant to be landed. Eventually C API and C++ API will be integrated.

Depends on D119406

tried adding #include "uniset.hpp" in regexp-parser.cc

Depends on D119452

Attachment #9232167 - Attachment description: Bug 1712872: Add ICU4X support to irregexp r=iain,mgaudet → Bug 1712872: Add ICU4X support to irregexp,patch 4 r=iain,mgaudet
Attachment #9230281 - Attachment description: Bug 1712872: Run mach vendor rust, add ICU4X uniset crates to irregexp r=iain,mgaudet → WIP: Bug 1712872: Run mach vendor rust, add ICU4X uniset crates to irregexp r=iain,mgaudet
Attachment #9229945 - Attachment is obsolete: true
Attachment #9229947 - Attachment is obsolete: true
Attachment #9230281 - Attachment is obsolete: true
Attachment #9230362 - Attachment is obsolete: true
Attachment #9232167 - Attachment is obsolete: true

The bug assignee didn't login in Bugzilla in the last 7 months.
:sdetar, could you have a look please?
For more information, please visit auto_nag documentation.

Assignee: ashwiniwankhede.aw → nobody
Flags: needinfo?(sdetar)

Iain, what should we do with this bug?

Flags: needinfo?(sdetar) → needinfo?(iireland)

Assigned this to myself.

Assignee: nobody → iireland
Flags: needinfo?(iireland)
Priority: P1 → P3
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: