(Idea) Lazily read APK path from variant about gradle-play-publisher

[ZacSweers]

Here’s an example of how this would look in action, from another gradle plugin we use in our project.

[ZacSweers]

Actually, after talking with that example library's developer a bit, he came up with an easier solution that might work here as well. Instead of getting the APK file upfront in apply(), he added an ApplicationVariant field to the upload task instead and sets the variant on that field instead. Then in the upload task (which runs almost last), it has the variant object and only then retrieves the APK file, which would have the updated name.

Here's the small commit where he implemented this. It's a fairly similar structure to this project as far as the configuration being in an apply method and the upload being a separate task.

What do you think?

[bhurling]

Looks very interesting. I have already been looking for a way to honor the custom APK names. The lazy approach looks very promising. Shouldn't be too hard to implement.

[ZacSweers]

Cool, please let me know how it goes or if you want me to take a shot at it :)

[ChristianKatzmann]

@bhurling Is the branch lazy_apk_name ready for review? Should we open a PR for this?

[ZacSweers]

I took a peak and left a comment with a couple suggested ways to avoid the
deprecation issue mentioned in the last commit on that branch, just
mentioning in case that's the only thing it's waiting on :)

On Mon, Dec 15, 2014, 2:24 AM Christian Becker [email protected]
wrote:

@bhurling https://github.com/bhurling Is the branch lazy_apk_name ready
for review? Should we open a PR for this?

—
Reply to this email directly or view it on GitHub
#28 (comment)
.

[bhurling]

I slightly changed the part where we find the apk file output:

def apkOutput = variant.outputs.find { variantOutput -> variantOutput instanceof ApkVariantOutput }

Just in case someone does not use the .apk extension in his custom filename. Also I dropped the part that checks for the zipAlign task. That part is already checked during the plugin setup.

What do you think?

[ZacSweers]

Didn't even think of that. If it works then I say go for it!

On Mon, Dec 15, 2014, 2:58 AM Björn Hurling [email protected]
wrote:

I slightly changed the part where we find the apk file output:

def apkOutput = variant.outputs.find { variantOutput -> variantOutput instanceof ApkVariantOutput }

Just in case someone does not use the .apk extension in his custom
filename. Also I dropped the part that checks for the zipAlign task. That
part is already checked during the plugin setup.

What do you think?

—
Reply to this email directly or view it on GitHub
#28 (comment)
.