Skip to content

Commit c5d8ac3

Browse files
committed
[GR-46899] Switch Python to shaded ICU4J module.
PullRequest: graalpython/2846
2 parents f6b379d + 3a3def6 commit c5d8ac3

File tree

10 files changed

+19
-74
lines changed

10 files changed

+19
-74
lines changed

graalpython/com.oracle.graal.python.pegparser/src/com/oracle/graal/python/pegparser/sst/StringLiteralUtils.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -1,5 +1,5 @@
11
/*
2-
* Copyright (c) 2019, 2022, Oracle and/or its affiliates. All rights reserved.
2+
* Copyright (c) 2019, 2023, Oracle and/or its affiliates. All rights reserved.
33
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
44
*
55
* The Universal Permissive License (UPL), Version 1.0
@@ -48,7 +48,7 @@
4848
import java.util.Locale;
4949
import java.util.Map;
5050

51-
import com.ibm.icu.lang.UCharacter;
51+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacter;
5252
import com.oracle.graal.python.pegparser.ErrorCallback;
5353
import com.oracle.graal.python.pegparser.ErrorCallback.WarningType;
5454
import com.oracle.graal.python.pegparser.FExprParser;

graalpython/com.oracle.graal.python.pegparser/src/com/oracle/graal/python/pegparser/tokenizer/Tokenizer.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
/* Copyright (c) 2021, 2022, Oracle and/or its affiliates.
1+
/* Copyright (c) 2021, 2023, Oracle and/or its affiliates.
22
* Copyright (C) 1996-2021 Python Software Foundation
33
*
44
* Licensed under the PYTHON SOFTWARE FOUNDATION LICENSE VERSION 2
@@ -13,8 +13,8 @@
1313
import java.util.Arrays;
1414
import java.util.EnumSet;
1515

16-
import com.ibm.icu.lang.UCharacter;
17-
import com.ibm.icu.lang.UProperty;
16+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacter;
17+
import org.graalvm.shadowed.com.ibm.icu.lang.UProperty;
1818
import com.oracle.graal.python.pegparser.ErrorCallback;
1919
import com.oracle.graal.python.pegparser.ErrorCallback.WarningType;
2020

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/modules/UnicodeDataModuleBuiltins.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -46,8 +46,8 @@
4646
import java.text.Normalizer;
4747
import java.util.List;
4848

49-
import com.ibm.icu.lang.UCharacter;
50-
import com.ibm.icu.lang.UProperty;
49+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacter;
50+
import org.graalvm.shadowed.com.ibm.icu.lang.UProperty;
5151
import com.oracle.graal.python.annotations.ArgumentClinic;
5252
import com.oracle.graal.python.builtins.Builtin;
5353
import com.oracle.graal.python.builtins.CoreFunctions;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/cext/common/CExtContext.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -53,8 +53,8 @@
5353

5454
import java.io.IOException;
5555

56-
import com.ibm.icu.impl.Punycode;
57-
import com.ibm.icu.text.StringPrepParseException;
56+
import org.graalvm.shadowed.com.ibm.icu.impl.Punycode;
57+
import org.graalvm.shadowed.com.ibm.icu.text.StringPrepParseException;
5858
import com.oracle.graal.python.builtins.objects.cext.capi.CApiContext;
5959
import com.oracle.graal.python.builtins.objects.cext.common.CExtCommonNodes.CheckFunctionResultNode;
6060
import com.oracle.graal.python.builtins.objects.cext.common.LoadCExtException.ApiInitException;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/str/StringBuiltins.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -78,9 +78,9 @@
7878
import java.util.regex.Matcher;
7979
import java.util.regex.Pattern;
8080

81-
import com.ibm.icu.lang.UCharacter;
82-
import com.ibm.icu.lang.UProperty;
83-
import com.ibm.icu.text.CaseMap;
81+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacter;
82+
import org.graalvm.shadowed.com.ibm.icu.lang.UProperty;
83+
import org.graalvm.shadowed.com.ibm.icu.text.CaseMap;
8484
import com.oracle.graal.python.PythonLanguage;
8585
import com.oracle.graal.python.annotations.ArgumentClinic;
8686
import com.oracle.graal.python.annotations.ArgumentClinic.ClinicConversion;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/str/StringUtils.java

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -50,9 +50,9 @@
5050

5151
import org.graalvm.nativeimage.ImageInfo;
5252

53-
import com.ibm.icu.lang.UCharacter;
54-
import com.ibm.icu.lang.UCharacterCategory;
55-
import com.ibm.icu.lang.UProperty;
53+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacter;
54+
import org.graalvm.shadowed.com.ibm.icu.lang.UCharacterCategory;
55+
import org.graalvm.shadowed.com.ibm.icu.lang.UProperty;
5656
import com.oracle.graal.python.util.PythonUtils;
5757
import com.oracle.truffle.api.CompilerDirectives.TruffleBoundary;
5858
import com.oracle.truffle.api.dsl.Cached;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/resources/reflect-config.json

Lines changed: 0 additions & 52 deletions
Original file line numberDiff line numberDiff line change
@@ -1,56 +1,4 @@
11
[
2-
{
3-
"name":"com.ibm.icu.impl.ICUCurrencyDisplayInfoProvider",
4-
"methods":[{"name":"<init>","parameterTypes":[] }]
5-
},
6-
{
7-
"name":"com.ibm.icu.impl.ICUCurrencyMetaInfo",
8-
"methods":[{"name":"<init>","parameterTypes":[] }]
9-
},
10-
{
11-
"name":"com.ibm.icu.impl.ICULangDataTables",
12-
"methods":[{"name":"<init>","parameterTypes":[] }]
13-
},
14-
{
15-
"name":"com.ibm.icu.impl.ICURegionDataTables",
16-
"methods":[{"name":"<init>","parameterTypes":[] }]
17-
},
18-
{
19-
"name":"com.ibm.icu.impl.LocaleDisplayNamesImpl",
20-
"methods":[
21-
{
22-
"name":"getInstance",
23-
"parameterTypes":[
24-
"com.ibm.icu.util.ULocale",
25-
"com.ibm.icu.text.DisplayContext[]"
26-
]
27-
}
28-
]
29-
},
30-
{
31-
"name":"com.ibm.icu.impl.TimeZoneNamesFactoryImpl",
32-
"methods":[{"name":"<init>","parameterTypes":[] }]
33-
},
34-
{
35-
"name":"com.ibm.icu.text.NumberFormatServiceShim",
36-
"methods":[{"name":"<init>","parameterTypes":[] }]
37-
},
38-
{
39-
"name":"com.ibm.icu.text.CollatorServiceShim",
40-
"methods":[{"name":"<init>","parameterTypes":[] }]
41-
},
42-
{
43-
"name":"com.ibm.icu.text.BreakIteratorFactory",
44-
"methods":[{"name":"<init>","parameterTypes":[] }]
45-
},
46-
{
47-
"name":"com.ibm.icu.charset.CharsetUTF7",
48-
"methods":[{"name":"<init>","parameterTypes":["java.lang.String", "java.lang.String", "java.lang.String[]"]}]
49-
},
50-
{
51-
"name":"com.ibm.icu.charset.CharsetHZ",
52-
"methods":[{"name":"<init>","parameterTypes":["java.lang.String", "java.lang.String", "java.lang.String[]"]}]
53-
},
542
{
553
"name":"java.util.Locale",
564
"fields":[{"name":"US"}],
Lines changed: 0 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -1,6 +1,5 @@
11
{
22
"resources":[
3-
{"pattern":"com/ibm/icu/impl/data/icudt.*"},
43
{"pattern":"org/graalvm/shadowed/org/jline.*"}
54
]
65
}

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/util/CharsetMapping.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -53,7 +53,7 @@
5353
import java.util.concurrent.ConcurrentHashMap;
5454
import java.util.concurrent.ConcurrentMap;
5555

56-
import com.ibm.icu.charset.CharsetICU;
56+
import org.graalvm.shadowed.com.ibm.icu.charset.CharsetICU;
5757
import com.oracle.graal.python.charset.PythonRawUnicodeEscapeCharset;
5858
import com.oracle.graal.python.charset.PythonUnicodeEscapeCharset;
5959
import com.oracle.graal.python.util.CharsetMappingFactory.NormalizeEncodingNameNodeGen;

mx.graalpython/suite.py

Lines changed: 3 additions & 5 deletions
Original file line numberDiff line numberDiff line change
@@ -260,7 +260,7 @@
260260
"sourceDirs": ["src"],
261261
"javaCompliance": "17+",
262262
"dependencies": [
263-
"truffle:ICU4J",
263+
"truffle:TRUFFLE_ICU4J",
264264
],
265265
"buildDependencies": [
266266
"com.oracle.graal.python.pegparser.generator",
@@ -375,8 +375,7 @@
375375
"sdk:GRAAL_SDK",
376376
"sulong:SULONG_API",
377377
"XZ-1.8",
378-
"truffle:ICU4J",
379-
"truffle:ICU4J-CHARSET",
378+
"truffle:TRUFFLE_ICU4J",
380379
"regex:TREGEX",
381380
"BOUNCYCASTLE-PROVIDER",
382381
"BOUNCYCASTLE-PKIX",
@@ -879,6 +878,7 @@
879878
"sdk:GRAAL_SDK",
880879
"sulong:SULONG_API",
881880
"sulong:SULONG_NATIVE", # this is actually just a runtime dependency
881+
"truffle:TRUFFLE_ICU4J",
882882
],
883883
"requires": [
884884
"java.base",
@@ -892,8 +892,6 @@
892892
"BOUNCYCASTLE-PROVIDER",
893893
"BOUNCYCASTLE-PKIX",
894894
"XZ-1.8",
895-
"truffle:ICU4J",
896-
"truffle:ICU4J-CHARSET",
897895
],
898896
"javaProperties": {
899897
"python.jni.library": "<lib:pythonjni>"

0 commit comments

Comments
 (0)