Skip to content

Commit

Permalink
Merge pull request #670 from gurunrao/jstl-fix
Browse files Browse the repository at this point in the history
jstl fix with COMPAT option.
  • Loading branch information
scottmarlow authored Mar 19, 2021
2 parents 0c49540 + b0d6be5 commit 8a1890f
Show file tree
Hide file tree
Showing 2 changed files with 11 additions and 2 deletions.
5 changes: 5 additions & 0 deletions docker/jstltck.sh
Original file line number Diff line number Diff line change
Expand Up @@ -97,6 +97,11 @@ cd $TCK_HOME/$GF_TOPLEVEL_DIR/glassfish/bin
./asadmin create-jvm-options -Djavax.xml.accessExternalStylesheet=all
./asadmin create-jvm-options -Djavax.xml.accessExternalSchema=all
./asadmin create-jvm-options -Djavax.xml.accessExternalDTD=file,http
#https://github.com/eclipse-ee4j/jakartaee-tck/issues/631
if [[ "$JDK" == "JDK11" || "$JDK" == "jdk11" ]]; then
./asadmin create-jvm-options -Djava.locale.providers=COMPAT
fi

./asadmin stop-domain
./asadmin start-domain

Expand Down
8 changes: 6 additions & 2 deletions docker/run_jakartaeetck.sh
Original file line number Diff line number Diff line change
@@ -1,8 +1,8 @@
#!/bin/bash -x

#
# Copyright (c) 2018, 2020 Oracle and/or its affiliates. All rights reserved.
# Copyright (c) 2019, 2020 Payara Foundation and/or its affiliates. All rights reserved.
# Copyright (c) 2018, 2021 Oracle and/or its affiliates. All rights reserved.
# Copyright (c) 2019, 2021 Payara Foundation and/or its affiliates. All rights reserved.
#
# This program and the accompanying materials are made available under the
# terms of the Eclipse Public License v. 2.0, which is available at
Expand Down Expand Up @@ -273,6 +273,10 @@ ${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --password
${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --passwordfile ${ADMIN_PASSWORD_FILE} start-domain
${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --passwordfile ${ADMIN_PASSWORD_FILE} version
${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --passwordfile ${ADMIN_PASSWORD_FILE} create-jvm-options -Djava.security.manager
#https://github.com/eclipse-ee4j/jakartaee-tck/issues/631
if [[ ("$JDK" == "JDK11" || "$JDK" == "jdk11") && $test_suite == jstl ]]; then
${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --passwordfile ${ADMIN_PASSWORD_FILE} create-jvm-options -Djava.locale.providers=COMPAT
fi
${CTS_HOME}/vi/$GF_VI_TOPLEVEL_DIR/glassfish/bin/asadmin --user admin --passwordfile ${ADMIN_PASSWORD_FILE} stop-domain

sleep 5
Expand Down

0 comments on commit 8a1890f

Please # to comment.