Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Use Android build system, generate AARs #189

Merged
merged 1 commit into from Jun 30, 2015

Conversation

dlew
Copy link
Collaborator

@dlew dlew commented Jun 30, 2015

Also updated to latest & greatest dependencies while I was at it.

This PR gets rid of gradle-rxjava-project-plugin because it's incompatible with the Android build plugins. It's actually a smorgasbord of plugins, some of which seem to be geared only towards the Java plugin.

I think we'll end up having to import the individual plugins as necessary for release. I'm thinking we can handle that in parallel with other development; having this library work easily in Android Studio should really help out development.

minSdkVersion rootProject.ext.minSdkVersion
targetSdkVersion rootProject.ext.targetSdkVersion
versionCode 1
versionName "1.0"
Copy link
Contributor

Choose a reason for hiding this comment

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

versionName value can be set with single quotes '1.0' (saves some nanoseconds on string parsing -> faster build 😄)

Also updated to latest & greatest dependencies while I was at it.
@dlew dlew force-pushed the dlew/android-plugin-builds branch from 863923d to 32ae9fc Compare June 30, 2015 19:02
@artem-zinnatullin
Copy link
Contributor

👍 for Android Studio support, probably you also need to add androidJavadoc and androidSources tasks here or in separate PR, because now they are not generated, as far as I see.

@austynmahoney
Copy link
Contributor

I opened this in Android Studio 1.3 Beta 1. I had the SDK tools fully updated and the m2 repos set up as I would normally. Sample runs, and project builds.

It would be nice if a named aar was created instead of the default library.aar. I can add this in another PR once this is merged if you like.

👍

JakeWharton added a commit that referenced this pull request Jun 30, 2015
Use Android build system, generate AARs
@JakeWharton JakeWharton merged commit 2094c5f into ReactiveX:0.x Jun 30, 2015
@JakeWharton
Copy link
Member

The files get named appropriately by default:

$ ls rxandroid/build/outputs/aar/
total 32
-rw-r--r--  1 jw  jw   7.4K Jun 30 16:23 rxandroid-debug.aar
-rw-r--r--  1 jw  jw   7.3K Jun 30 16:23 rxandroid-release.aar

@dlew dlew deleted the dlew/android-plugin-builds branch June 30, 2015 20:45
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.

None yet

5 participants