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

Working with retrolambda #79

Closed
wants to merge 3 commits into from

Conversation

promeG
Copy link
Contributor

@promeG promeG commented Mar 4, 2015

Retrolambda perform the bytecode weaving after javaComplie task which will override hugo's weaving.

Change the timing of hugo's weaving task from javaComplie.dolast to dex.doFirst can solve this problem.
#77 #78

@JakeWharton
Copy link
Owner

I can't tell what the change is because you reformatted a bunch of lines with four spaces instead of two. Can you reformat back to match the project style?

@promeG promeG force-pushed the working-with-retrolambda branch from 98f6d5b to d7da97e Compare March 6, 2015 07:13
@promeG
Copy link
Contributor Author

promeG commented Mar 6, 2015

Hi Wharton,

I have modified the code format.

Thanks.

@TonyTangAndroid
Copy link

Hello Jake, might I ask when this pull request will be merged and release a new build? As for now, when I use retrolambda, I have always run into the issue that promeG mentioned. It will be highly appreciated if you are kind to do so in the near future.

@JakeWharton
Copy link
Owner

Cleaned up and merged the first two commits. I'm not sure removing library project support is needed at this time.

@JakeWharton JakeWharton closed this May 6, 2015
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.

3 participants