Skip to content

Conversation

@TChukwuleta
Copy link
Collaborator

No description provided.

@Ghander
Copy link
Collaborator

Ghander commented Jul 1, 2025

have you pulled the latest changes from master?

@TChukwuleta
Copy link
Collaborator Author

Just pulled latest...

This PR includes unarchive cta to the archive done in #225

@TChukwuleta
Copy link
Collaborator Author

TChukwuleta commented Jul 1, 2025

Hey @Ghander the android build is failing

BTCPayApp.UI -> D:\a\app\app\BTCPayApp.UI\bin\Release\net8.0\BTCPayApp.UI.dll
C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.0.154\tools\Xamarin.Android.Common.targets(2447,2): error MSB6006: "java.exe" exited with code 2. [D:\a\app\app\BTCPayApp.Maui\BTCPayApp.Maui.csproj::TargetFramework=net8.0-android]

@Ghander
Copy link
Collaborator

Ghander commented Jul 1, 2025

ok, I am building your code locally to see if I can recreate the issue.

@Ghander
Copy link
Collaborator

Ghander commented Jul 1, 2025

Hey @Ghander the android build is failing

BTCPayApp.UI -> D:\a\app\app\BTCPayApp.UI\bin\Release\net8.0\BTCPayApp.UI.dll C:\Program Files\dotnet\packs\Microsoft.Android.Sdk.Windows\34.0.154\tools\Xamarin.Android.Common.targets(2447,2): error MSB6006: "java.exe" exited with code 2. [D:\a\app\app\BTCPayApp.Maui\BTCPayApp.Maui.csproj::TargetFramework=net8.0-android]

Does the maui project build for you?

@TChukwuleta
Copy link
Collaborator Author

yes.. it does build locally on my end

change workflow trigger from pull_request to pull_request_target, which runs in the context of the base repository and can access secrets.
@Ghander
Copy link
Collaborator

Ghander commented Jul 1, 2025

@TChukwuleta pull the latest changes from master and make another PR for me, if you would.

@TChukwuleta
Copy link
Collaborator Author

Just rebased .. do you want me to raise an entirely new PR @Ghander

@TChukwuleta
Copy link
Collaborator Author

It passed @Ghander

@dstrukt dstrukt self-requested a review July 2, 2025 15:13
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants