Propagate build results to dependent projects using artifacts#81
Merged
paxbun merged 22 commits intoApr 3, 2025
Merged
Conversation
paxbun
added a commit
that referenced
this pull request
Mar 31, 2025
… on Android 34 or higher (#82) Made the Android bindings fallback to `UniffiJnaCleaner` when `AndroidSystemCleaner` is not available even for API level 34 or higher. This is for Compose preview. Manually tested with the Android tutorial added by #62. Needs #81 to be merged. Setting `android.render.sandbox=false` in `bin/idea.properties` is also required.
a074323 to
89418ee
Compare
SalvatoreT
reviewed
Apr 2, 2025
This reverts commit 54ff42a.
bda763b to
5440d1f
Compare
SalvatoreT
approved these changes
Apr 3, 2025
This was referenced Apr 3, 2025
paxbun
added a commit
to paxbun/gobley
that referenced
this pull request
Apr 4, 2025
paxbun
added a commit
to paxbun/gobley
that referenced
this pull request
Apr 4, 2025
paxbun
added a commit
to paxbun/gobley
that referenced
this pull request
Apr 4, 2025
paxbun
added a commit
to paxbun/gobley
that referenced
this pull request
Apr 4, 2025
paxbun
added a commit
that referenced
this pull request
Apr 4, 2025
…ries (#97) ## Changes - Added `CargoJvmBuildVariant.{jarTaskProvider, libraryFiles}`. This is useful for users requiring custom publication logic. - Added `GradleUtils`. This object contains heuristics to detect specific IDE use cases. - Restored the copy task removed by #81, on which the main sourceSet's `ProcessResource` task was dependent. The dependency is active only when `GradleUtils. invokedByKotlinJvmBuild()` is true, which means it was invoked by the run button in IntelliJ/Android Studio. ## Testing - Tested with the JVM tutorial added by #62. The project was cleaned before testing. - Tested with a multi-module project as well. ## Issues Fixed Fixes: #95
This file contains hidden or bidirectional Unicode text that may be interpreted or compiled differently than what appears below. To review, open the file in an editor that reveals hidden Unicode characters.
Learn more about bidirectional Unicode characters
Sign up for free
to join this conversation on GitHub.
Already have an account?
Sign in to comment
2 participants
Add this suggestion to a batch that can be applied as a single commit.This suggestion is invalid because no changes were made to the code.Suggestions cannot be applied while the pull request is closed.Suggestions cannot be applied while viewing a subset of changes.Only one suggestion per line can be applied in a batch.Add this suggestion to a batch that can be applied as a single commit.Applying suggestions on deleted lines is not supported.You must change the existing code in this line in order to create a valid suggestion.Outdated suggestions cannot be applied.This suggestion has been applied or marked resolved.Suggestions cannot be applied from pending reviews.Suggestions cannot be applied on multi-line comments.Suggestions cannot be applied while the pull request is queued to merge.Suggestion cannot be applied right now. Please check back later.
Changes
jarJvmRustRuntime*tasks wrapping Rust dynamic libraries into Jar files. Previously, Rust dynamic libraries were packaged into one single Jar file with.classfiles.ProjectDependency.dependencyProjectAPI fromDependencyUtils.Config.crateNameandpackageRoot, which are intended to be used only on the Gradle side.GobleyKotlinSourceSetCollectioninternally, which provides properties computing sourceSet names and returning those sourceSets, preventing manual branching by plugin IDs.PluginUtils.withKotlinPlugin.androidx.test.MergeUniffiConfigTaskuses the wrong value for external crate names.MergeUniffiConfigTaskfrombuild/generated/uniffi/uniffi.tomltobuild/intermediates/merged_uniffi_config/uniffi.toml.Testing
Added an Android local unit test (not an instrumented test) to
:tests:uniffi:coverall-android. Added Cargo packages for:tests:uniffi:coverall-jvmand:tests:uniffi:coverall-android.Issues Fixed
Fixes: #80, Fixes: #84