Skip to content

Commit 2412ae3

Browse files
committed
[GR-33942] JDK17 fixes
PullRequest: graalpython/1972
2 parents c6fc7c0 + 8eaff63 commit 2412ae3

File tree

8 files changed

+310
-79
lines changed

8 files changed

+310
-79
lines changed

graalpython/com.oracle.graal.python.shell/src/com/oracle/graal/python/shell/GraalPythonMain.java

Lines changed: 2 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -388,13 +388,13 @@ private static String getLauncherExecName() {
388388
* <dt><b>Program name is an absolute path</b></dt>
389389
* <dd>Just return {@code program}.</dd>
390390
* <dt><b>Program name is a relative path</b></dt>
391-
* <dd>it will resolve it to an absolute path. E.g. {@code "./python3"} will become {@code
391+
* <dd>it will resolve it to an absolute path. E.g. {@code "./python3"} will become {@code
392392
* "<current_working_dir>/python3"}/dd>
393393
* <dt><b>Program name is neither an absolute nor a relative path</b></dt>
394394
* <dd>It will resolve the program name wrt. to the {@code PATH} env variable. Since it may be
395395
* that the {@code PATH} variable is not available, this method will return {@code null}</dd>
396396
* </dl>
397-
*
397+
*
398398
* @param program The program name as passed in the process' argument vector (position 0).
399399
* @return The absolute path to the program or {@code null}.
400400
*/
@@ -1027,11 +1027,9 @@ static List<String> getCmdline(List<String> args, List<String> subProcessDefs) {
10271027
switch (System.getProperty("java.vm.name")) {
10281028
case "Java HotSpot(TM) 64-Bit Server VM":
10291029
cmd.add("-server");
1030-
cmd.add("-d64");
10311030
break;
10321031
case "Java HotSpot(TM) 64-Bit Client VM":
10331032
cmd.add("-client");
1034-
cmd.add("-d64");
10351033
break;
10361034
default:
10371035
break;

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/ssl/ASN1Helper.java

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -68,8 +68,8 @@ public class ASN1Helper {
6868
static final String ASN1_EMAIL = "EMAIL";
6969

7070
static final String OID_CRL_DISTRIBUTION_POINTS = "2.5.29.31";
71-
static final String OID_OCSP = "1.3.6.1.5.5.7.48.1";
72-
static final String OID_CA_ISSUERS = "1.3.6.1.5.5.7.48.2";
71+
static final byte[] OID_OCSP = new byte[]{/* 1, 3, 6, */ 1, 5, 5, 7, 48, 1};
72+
static final byte[] OID_CA_ISSUERS = new byte[]{/* 1, 3, 6, */ 1, 5, 5, 7, 48, 2};
7373
static final String OID_AUTHORITY_INFO_ACCESS = "1.3.6.1.5.5.7.1.1";
7474

7575
private static final Map<String, String> javaToName = new HashMap<>();

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/ssl/CertUtils.java

Lines changed: 237 additions & 68 deletions
Large diffs are not rendered by default.

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/ssl/SSLContextBuiltins.java

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -982,7 +982,7 @@ Object getCerts(PSSLContext self, @SuppressWarnings("unused") boolean binary_for
982982
}
983983
}
984984
return factory().createList(result.toArray(new Object[result.size()]));
985-
} catch (KeyStoreException | NoSuchAlgorithmException | IOException | CertificateParsingException ex) {
985+
} catch (KeyStoreException | NoSuchAlgorithmException | CertificateParsingException ex) {
986986
throw PRaiseSSLErrorNode.raiseUncached(this, SSLErrorCode.ERROR_SSL, ex);
987987
}
988988
}

graalpython/com.oracle.graal.python/src/com/oracle/graal/python/builtins/objects/ssl/SSLSocketBuiltins.java

Lines changed: 0 additions & 4 deletions
Original file line numberDiff line numberDiff line change
@@ -41,10 +41,8 @@
4141
package com.oracle.graal.python.builtins.objects.ssl;
4242

4343
import static com.oracle.graal.python.builtins.PythonBuiltinClassType.NotImplementedError;
44-
import static com.oracle.graal.python.builtins.PythonBuiltinClassType.SSLError;
4544
import static com.oracle.graal.python.builtins.PythonBuiltinClassType.ValueError;
4645

47-
import java.io.IOException;
4846
import java.nio.ByteBuffer;
4947
import java.security.cert.Certificate;
5048
import java.security.cert.CertificateEncodingException;
@@ -320,8 +318,6 @@ PDict getPeerCertDict(PSSLSocket self, @SuppressWarnings("unused") boolean der)
320318
return CertUtils.decodeCertificate(this, (X509Certificate) certificate);
321319
} catch (CertificateParsingException e) {
322320
return factory().createDict();
323-
} catch (IOException ex) {
324-
throw raise(SSLError, ex);
325321
}
326322
}
327323
return factory().createDict();
Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
DISABLE_INSTALLABLES=true
2+
DYNAMIC_IMPORTS=/substratevm
3+
COMPONENTS=LibGraal,Graal.Python
4+
FORCE_BASH_LAUNCHERS=true
5+
SKIP_LIBRARIES=
6+
NO_LICENSES=true

mx.graalpython/graalpython-launcher

Lines changed: 6 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,6 @@
1+
DISABLE_INSTALLABLES=true
2+
DYNAMIC_IMPORTS=/substratevm
3+
COMPONENTS=LibGraal,Graal.Python
4+
FORCE_BASH_LAUNCHERS=graalvm-native-clang,graalvm-native-clang++,graalvm-native-ld,graalvm-native-binutil
5+
SKIP_LIBRARIES=
6+
NO_LICENSES=true

scripts/venv_deploy.sh

Lines changed: 56 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -0,0 +1,56 @@
1+
# Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
2+
# DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
3+
#
4+
# The Universal Permissive License (UPL), Version 1.0
5+
#
6+
# Subject to the condition set forth below, permission is hereby granted to any
7+
# person obtaining a copy of this software, associated documentation and/or
8+
# data (collectively the "Software"), free of charge and under any and all
9+
# copyright rights in the Software, and any and all patent rights owned or
10+
# freely licensable by each licensor hereunder covering either (i) the
11+
# unmodified Software as contributed to or provided by such licensor, or (ii)
12+
# the Larger Works (as defined below), to deal in both
13+
#
14+
# (a) the Software, and
15+
#
16+
# (b) any piece of software and/or hardware listed in the lrgrwrks.txt file if
17+
# one is included with the Software each a "Larger Work" to which the Software
18+
# is contributed by such licensors),
19+
#
20+
# without restriction, including without limitation the rights to copy, create
21+
# derivative works of, display, perform, and distribute the Software and make,
22+
# use, sell, offer for sale, import, export, have made, and have sold the
23+
# Software and the Larger Work(s), and to sublicense the foregoing rights on
24+
# either these or other terms.
25+
#
26+
# This license is subject to the following condition:
27+
#
28+
# The above copyright notice and either this complete permission notice or at a
29+
# minimum a reference to the UPL must be included in all copies or substantial
30+
# portions of the Software.
31+
#
32+
# THE SOFTWARE IS PROVIDED "AS IS", WITHOUT WARRANTY OF ANY KIND, EXPRESS OR
33+
# IMPLIED, INCLUDING BUT NOT LIMITED TO THE WARRANTIES OF MERCHANTABILITY,
34+
# FITNESS FOR A PARTICULAR PURPOSE AND NONINFRINGEMENT. IN NO EVENT SHALL THE
35+
# AUTHORS OR COPYRIGHT HOLDERS BE LIABLE FOR ANY CLAIM, DAMAGES OR OTHER
36+
# LIABILITY, WHETHER IN AN ACTION OF CONTRACT, TORT OR OTHERWISE, ARISING FROM,
37+
# OUT OF OR IN CONNECTION WITH THE SOFTWARE OR THE USE OR OTHER DEALINGS IN THE
38+
# SOFTWARE.
39+
40+
#!/bin/sh
41+
set -ex
42+
43+
venv="$1"
44+
45+
if [ ! -e "$venv" ]; then
46+
echo "Usage: $0 PATH-TO-VENV"
47+
exit
48+
fi
49+
50+
rm -rf "$venv"/bin
51+
rm -rf "$venv"/lib/python*/site-packages/pip*
52+
rm -rf "$venv"/lib/python*/site-packages/setuptools*
53+
rm -rf "$venv"/lib/python*/site-packages/pkg_resources*
54+
rm -rf "$venv"/lib/python*/site-packages/easy_install.py
55+
find "$venv" -name "__pycache__" -exec rm -rf "{}" ";"
56+
echo "include-system-site-packages = false\n" > "$venv"/pyvenv.cfg

0 commit comments

Comments
 (0)