From mboxrd@z Thu Jan  1 00:00:00 1970
Return-Path: <pve-devel-bounces@lists.proxmox.com>
Received: from firstgate.proxmox.com (firstgate.proxmox.com [IPv6:2a01:7e0:0:424::9])
	by lore.proxmox.com (Postfix) with ESMTPS id 1A82F1FF176
	for <inbox@lore.proxmox.com>; Fri, 21 Feb 2025 15:45:43 +0100 (CET)
Received: from firstgate.proxmox.com (localhost [127.0.0.1])
	by firstgate.proxmox.com (Proxmox) with ESMTP id A1F1D4D59;
	Fri, 21 Feb 2025 15:45:40 +0100 (CET)
Mime-Version: 1.0
Date: Fri, 21 Feb 2025 15:45:05 +0100
Message-Id: <D7Y7DJ4Y5JAT.KAPGBO9UNXY2@proxmox.com>
From: "Shannon Sterz" <s.sterz@proxmox.com>
To: "Proxmox VE development discussion" <pve-devel@lists.proxmox.com>
X-Mailer: aerc 0.18.2-0-ge037c095a049-dirty
References: <20250220143745.135908-1-a.abraham@proxmox.com>
In-Reply-To: <20250220143745.135908-1-a.abraham@proxmox.com>
X-SPAM-LEVEL: Spam detection results:  0
 AWL -0.081 Adjusted score from AWL reputation of From: address
 BAYES_00                 -1.9 Bayes spam probability is 0 to 1%
 DMARC_MISSING             0.1 Missing DMARC policy
 KAM_DMARC_STATUS 0.01 Test Rule for DKIM or SPF Failure with Strict Alignment
 RCVD_IN_VALIDITY_CERTIFIED_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to
 Validity was blocked. See
 https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more
 information.
 RCVD_IN_VALIDITY_RPBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to
 Validity was blocked. See
 https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more
 information.
 RCVD_IN_VALIDITY_SAFE_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to
 Validity was blocked. See
 https://knowledge.validity.com/hc/en-us/articles/20961730681243 for more
 information.
 SPF_HELO_NONE           0.001 SPF: HELO does not publish an SPF Record
 SPF_PASS               -0.001 SPF: sender matches SPF record
 URIBL_BLOCKED 0.001 ADMINISTRATOR NOTICE: The query to URIBL was blocked. See
 http://wiki.apache.org/spamassassin/DnsBlocklists#dnsbl-block for more
 information. [kotlin.android, android.com, gradle.org]
 URIBL_SBL_A 0.1 Contains URL's A record listed in the Spamhaus SBL blocklist
 [142.250.185.206]
Subject: Re: [pve-devel] [PATCH pve_flutter_frontend] Updated Gradle version
 and some build dependencies.
X-BeenThere: pve-devel@lists.proxmox.com
X-Mailman-Version: 2.1.29
Precedence: list
List-Id: Proxmox VE development discussion <pve-devel.lists.proxmox.com>
List-Unsubscribe: <https://lists.proxmox.com/cgi-bin/mailman/options/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=unsubscribe>
List-Archive: <http://lists.proxmox.com/pipermail/pve-devel/>
List-Post: <mailto:pve-devel@lists.proxmox.com>
List-Help: <mailto:pve-devel-request@lists.proxmox.com?subject=help>
List-Subscribe: <https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel>, 
 <mailto:pve-devel-request@lists.proxmox.com?subject=subscribe>
Reply-To: Proxmox VE development discussion <pve-devel@lists.proxmox.com>
Cc: Alexander Abraham <a.abraham@proxmox.com>
Content-Type: text/plain; charset="us-ascii"
Content-Transfer-Encoding: 7bit
Errors-To: pve-devel-bounces@lists.proxmox.com
Sender: "pve-devel" <pve-devel-bounces@lists.proxmox.com>

On Thu Feb 20, 2025 at 3:37 PM CET, Alexander Abraham wrote:
> The Flutter frontend of PVE was not compilable for Android
> with the versions of different tools set in the project files.
> The versions of the tools causing this problem was updated
> and the app compiles for Android.
>
> Signed-off-by: Alexander Abraham <a.abraham@proxmox.com>
> ---
>  android/app/build.gradle                         | 10 +++++++++-
>  android/gradle/wrapper/gradle-wrapper.properties |  2 +-
>  android/settings.gradle                          |  4 ++--
>  3 files changed, 12 insertions(+), 4 deletions(-)
>
> diff --git a/android/app/build.gradle b/android/app/build.gradle
> index 4f827a5..59c318d 100644
> --- a/android/app/build.gradle
> +++ b/android/app/build.gradle
> @@ -45,6 +45,7 @@ def keystoreSecret = System.getenv('PROXMOX_APP_KEYSTORE_SECRET')
>
>  android {
>      compileSdkVersion 34
> +    namespace = "com.proxmox.app.pve_flutter_frontend"

i haven't tested this, don't have a flutter setup here, but the
inconsistent use of "=" seems strange to me. looking at the android
documentation you are supposed to use either "=" consistently in a
build.gradle.kts file or not in a build.gradle file.

did you mix them intentionally?

[1]: https://developer.android.com/build/gradle-build-overview#kts

>
>      sourceSets {
>          main.java.srcDirs += 'src/main/kotlin'
> @@ -52,7 +53,7 @@ android {
>
>
>      defaultConfig {
> -        applicationId "com.proxmox.app.pve_flutter_frontend"
> +        applicationId namespace
>          minSdkVersion 23
>          targetSdkVersion 34
>          versionCode flutterVersionCode.toInteger()
> @@ -74,9 +75,16 @@ android {
>              signingConfig signingConfigs.release
>          }
>      }
> +    compileOptions {
> +        sourceCompatibility JavaVersion.VERSION_17
> +        targetCompatibility JavaVersion.VERSION_17
> +    }
>      lint {
>          disable 'InvalidPackage'
>      }
> +    kotlinOptions{
> +        jvmTarget = '17'
> +    }
>  }
>
>  flutter {
> diff --git a/android/gradle/wrapper/gradle-wrapper.properties b/android/gradle/wrapper/gradle-wrapper.properties
> index 0732d12..90d1364 100644
> --- a/android/gradle/wrapper/gradle-wrapper.properties
> +++ b/android/gradle/wrapper/gradle-wrapper.properties
> @@ -1,6 +1,6 @@
>  #Mon Apr 24 14:18:06 CEST 2023
>  distributionBase=GRADLE_USER_HOME
>  distributionPath=wrapper/dists
> -distributionUrl=https\://services.gradle.org/distributions/gradle-7.6.1-bin.zip
> +distributionUrl=https\://services.gradle.org/distributions/gradle-8.10.2-all.zip
>  zipStoreBase=GRADLE_USER_HOME
>  zipStorePath=wrapper/dists
> diff --git a/android/settings.gradle b/android/settings.gradle
> index 4fc86fd..1da49ad 100644
> --- a/android/settings.gradle
> +++ b/android/settings.gradle
> @@ -18,8 +18,8 @@ pluginManagement {
>
>  plugins {
>      id "dev.flutter.flutter-plugin-loader" version "1.0.0"
> -    id "com.android.application" version "7.4.2" apply false
> -    id "org.jetbrains.kotlin.android" version "1.7.20" apply false
> +    id "com.android.application" version "8.7.0" apply false
> +    id "org.jetbrains.kotlin.android" version "1.8.22" apply false
>  }
>
>  include ':app'



_______________________________________________
pve-devel mailing list
pve-devel@lists.proxmox.com
https://lists.proxmox.com/cgi-bin/mailman/listinfo/pve-devel