-
Notifications
You must be signed in to change notification settings - Fork 6.1k
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
Add Glide.setModulesEnabled API for 3.x to allow apps to disable Manifest parsing #1754
Conversation
@@ -137,6 +139,21 @@ public static File getPhotoCacheDir(Context context, String cacheName) { | |||
} | |||
|
|||
/** | |||
* Enable or disable the parsing of AndroidManifest.xml | |||
* looking for {@link GlideModule} implementations. |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Line break is a little early here.
Thanks! |
Is there a snapshot build in maven anywhere that has this fix? as far as I can tell, https://oss.sonatype.org/content/repositories/snapshots/com/github/bumptech/glide/glide/3.8.0-SNAPSHOT/ is pretty out of date. If not, I can always build myself. |
Not totally sure how this works, but it appears that there are snapshots built from 2/25 in that same folder: https://oss.sonatype.org/content/repositories/snapshots/com/github/bumptech/glide/glide/3.8.0-SNAPSHOT/glide-3.8.0-20170225.215140-15.jar It looks like it also keeps around a few older versions? |
Description
Adding a static flag to Glide to disable parsing the manifest to find configured modules.
Added unit tests.
Same as #1753 but backported to 3.0 branch
Fixes #684
Motivation and Context
he Dropbox app has been crashing on launch approximately 9000 times a day due to RuntimeExceptions thrown by PackageManager. We'd like to remove the dependency on PackageManager from our startup path, and we don't use any GlideModules. While #1742 in 4.0 will address this long term, we'd like a short term way to run Glide without runtime manifest parsing.
In order to enable unit testing, I decided to make this a set method that takes a boolean.