From 3b8513156108c9c3d9d9f76d7b73db6e30373cf4 Mon Sep 17 00:00:00 2001 From: Norman Breau Date: Tue, 13 Apr 2021 07:13:01 -0300 Subject: [PATCH] breaking: remove keystore password prompt (#1048) --- bin/templates/project/app/build.gradle | 24 ++---------------------- framework/cordova.gradle | 25 ------------------------- 2 files changed, 2 insertions(+), 47 deletions(-) diff --git a/bin/templates/project/app/build.gradle b/bin/templates/project/app/build.gradle index 7f82d47fe2..022bb8951f 100644 --- a/bin/templates/project/app/build.gradle +++ b/bin/templates/project/app/build.gradle @@ -308,9 +308,9 @@ android { release { // These must be set or Gradle will complain (even if they are overridden). keyAlias = "" - keyPassword = "__unset" // And these must be set to non-empty in order to have the signing step added to the task graph. + keyPassword = "" storeFile = null - storePassword = "__unset" + storePassword = "" } } buildTypes { @@ -350,26 +350,6 @@ dependencies { // SUB-PROJECT DEPENDENCIES END } -def promptForReleaseKeyPassword() { - if (!cdvReleaseSigningPropertiesFile) { - return; - } - if ('__unset'.equals(android.signingConfigs.release.storePassword)) { - android.signingConfigs.release.storePassword = privateHelpers.promptForPassword('Enter key store password: ') - } - if ('__unset'.equals(android.signingConfigs.release.keyPassword)) { - android.signingConfigs.release.keyPassword = privateHelpers.promptForPassword('Enter key password: '); - } -} - -gradle.taskGraph.whenReady { taskGraph -> - taskGraph.getAllTasks().each() { task -> - if(['validateReleaseSigning', 'validateSigningRelease', 'validateSigningArmv7Release', 'validateSigningX76Release'].contains(task.name)) { - promptForReleaseKeyPassword() - } - } -} - def addSigningProps(propsFilePath, signingConfig) { def propsFile = file(propsFilePath) def props = new Properties() diff --git a/framework/cordova.gradle b/framework/cordova.gradle index 36750b68fa..ba9b37a19b 100644 --- a/framework/cordova.gradle +++ b/framework/cordova.gradle @@ -131,30 +131,6 @@ def doExtractStringFromManifest(name) { return matcher.group(1) } -def doPromptForPassword(msg) { - if (System.console() == null) { - def ret = null - new SwingBuilder().edt { - dialog(modal: true, title: 'Enter password', alwaysOnTop: true, resizable: false, locationRelativeTo: null, pack: true, show: true) { - vbox { - label(text: msg) - def input = passwordField() - button(defaultButton: true, text: 'OK', actionPerformed: { - ret = input.password; - dispose(); - }) - } - } - } - if (!ret) { - throw new GradleException('User canceled build') - } - return new String(ret) - } else { - return System.console().readPassword('\n' + msg); - } -} - def doGetConfigXml() { def xml = file("src/main/res/xml/config.xml").getText() // Disable namespace awareness since Cordova doesn't use them properly @@ -183,7 +159,6 @@ ext { privateHelpers.findLatestInstalledBuildTools = { doFindLatestInstalledBuildTools('19.1.0') } privateHelpers.extractIntFromManifest = { name -> doExtractIntFromManifest(name) } privateHelpers.extractStringFromManifest = { name -> doExtractStringFromManifest(name) } - privateHelpers.promptForPassword = { msg -> doPromptForPassword(msg) } privateHelpers.ensureValueExists = { filePath, props, key -> doEnsureValueExists(filePath, props, key) } // These helpers can be used by plugins / projects and will not change.