-
Notifications
You must be signed in to change notification settings - Fork 7
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
gradle-eta is broken for Gradle >= 5.1 #43
Comments
@rahulmutt just finding my way around ETA and having great fun. Could this issue be causing building (for example) the simple example within this repo from failing when attempting to build with Gradle 5.4.1? The error I am seeing is
Incidentally this is when I attempt
or
|
I got the following:
Is that the same error you're getting? |
Nope that wasn't the error that I got. The following is a off of a clean clone:
Incidentally on the above quote I have not attempted to run "gradle wrapper" so it is running on 4.7. When I do upgrade it works again. Again the quote below off of my terminal:
Not sure how to proceed or assist you guys from here. |
BTW the upgrading of the wrapper on the within the test sample makes no difference - sample still fails. |
Ok I think there are several issues here that occur at different JDK versions + different Gradle versions. I'll see if I can do a setup that tests this properly. |
@graeme-lockley I just released v0.8.1 which should fix the issues you're facing. On my system, Gradle 5.4.1 with JDK 12 seems to be working. Can you try again and let me know it goes? |
I am able to build at the top level with Gradle 5.4.1, however, when I add "apply plugin: 'eta-android'" to my app/build.gradle, I receive these errors:
|
@argideritzalpea This seems specific to the |
Thank you @rahulmutt ! Can't wait to get it working... |
I'm facing the same issue; the 'latest' set of plugins I can get working together is:
Anything higher runs into one of the problems outlined in the discussion. Although it works, it's not great as it's missing all the updates and bug fixes in later versions of gradle-eta. So it's pretty common to run into some hard-to-scrutinize exit error 1 / 2 at build phase, especially when I need to modify some dependencies... |
Using getStringName in eta-android Should this be changed to Here is what I found: android source gradle-3.4.0 branch |
Gradle v4+ and v5 deprecates a lots, in v4, the methods still exist, but are purged in v5. The current source can be fix by changing Also in public DefaultEtaSourceSet(Optional<SourceSet> sourceSet,
String name,
String displayName,
SourceDirectorySetFactory sourceDirectorySetFactory) {
this.sourceSet = sourceSet.isPresent() ? sourceSet.get() : null;
/// ...
} There are many more deprecated methods, but fixing these passes the compile error, gradle configure build. However I cannot get the build to successfully loaded on Gradle Android project. The error code return is always |
So the news is I got Android Studio to pass the build after register the resolvedEta provider with this: // EtaExtension.java
resolvedEta.set(new ResolvedExecutable(
parseStringProperty(project, "etlasPath"),
parseStringProperty(project, "etlasVersion")
)); But I think my project configuration is wrong because I got:
I don't know why |
Great news! could you share your project configuration (ideally a link to the project) and the entire log produced by |
No description provided.
The text was updated successfully, but these errors were encountered: