From 04f24fe9adcb19c356d43921638a3d1623dcbae7 Mon Sep 17 00:00:00 2001
From: Andy Herrick <herrick@openjdk.org>
Date: Thu, 1 Apr 2021 20:39:38 +0000
Subject: [PATCH] 8264403: [macos]: App names containing '.' characters results
 in an error message when launching

Reviewed-by: asemenyuk
---
 .../share/native/applauncher/AppLauncher.cpp  |  5 +-
 .../share/native/common/FileUtils.h           |  3 +
 .../unix/native/common/UnixFileUtils.cpp      |  5 ++
 .../windows/native/common/WinFileUtils.cpp    |  9 +++
 .../jdk/jpackage/tests/DotInNameTest.java     | 61 +++++++++++++++++++
 5 files changed, 80 insertions(+), 3 deletions(-)
 create mode 100644 test/jdk/tools/jpackage/share/jdk/jpackage/tests/DotInNameTest.java

diff --git a/src/jdk.jpackage/share/native/applauncher/AppLauncher.cpp b/src/jdk.jpackage/share/native/applauncher/AppLauncher.cpp
index a0143e1da71..0c70ccdc7f7 100644
--- a/src/jdk.jpackage/share/native/applauncher/AppLauncher.cpp
+++ b/src/jdk.jpackage/share/native/applauncher/AppLauncher.cpp
@@ -93,9 +93,8 @@ tstring findJvmLib(const CfgFile& cfgFile, const tstring& defaultRuntimePath,
 
 Jvm* AppLauncher::createJvmLauncher() const {
     const tstring cfgFilePath = FileUtils::mkpath()
-        << appDirPath
-        << FileUtils::basename(FileUtils::replaceSuffix(
-                launcherPath, _T(".cfg")));
+        << appDirPath << FileUtils::stripExeSuffix(
+            FileUtils::basename(launcherPath)) + _T(".cfg");
 
     LOG_TRACE(tstrings::any() << "Launcher config file path: \""
             << cfgFilePath << "\"");
diff --git a/src/jdk.jpackage/share/native/common/FileUtils.h b/src/jdk.jpackage/share/native/common/FileUtils.h
index 6b20e7c787d..c3cea28ed46 100644
--- a/src/jdk.jpackage/share/native/common/FileUtils.h
+++ b/src/jdk.jpackage/share/native/common/FileUtils.h
@@ -72,6 +72,9 @@ namespace FileUtils {
      */
     tstring replaceSuffix(const tstring& path, const tstring& suffix=tstring());
 
+    // remove the executable suffix if there is one
+    tstring stripExeSuffix(const tstring& path);
+
     /**
      * Returns absolute path of the given path.
      * If the given string is empty, returns absolute path to the current
diff --git a/src/jdk.jpackage/unix/native/common/UnixFileUtils.cpp b/src/jdk.jpackage/unix/native/common/UnixFileUtils.cpp
index d8584e87b49..b7f984c121c 100644
--- a/src/jdk.jpackage/unix/native/common/UnixFileUtils.cpp
+++ b/src/jdk.jpackage/unix/native/common/UnixFileUtils.cpp
@@ -62,4 +62,9 @@ tstring toAbsolutePath(const tstring& path) {
     return mkpath() << toAbsolutePath("") << path;
 }
 
+tstring stripExeSuffix(const tstring& path) {
+    // for unix - there is no suffix to remove
+    return path;
+}
+
 } //  namespace FileUtils
diff --git a/src/jdk.jpackage/windows/native/common/WinFileUtils.cpp b/src/jdk.jpackage/windows/native/common/WinFileUtils.cpp
index b769f40783a..3ad250db387 100644
--- a/src/jdk.jpackage/windows/native/common/WinFileUtils.cpp
+++ b/src/jdk.jpackage/windows/native/common/WinFileUtils.cpp
@@ -658,4 +658,13 @@ void FileWriter::finalize() {
     cleaner.cancel();
 }
 
+tstring stripExeSuffix(const tstring& path) {
+    // for windows - there is a ".exe" suffix to remove
+    const tstring::size_type pos = path.rfind(_T(".exe"));
+    if (pos == tstring::npos) {
+        return path;
+    }
+    return path.substr(0, pos);
+}
+
 } //  namespace FileUtils
diff --git a/test/jdk/tools/jpackage/share/jdk/jpackage/tests/DotInNameTest.java b/test/jdk/tools/jpackage/share/jdk/jpackage/tests/DotInNameTest.java
new file mode 100644
index 00000000000..87c9381bd2d
--- /dev/null
+++ b/test/jdk/tools/jpackage/share/jdk/jpackage/tests/DotInNameTest.java
@@ -0,0 +1,61 @@
+/*
+ * Copyright (c) 2021, Oracle and/or its affiliates. All rights reserved.
+ * DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER.
+ *
+ * This code is free software; you can redistribute it and/or modify it
+ * under the terms of the GNU General Public License version 2 only, as
+ * published by the Free Software Foundation.
+ *
+ * This code is distributed in the hope that it will be useful, but WITHOUT
+ * ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or
+ * FITNESS FOR A PARTICULAR PURPOSE.  See the GNU General Public License
+ * version 2 for more details (a copy is included in the LICENSE file that
+ * accompanied this code).
+ *
+ * You should have received a copy of the GNU General Public License version
+ * 2 along with this work; if not, write to the Free Software Foundation,
+ * Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA.
+ *
+ * Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA
+ * or visit www.oracle.com if you need additional information or have any
+ * questions.
+ */
+
+package jdk.jpackage.tests;
+
+import jdk.jpackage.test.JPackageCommand;
+import jdk.jpackage.test.HelloApp;
+import jdk.jpackage.test.TKit;
+import jdk.jpackage.test.Annotations.Parameters;
+import jdk.jpackage.test.Annotations.Test;
+
+
+/*
+ * @test
+ * @summary jpackage create image with --java-options test
+ * @library ../../../../helpers
+ * @build jdk.jpackage.test.*
+ * @modules jdk.jpackage/jdk.jpackage.internal
+ * @compile DotInNameTest.java
+ * @run main/othervm/timeout=360 -Xmx512m jdk.jpackage.test.Main
+ *  --jpt-run=jdk.jpackage.tests.DotInNameTest
+ *  --jpt-before-run=jdk.jpackage.test.JPackageCommand.useToolProviderByDefault
+ */
+
+public class DotInNameTest {
+
+    @Test
+    public void test() {
+        JPackageCommand cmd = JPackageCommand.helloAppImage("Hello");
+
+        cmd.setArgumentValue("--name",
+                cmd.getArgumentValue("--name") + ".with.dot");
+        // run the jpackage command
+        cmd.executeAndAssertImageCreated();
+
+        // run the launcher it generated
+        HelloApp.executeLauncherAndVerifyOutput(cmd);
+
+    }
+
+}