author | herrick |
Fri, 04 Oct 2019 14:56:02 -0400 | |
branch | JDK-8200758-branch |
changeset 58464 | d82489644b15 |
parent 58415 | 73f8e557549a |
child 58581 | 657e25d9e4e4 |
permissions | -rw-r--r-- |
57038 | 1 |
/* |
57106
ea870b9ce89a
8216492: Update copyright of all new jpackage fils to 2019
kcr
parents:
57096
diff
changeset
|
2 |
* Copyright (c) 2014, 2019, Oracle and/or its affiliates. All rights reserved. |
57038 | 3 |
* DO NOT ALTER OR REMOVE COPYRIGHT NOTICES OR THIS FILE HEADER. |
4 |
* |
|
5 |
* This code is free software; you can redistribute it and/or modify it |
|
6 |
* under the terms of the GNU General Public License version 2 only, as |
|
7 |
* published by the Free Software Foundation. Oracle designates this |
|
8 |
* particular file as subject to the "Classpath" exception as provided |
|
9 |
* by Oracle in the LICENSE file that accompanied this code. |
|
10 |
* |
|
11 |
* This code is distributed in the hope that it will be useful, but WITHOUT |
|
12 |
* ANY WARRANTY; without even the implied warranty of MERCHANTABILITY or |
|
13 |
* FITNESS FOR A PARTICULAR PURPOSE. See the GNU General Public License |
|
14 |
* version 2 for more details (a copy is included in the LICENSE file that |
|
15 |
* accompanied this code). |
|
16 |
* |
|
17 |
* You should have received a copy of the GNU General Public License version |
|
18 |
* 2 along with this work; if not, write to the Free Software Foundation, |
|
19 |
* Inc., 51 Franklin St, Fifth Floor, Boston, MA 02110-1301 USA. |
|
20 |
* |
|
21 |
* Please contact Oracle, 500 Oracle Parkway, Redwood Shores, CA 94065 USA |
|
22 |
* or visit www.oracle.com if you need additional information or have any |
|
23 |
* questions. |
|
24 |
*/ |
|
25 |
||
57039 | 26 |
package jdk.jpackage.internal; |
57038 | 27 |
|
28 |
import java.io.File; |
|
29 |
import java.io.IOException; |
|
30 |
import java.io.PrintStream; |
|
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
31 |
import java.io.PrintWriter; |
57038 | 32 |
import java.io.Writer; |
33 |
import java.net.URLEncoder; |
|
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
34 |
import java.nio.file.Files; |
57331
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
35 |
import java.nio.file.Path; |
57038 | 36 |
import java.text.MessageFormat; |
37 |
import java.util.ArrayList; |
|
38 |
import java.util.HashMap; |
|
39 |
import java.util.List; |
|
40 |
import java.util.Map; |
|
41 |
import java.util.Optional; |
|
42 |
import java.util.ResourceBundle; |
|
43 |
||
57039 | 44 |
import static jdk.jpackage.internal.StandardBundlerParam.*; |
57067
9c17d779808e
8213425: Analyze output from Source code scanner and fix where needed.
herrick
parents:
57059
diff
changeset
|
45 |
import static jdk.jpackage.internal.MacBaseInstallerBundler.SIGNING_KEYCHAIN; |
9c17d779808e
8213425: Analyze output from Source code scanner and fix where needed.
herrick
parents:
57059
diff
changeset
|
46 |
import static jdk.jpackage.internal.MacBaseInstallerBundler.SIGNING_KEY_USER; |
58415 | 47 |
import static jdk.jpackage.internal.MacAppImageBuilder.MAC_CF_BUNDLE_IDENTIFIER; |
57038 | 48 |
|
49 |
public class MacPkgBundler extends MacBaseInstallerBundler { |
|
50 |
||
51 |
private static final ResourceBundle I18N = ResourceBundle.getBundle( |
|
57059 | 52 |
"jdk.jpackage.internal.resources.MacResources"); |
57038 | 53 |
|
57908
9a005146bc1c
8230191: Replace javacup logo with duke everywhere in jpackage
herrick
parents:
57554
diff
changeset
|
54 |
private static final String DEFAULT_BACKGROUND_IMAGE = "background_dmg.png"; |
57038 | 55 |
|
56 |
private static final String TEMPLATE_PREINSTALL_SCRIPT = |
|
57 |
"preinstall.template"; |
|
58 |
private static final String TEMPLATE_POSTINSTALL_SCRIPT = |
|
59 |
"postinstall.template"; |
|
60 |
||
61 |
private static final BundlerParamInfo<File> PACKAGES_ROOT = |
|
62 |
new StandardBundlerParam<>( |
|
63 |
"mac.pkg.packagesRoot", |
|
64 |
File.class, |
|
65 |
params -> { |
|
66 |
File packagesRoot = |
|
57256 | 67 |
new File(TEMP_ROOT.fetchFrom(params), "packages"); |
57038 | 68 |
packagesRoot.mkdirs(); |
69 |
return packagesRoot; |
|
70 |
}, |
|
71 |
(s, p) -> new File(s)); |
|
72 |
||
73 |
||
74 |
protected final BundlerParamInfo<File> SCRIPTS_DIR = |
|
75 |
new StandardBundlerParam<>( |
|
76 |
"mac.pkg.scriptsDir", |
|
77 |
File.class, |
|
78 |
params -> { |
|
79 |
File scriptsDir = |
|
80 |
new File(CONFIG_ROOT.fetchFrom(params), "scripts"); |
|
81 |
scriptsDir.mkdirs(); |
|
82 |
return scriptsDir; |
|
83 |
}, |
|
84 |
(s, p) -> new File(s)); |
|
85 |
||
86 |
public static final |
|
87 |
BundlerParamInfo<String> DEVELOPER_ID_INSTALLER_SIGNING_KEY = |
|
88 |
new StandardBundlerParam<>( |
|
89 |
"mac.signing-key-developer-id-installer", |
|
90 |
String.class, |
|
91 |
params -> { |
|
92 |
String result = MacBaseInstallerBundler.findKey( |
|
93 |
"Developer ID Installer: " |
|
94 |
+ SIGNING_KEY_USER.fetchFrom(params), |
|
95 |
SIGNING_KEYCHAIN.fetchFrom(params), |
|
96 |
VERBOSE.fetchFrom(params)); |
|
97 |
if (result != null) { |
|
98 |
MacCertificate certificate = new MacCertificate( |
|
99 |
result, VERBOSE.fetchFrom(params)); |
|
100 |
||
101 |
if (!certificate.isValid()) { |
|
102 |
Log.error(MessageFormat.format( |
|
103 |
I18N.getString("error.certificate.expired"), |
|
104 |
result)); |
|
105 |
} |
|
106 |
} |
|
107 |
||
108 |
return result; |
|
109 |
}, |
|
110 |
(s, p) -> s); |
|
111 |
||
112 |
public static final BundlerParamInfo<String> MAC_INSTALL_DIR = |
|
113 |
new StandardBundlerParam<>( |
|
114 |
"mac-install-dir", |
|
115 |
String.class, |
|
116 |
params -> { |
|
117 |
String dir = INSTALL_DIR.fetchFrom(params); |
|
118 |
return (dir != null) ? dir : "/Applications"; |
|
119 |
}, |
|
120 |
(s, p) -> s |
|
121 |
); |
|
122 |
||
123 |
public static final BundlerParamInfo<String> INSTALLER_SUFFIX = |
|
124 |
new StandardBundlerParam<> ( |
|
125 |
"mac.pkg.installerName.suffix", |
|
126 |
String.class, |
|
127 |
params -> "", |
|
128 |
(s, p) -> s); |
|
129 |
||
57140 | 130 |
public File bundle(Map<String, ? super Object> params, |
131 |
File outdir) throws PackagerException { |
|
57038 | 132 |
Log.verbose(MessageFormat.format(I18N.getString("message.building-pkg"), |
133 |
APP_NAME.fetchFrom(params))); |
|
57407 | 134 |
|
135 |
IOUtils.writableOutputDir(outdir.toPath()); |
|
57038 | 136 |
|
137 |
try { |
|
57554 | 138 |
File appImageDir = prepareAppBundle(params); |
57038 | 139 |
|
140 |
if (appImageDir != null && prepareConfigFiles(params)) { |
|
141 |
||
142 |
File configScript = getConfig_Script(params); |
|
143 |
if (configScript.exists()) { |
|
144 |
Log.verbose(MessageFormat.format(I18N.getString( |
|
145 |
"message.running-script"), |
|
146 |
configScript.getAbsolutePath())); |
|
57391 | 147 |
IOUtils.run("bash", configScript); |
57038 | 148 |
} |
149 |
||
150 |
return createPKG(params, outdir, appImageDir); |
|
151 |
} |
|
152 |
return null; |
|
153 |
} catch (IOException ex) { |
|
154 |
Log.verbose(ex); |
|
57140 | 155 |
throw new PackagerException(ex); |
57038 | 156 |
} |
157 |
} |
|
158 |
||
159 |
private File getPackages_AppPackage(Map<String, ? super Object> params) { |
|
160 |
return new File(PACKAGES_ROOT.fetchFrom(params), |
|
57119 | 161 |
APP_NAME.fetchFrom(params) + "-app.pkg"); |
57038 | 162 |
} |
163 |
||
164 |
private File getConfig_DistributionXMLFile( |
|
165 |
Map<String, ? super Object> params) { |
|
166 |
return new File(CONFIG_ROOT.fetchFrom(params), "distribution.dist"); |
|
167 |
} |
|
168 |
||
169 |
private File getConfig_BackgroundImage(Map<String, ? super Object> params) { |
|
170 |
return new File(CONFIG_ROOT.fetchFrom(params), |
|
171 |
APP_NAME.fetchFrom(params) + "-background.png"); |
|
172 |
} |
|
173 |
||
174 |
private File getScripts_PreinstallFile(Map<String, ? super Object> params) { |
|
175 |
return new File(SCRIPTS_DIR.fetchFrom(params), "preinstall"); |
|
176 |
} |
|
177 |
||
178 |
private File getScripts_PostinstallFile( |
|
179 |
Map<String, ? super Object> params) { |
|
180 |
return new File(SCRIPTS_DIR.fetchFrom(params), "postinstall"); |
|
181 |
} |
|
182 |
||
183 |
private String getAppIdentifier(Map<String, ? super Object> params) { |
|
184 |
return IDENTIFIER.fetchFrom(params); |
|
185 |
} |
|
186 |
||
187 |
private void preparePackageScripts(Map<String, ? super Object> params) |
|
188 |
throws IOException { |
|
189 |
Log.verbose(I18N.getString("message.preparing-scripts")); |
|
190 |
||
191 |
Map<String, String> data = new HashMap<>(); |
|
192 |
||
57290
47c2aca6c306
8215241: Permissions are not set correctly on sub-folders in /Applications
ssadetsky
parents:
57256
diff
changeset
|
193 |
data.put("INSTALL_LOCATION", MAC_INSTALL_DIR.fetchFrom(params)); |
57038 | 194 |
|
57390 | 195 |
try (Writer w = Files.newBufferedWriter( |
196 |
getScripts_PreinstallFile(params).toPath())) { |
|
197 |
String content = preprocessTextResource( |
|
198 |
getScripts_PreinstallFile(params).getName(), |
|
199 |
I18N.getString("resource.pkg-preinstall-script"), |
|
200 |
TEMPLATE_PREINSTALL_SCRIPT, |
|
201 |
data, |
|
202 |
VERBOSE.fetchFrom(params), |
|
203 |
RESOURCE_DIR.fetchFrom(params)); |
|
204 |
w.write(content); |
|
205 |
} |
|
57038 | 206 |
getScripts_PreinstallFile(params).setExecutable(true, false); |
207 |
||
57390 | 208 |
try (Writer w = Files.newBufferedWriter( |
209 |
getScripts_PostinstallFile(params).toPath())) { |
|
210 |
String content = preprocessTextResource( |
|
211 |
getScripts_PostinstallFile(params).getName(), |
|
212 |
I18N.getString("resource.pkg-postinstall-script"), |
|
213 |
TEMPLATE_POSTINSTALL_SCRIPT, |
|
214 |
data, |
|
215 |
VERBOSE.fetchFrom(params), |
|
216 |
RESOURCE_DIR.fetchFrom(params)); |
|
217 |
w.write(content); |
|
218 |
} |
|
57038 | 219 |
getScripts_PostinstallFile(params).setExecutable(true, false); |
220 |
} |
|
221 |
||
222 |
private void prepareDistributionXMLFile(Map<String, ? super Object> params) |
|
223 |
throws IOException { |
|
224 |
File f = getConfig_DistributionXMLFile(params); |
|
225 |
||
226 |
Log.verbose(MessageFormat.format(I18N.getString( |
|
227 |
"message.preparing-distribution-dist"), f.getAbsolutePath())); |
|
228 |
||
57390 | 229 |
try (PrintStream out = new PrintStream(f)) { |
57038 | 230 |
|
57390 | 231 |
out.println( |
57038 | 232 |
"<?xml version=\"1.0\" encoding=\"utf-8\" standalone=\"no\"?>"); |
57390 | 233 |
out.println("<installer-gui-script minSpecVersion=\"1\">"); |
57038 | 234 |
|
57390 | 235 |
out.println("<title>" + APP_NAME.fetchFrom(params) + "</title>"); |
236 |
out.println("<background" + " file=\"" |
|
237 |
+ getConfig_BackgroundImage(params).getName() |
|
238 |
+ "\"" |
|
239 |
+ " mime-type=\"image/png\"" |
|
240 |
+ " alignment=\"bottomleft\" " |
|
241 |
+ " scaling=\"none\"" |
|
242 |
+ "/>"); |
|
57038 | 243 |
|
57390 | 244 |
String licFileStr = LICENSE_FILE.fetchFrom(params); |
245 |
if (licFileStr != null) { |
|
246 |
File licFile = new File(licFileStr); |
|
247 |
out.println("<license" |
|
248 |
+ " file=\"" + licFile.getAbsolutePath() + "\"" |
|
249 |
+ " mime-type=\"text/rtf\"" |
|
250 |
+ "/>"); |
|
251 |
} |
|
57038 | 252 |
|
57390 | 253 |
/* |
254 |
* Note that the content of the distribution file |
|
255 |
* below is generated by productbuild --synthesize |
|
256 |
*/ |
|
57038 | 257 |
|
57390 | 258 |
String appId = getAppIdentifier(params); |
57038 | 259 |
|
57390 | 260 |
out.println("<pkg-ref id=\"" + appId + "\"/>"); |
261 |
out.println( |
|
262 |
"<options customize=\"never\" require-scripts=\"false\"/>"); |
|
263 |
out.println("<choices-outline>"); |
|
264 |
out.println(" <line choice=\"default\">"); |
|
265 |
out.println(" <line choice=\"" + appId + "\"/>"); |
|
266 |
out.println(" </line>"); |
|
267 |
out.println("</choices-outline>"); |
|
268 |
out.println("<choice id=\"default\"/>"); |
|
269 |
out.println("<choice id=\"" + appId + "\" visible=\"false\">"); |
|
270 |
out.println(" <pkg-ref id=\"" + appId + "\"/>"); |
|
271 |
out.println("</choice>"); |
|
272 |
out.println("<pkg-ref id=\"" + appId + "\" version=\"" |
|
273 |
+ VERSION.fetchFrom(params) + "\" onConclusion=\"none\">" |
|
274 |
+ URLEncoder.encode( |
|
275 |
getPackages_AppPackage(params).getName(), |
|
276 |
"UTF-8") + "</pkg-ref>"); |
|
57038 | 277 |
|
57390 | 278 |
out.println("</installer-gui-script>"); |
57038 | 279 |
|
57390 | 280 |
} |
57038 | 281 |
} |
282 |
||
283 |
private boolean prepareConfigFiles(Map<String, ? super Object> params) |
|
284 |
throws IOException { |
|
285 |
File imageTarget = getConfig_BackgroundImage(params); |
|
57091
06bc4bd64599
8215515: Add a command line option to override internal resources.
herrick
parents:
57080
diff
changeset
|
286 |
fetchResource(imageTarget.getName(), |
57038 | 287 |
I18N.getString("resource.pkg-background-image"), |
288 |
DEFAULT_BACKGROUND_IMAGE, |
|
289 |
imageTarget, |
|
290 |
VERBOSE.fetchFrom(params), |
|
57091
06bc4bd64599
8215515: Add a command line option to override internal resources.
herrick
parents:
57080
diff
changeset
|
291 |
RESOURCE_DIR.fetchFrom(params)); |
57038 | 292 |
|
293 |
prepareDistributionXMLFile(params); |
|
294 |
||
57091
06bc4bd64599
8215515: Add a command line option to override internal resources.
herrick
parents:
57080
diff
changeset
|
295 |
fetchResource(getConfig_Script(params).getName(), |
57038 | 296 |
I18N.getString("resource.post-install-script"), |
297 |
(String) null, |
|
298 |
getConfig_Script(params), |
|
299 |
VERBOSE.fetchFrom(params), |
|
57091
06bc4bd64599
8215515: Add a command line option to override internal resources.
herrick
parents:
57080
diff
changeset
|
300 |
RESOURCE_DIR.fetchFrom(params)); |
57038 | 301 |
|
302 |
return true; |
|
303 |
} |
|
304 |
||
305 |
// name of post-image script |
|
306 |
private File getConfig_Script(Map<String, ? super Object> params) { |
|
307 |
return new File(CONFIG_ROOT.fetchFrom(params), |
|
308 |
APP_NAME.fetchFrom(params) + "-post-image.sh"); |
|
309 |
} |
|
310 |
||
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
311 |
private void patchCPLFile(File cpl) throws IOException { |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
312 |
String cplData = Files.readString(cpl.toPath()); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
313 |
String[] lines = cplData.split("\n"); |
57390 | 314 |
try (PrintWriter out = new PrintWriter(Files.newBufferedWriter( |
315 |
cpl.toPath()))) { |
|
57529
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
316 |
int skip = 0; |
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
317 |
// Used to skip Java.runtime bundle, since |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
318 |
// pkgbuild with --root will find two bundles app and Java runtime. |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
319 |
// We cannot generate component proprty list when using |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
320 |
// --component argument. |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
321 |
for (int i = 0; i < lines.length; i++) { |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
322 |
if (lines[i].trim().equals("<key>BundleIsRelocatable</key>")) { |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
323 |
out.println(lines[i]); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
324 |
out.println("<false/>"); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
325 |
i++; |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
326 |
} else if (lines[i].trim().equals("<key>ChildBundles</key>")) { |
57529
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
327 |
++skip; |
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
328 |
} else if ((skip > 0) && lines[i].trim().equals("</array>")) { |
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
329 |
--skip; |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
330 |
} else { |
57529
e3b156ad362f
8224627: Creating installer with --runtime-image on OS X fails
herrick
parents:
57438
diff
changeset
|
331 |
if (skip == 0) { |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
332 |
out.println(lines[i]); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
333 |
} |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
334 |
} |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
335 |
} |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
336 |
} |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
337 |
} |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
338 |
|
57331
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
339 |
// pkgbuild includes all components from "--root" and subfolders, |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
340 |
// so if we have app image in folder which contains other images, then they |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
341 |
// will be included as well. It does have "--filter" option which use regex |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
342 |
// to exclude files/folder, but it will overwrite default one which excludes |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
343 |
// based on doc "any .svn or CVS directories, and any .DS_Store files". |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
344 |
// So easy aproach will be to copy user provided app-image into temp folder |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
345 |
// if root path contains other files. |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
346 |
private String getRoot(Map<String, ? super Object> params, |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
347 |
File appLocation) throws IOException { |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
348 |
String root = appLocation.getParent() == null ? |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
349 |
"." : appLocation.getParent(); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
350 |
File rootDir = new File(root); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
351 |
File[] list = rootDir.listFiles(); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
352 |
if (list != null) { // Should not happend |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
353 |
// We should only have app image and/or .DS_Store |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
354 |
if (list.length == 1) { |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
355 |
return root; |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
356 |
} else if (list.length == 2) { |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
357 |
// Check case with app image and .DS_Store |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
358 |
if (list[0].toString().toLowerCase().endsWith(".ds_store") || |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
359 |
list[1].toString().toLowerCase().endsWith(".ds_store")) { |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
360 |
return root; // Only app image and .DS_Store |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
361 |
} |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
362 |
} |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
363 |
} |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
364 |
|
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
365 |
// Copy to new root |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
366 |
Path newRoot = Files.createTempDirectory( |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
367 |
TEMP_ROOT.fetchFrom(params).toPath(), |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
368 |
"root-"); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
369 |
|
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
370 |
IOUtils.copyRecursive(appLocation.toPath(), |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
371 |
newRoot.resolve(appLocation.getName())); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
372 |
|
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
373 |
return newRoot.toString(); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
374 |
} |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
375 |
|
57038 | 376 |
private File createPKG(Map<String, ? super Object> params, |
377 |
File outdir, File appLocation) { |
|
378 |
// generic find attempt |
|
379 |
try { |
|
380 |
File appPKG = getPackages_AppPackage(params); |
|
381 |
||
57331
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
382 |
String root = getRoot(params, appLocation); |
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
383 |
|
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
384 |
// Generate default CPL file |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
385 |
File cpl = new File(CONFIG_ROOT.fetchFrom(params).getAbsolutePath() |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
386 |
+ File.separator + "cpl.plist"); |
57038 | 387 |
ProcessBuilder pb = new ProcessBuilder("pkgbuild", |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
388 |
"--root", |
57331
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
389 |
root, |
57038 | 390 |
"--install-location", |
391 |
MAC_INSTALL_DIR.fetchFrom(params), |
|
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
392 |
"--analyze", |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
393 |
cpl.getAbsolutePath()); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
394 |
|
57391 | 395 |
IOUtils.exec(pb); |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
396 |
|
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
397 |
patchCPLFile(cpl); |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
398 |
|
57290
47c2aca6c306
8215241: Permissions are not set correctly on sub-folders in /Applications
ssadetsky
parents:
57256
diff
changeset
|
399 |
preparePackageScripts(params); |
47c2aca6c306
8215241: Permissions are not set correctly on sub-folders in /Applications
ssadetsky
parents:
57256
diff
changeset
|
400 |
|
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
401 |
// build application package |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
402 |
pb = new ProcessBuilder("pkgbuild", |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
403 |
"--root", |
57331
221a589c52ee
8222676: create-installer with --app-image fails on mac
herrick
parents:
57290
diff
changeset
|
404 |
root, |
57217
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
405 |
"--install-location", |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
406 |
MAC_INSTALL_DIR.fetchFrom(params), |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
407 |
"--component-plist", |
b1291b2cedc9
8219144: Cannot find installed application on Mac
herrick
parents:
57150
diff
changeset
|
408 |
cpl.getAbsolutePath(), |
57290
47c2aca6c306
8215241: Permissions are not set correctly on sub-folders in /Applications
ssadetsky
parents:
57256
diff
changeset
|
409 |
"--scripts", |
47c2aca6c306
8215241: Permissions are not set correctly on sub-folders in /Applications
ssadetsky
parents:
57256
diff
changeset
|
410 |
SCRIPTS_DIR.fetchFrom(params).getAbsolutePath(), |
57038 | 411 |
appPKG.getAbsolutePath()); |
57391 | 412 |
IOUtils.exec(pb); |
57038 | 413 |
|
414 |
// build final package |
|
415 |
File finalPKG = new File(outdir, INSTALLER_NAME.fetchFrom(params) |
|
416 |
+ INSTALLER_SUFFIX.fetchFrom(params) |
|
417 |
+ ".pkg"); |
|
418 |
outdir.mkdirs(); |
|
419 |
||
420 |
List<String> commandLine = new ArrayList<>(); |
|
421 |
commandLine.add("productbuild"); |
|
422 |
||
423 |
commandLine.add("--resources"); |
|
424 |
commandLine.add(CONFIG_ROOT.fetchFrom(params).getAbsolutePath()); |
|
425 |
||
426 |
// maybe sign |
|
427 |
if (Optional.ofNullable(MacAppImageBuilder. |
|
428 |
SIGN_BUNDLE.fetchFrom(params)).orElse(Boolean.TRUE)) { |
|
429 |
if (Platform.getMajorVersion() > 10 || |
|
430 |
(Platform.getMajorVersion() == 10 && |
|
431 |
Platform.getMinorVersion() >= 12)) { |
|
432 |
// we need this for OS X 10.12+ |
|
433 |
Log.verbose(I18N.getString("message.signing.pkg")); |
|
434 |
} |
|
435 |
||
436 |
String signingIdentity = |
|
437 |
DEVELOPER_ID_INSTALLER_SIGNING_KEY.fetchFrom(params); |
|
438 |
if (signingIdentity != null) { |
|
439 |
commandLine.add("--sign"); |
|
440 |
commandLine.add(signingIdentity); |
|
441 |
} |
|
442 |
||
443 |
String keychainName = SIGNING_KEYCHAIN.fetchFrom(params); |
|
444 |
if (keychainName != null && !keychainName.isEmpty()) { |
|
445 |
commandLine.add("--keychain"); |
|
446 |
commandLine.add(keychainName); |
|
447 |
} |
|
448 |
} |
|
449 |
||
450 |
commandLine.add("--distribution"); |
|
451 |
commandLine.add( |
|
452 |
getConfig_DistributionXMLFile(params).getAbsolutePath()); |
|
453 |
commandLine.add("--package-path"); |
|
454 |
commandLine.add(PACKAGES_ROOT.fetchFrom(params).getAbsolutePath()); |
|
455 |
||
456 |
commandLine.add(finalPKG.getAbsolutePath()); |
|
457 |
||
458 |
pb = new ProcessBuilder(commandLine); |
|
57391 | 459 |
IOUtils.exec(pb); |
57038 | 460 |
|
461 |
return finalPKG; |
|
462 |
} catch (Exception ignored) { |
|
463 |
Log.verbose(ignored); |
|
464 |
return null; |
|
465 |
} |
|
466 |
} |
|
467 |
||
468 |
////////////////////////////////////////////////////////////////////////// |
|
469 |
// Implement Bundler |
|
470 |
////////////////////////////////////////////////////////////////////////// |
|
471 |
||
472 |
@Override |
|
473 |
public String getName() { |
|
57059 | 474 |
return I18N.getString("pkg.bundler.name"); |
57038 | 475 |
} |
476 |
||
477 |
@Override |
|
478 |
public String getID() { |
|
479 |
return "pkg"; |
|
480 |
} |
|
481 |
||
482 |
@Override |
|
483 |
public boolean validate(Map<String, ? super Object> params) |
|
57438
4a31db8d42bd
8226599: use code coverage results to remove dead code
herrick
parents:
57407
diff
changeset
|
484 |
throws ConfigException { |
57038 | 485 |
try { |
486 |
if (params == null) throw new ConfigException( |
|
487 |
I18N.getString("error.parameters-null"), |
|
488 |
I18N.getString("error.parameters-null.advice")); |
|
489 |
||
490 |
// run basic validation to ensure requirements are met |
|
491 |
// we are not interested in return code, only possible exception |
|
492 |
validateAppImageAndBundeler(params); |
|
493 |
||
58415 | 494 |
if (MAC_CF_BUNDLE_IDENTIFIER.fetchFrom(params) == null) { |
495 |
throw new ConfigException( |
|
496 |
I18N.getString("message.app-image-requires-identifier"), |
|
497 |
I18N.getString( |
|
498 |
"message.app-image-requires-identifier.advice")); |
|
499 |
} |
|
500 |
||
57038 | 501 |
// reject explicitly set sign to true and no valid signature key |
502 |
if (Optional.ofNullable(MacAppImageBuilder. |
|
503 |
SIGN_BUNDLE.fetchFrom(params)).orElse(Boolean.FALSE)) { |
|
504 |
String signingIdentity = |
|
505 |
DEVELOPER_ID_INSTALLER_SIGNING_KEY.fetchFrom(params); |
|
506 |
if (signingIdentity == null) { |
|
507 |
throw new ConfigException( |
|
508 |
I18N.getString("error.explicit-sign-no-cert"), |
|
509 |
I18N.getString( |
|
510 |
"error.explicit-sign-no-cert.advice")); |
|
511 |
} |
|
512 |
} |
|
513 |
||
514 |
// hdiutil is always available so there's no need |
|
515 |
// to test for availability. |
|
516 |
||
517 |
return true; |
|
518 |
} catch (RuntimeException re) { |
|
519 |
if (re.getCause() instanceof ConfigException) { |
|
520 |
throw (ConfigException) re.getCause(); |
|
521 |
} else { |
|
522 |
throw new ConfigException(re); |
|
523 |
} |
|
524 |
} |
|
525 |
} |
|
526 |
||
527 |
@Override |
|
57140 | 528 |
public File execute(Map<String, ? super Object> params, |
529 |
File outputParentDir) throws PackagerException { |
|
57038 | 530 |
return bundle(params, outputParentDir); |
531 |
} |
|
532 |
||
533 |
@Override |
|
57150 | 534 |
public boolean supported(boolean runtimeInstaller) { |
57438
4a31db8d42bd
8226599: use code coverage results to remove dead code
herrick
parents:
57407
diff
changeset
|
535 |
return true; |
57038 | 536 |
} |
537 |
||
58114
42df7de58e39
8230519: jpackage "--package-type" values and default
herrick
parents:
58037
diff
changeset
|
538 |
@Override |
42df7de58e39
8230519: jpackage "--package-type" values and default
herrick
parents:
58037
diff
changeset
|
539 |
public boolean isDefault() { |
42df7de58e39
8230519: jpackage "--package-type" values and default
herrick
parents:
58037
diff
changeset
|
540 |
return false; |
42df7de58e39
8230519: jpackage "--package-type" values and default
herrick
parents:
58037
diff
changeset
|
541 |
} |
42df7de58e39
8230519: jpackage "--package-type" values and default
herrick
parents:
58037
diff
changeset
|
542 |
|
57038 | 543 |
} |