Skip to content

Conversation

petekanev
Copy link
Contributor

@petekanev petekanev commented Apr 4, 2018

Fixes #993 by removing the use of the deprecated compile methods when declaring project dependencies, and slightly altering the dependency artifacts traversal logic.

Update: the fix works when tested locally on my Windows workstation with Android Build-Tools 28.0.0-rc1, however it fails on both the Jenkins and Travis CI

@petekanev petekanev requested review from Plamen5kov and darind April 4, 2018 11:33
@petekanev petekanev changed the base branch from master to release April 4, 2018 11:36
@ns-bot
Copy link

ns-bot commented Apr 4, 2018

💔

1 similar comment
@ns-bot
Copy link

ns-bot commented Apr 4, 2018

💔

@petekanev petekanev force-pushed the pete/fix-compile-deps branch from 60d4521 to 8b7870b Compare April 4, 2018 11:56
@ns-bot
Copy link

ns-bot commented Apr 4, 2018

💔

@petekanev petekanev removed the request for review from Plamen5kov April 5, 2018 07:45
@petekanev petekanev force-pushed the pete/fix-compile-deps branch from 8b7870b to 83575d6 Compare April 5, 2018 08:36
@ns-bot
Copy link

ns-bot commented Apr 5, 2018

💔

@petekanev petekanev force-pushed the pete/fix-compile-deps branch from 83575d6 to 2b12f03 Compare April 5, 2018 08:51
@ns-bot
Copy link

ns-bot commented Apr 5, 2018

💔

@petekanev petekanev force-pushed the pete/fix-compile-deps branch from 2b12f03 to 7fd5f0a Compare April 5, 2018 10:22
@ns-bot
Copy link

ns-bot commented Apr 5, 2018

💔

@ns-bot
Copy link

ns-bot commented Apr 10, 2018

💚

@ns-bot
Copy link

ns-bot commented Apr 12, 2018

💚

@ns-bot
Copy link

ns-bot commented Apr 16, 2018

💔

@ns-bot
Copy link

ns-bot commented Apr 16, 2018

💚

@ns-bot
Copy link

ns-bot commented Apr 16, 2018

💚

@vtrifonov vtrifonov merged commit 0917329 into release Apr 16, 2018
@vtrifonov vtrifonov deleted the pete/fix-compile-deps branch April 16, 2018 13:58
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.

4 participants