Skip to content

fix: Overwrite Android SDK during build#2031

Merged
bitsandfoxes merged 5 commits into
mainfrom
fix/android-sdk-copy
Feb 25, 2025
Merged

fix: Overwrite Android SDK during build#2031
bitsandfoxes merged 5 commits into
mainfrom
fix/android-sdk-copy

Conversation

@bitsandfoxes

Copy link
Copy Markdown
Contributor

Fixes #1847

These changes make sure that the SDK now overwrites the Android SDK in the generated gradle project - if already present.

@tobias-wilfert tobias-wilfert left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Rubber stamp

File.Copy(file, destinationFile);
}
var fileName = Path.GetFileName(file);
_logger.LogDebug("Copying '{0}'", fileName);

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

To where? Above we log something with gradlePath but below we do targetPath

@bruno-garcia bruno-garcia left a comment

Copy link
Copy Markdown
Member

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

forgot to unblock

@bitsandfoxes bitsandfoxes merged commit 4327a3f into main Feb 25, 2025
@bitsandfoxes bitsandfoxes deleted the fix/android-sdk-copy branch February 25, 2025 11:07
@github-actions

Copy link
Copy Markdown
Contributor
Fails
🚫 Please consider adding a changelog entry for the next release.

Instructions and example for changelog

Please add an entry to CHANGELOG.md to the "Unreleased" section. Make sure the entry includes this PR's number.

Example:

## Unreleased

- Overwrite Android SDK during build ([#2031](https://github.com/getsentry/sentry-unity/pull/2031))

If none of the above apply, you can opt out of this check by adding #skip-changelog to the PR description.

Generated by 🚫 dangerJS against 3f645dd

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.

SDK not overwriting Android libs when building

4 participants