Skip to content
This repository has been archived by the owner on Jan 15, 2024. It is now read-only.

Fix build fail caused by Glide #105

Merged
merged 3 commits into from
Jul 23, 2017
Merged

Fix build fail caused by Glide #105

merged 3 commits into from
Jul 23, 2017

Conversation

bobheadxi
Copy link
Collaborator

@bobheadxi bobheadxi commented Jul 23, 2017

Issues

#104

Changes

Temporarily fix build fail caused by Glide v4 update by updating implementation of GlideConfigModule

@bobheadxi bobheadxi requested a review from d4rken July 23, 2017 01:57
@bobheadxi bobheadxi mentioned this pull request Jul 23, 2017
Copy link
Member

@d4rken d4rken left a comment

Choose a reason for hiding this comment

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

@d4rken
Copy link
Member

d4rken commented Jul 23, 2017

The issue is that 4.0.0-SNAPSHOT is always the latest bleeding edge version, so depending on when the gradle cache is build, 4.0.0-SNAPSHOT is different then before.

The solution is to replace 4.0.0-SNAPSHOT with a static reference or to fix the code that throws the error.

@@ -19,7 +20,7 @@ public void applyOptions(Context context, GlideBuilder builder) {
}

@Override
public void registerComponents(Context context, Registry registry) {
public void registerComponents(Context context, Glide glide, Registry registry) {
Copy link
Collaborator Author

@bobheadxi bobheadxi Jul 23, 2017

Choose a reason for hiding this comment

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

This was a change recommended by Android Studio, although according to https://github.com/bumptech/glide/blob/master/library/src/main/java/com/bumptech/glide/module/GlideModule.java looks like GlideModule is slated to be removed for 4.0.0

@coveralls
Copy link

Coverage Status

Coverage increased (+0.3%) to 0.615% when pulling 750c2d6 on bobheadxi:issue-104 into 6996d5e on d4rken:dev.

@bobheadxi bobheadxi merged commit a74a93a into d4rken-org:dev Jul 23, 2017
@bobheadxi bobheadxi changed the title Add specific Glide snapshot to gradle Fix build fail caused by Glide Jul 23, 2017
bobheadxi added a commit that referenced this pull request Jul 23, 2017
@bobheadxi
Copy link
Collaborator Author

Ah oops, forgot to change the commit name

@bobheadxi bobheadxi deleted the issue-104 branch July 28, 2017 03:08
Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

None yet

3 participants