-
Notifications
You must be signed in to change notification settings - Fork 5.9k
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 read-only persistent cache #8049
Conversation
@@ -539,6 +539,10 @@ typedef struct { | |||
// Flutter application (such as compiled shader programs used by Skia). | |||
// This is optional. The string must be NULL terminated. | |||
const char* persistent_cache_path; | |||
|
|||
// If true, we'll only read the existing cache, but not write new ones. | |||
bool is_persistent_cache_read_only; |
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.
Instead of "we", please say "the engine". Also, we should only read this field if the persistent_cache_path
is non-null.
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.
Changed "we" to "engine".
Shall we also allow read-only cache for the default cache path where persistent_cache_path is nullptr?
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.
I dont think so. On Linux, the default cache path is empty and the persistent cache wont be setup in such cases.
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.
Ok. Done.
3923e9e
to
692af11
Compare
flutter/engine@6031407...87edd94 git log 6031407..87edd94 --no-merges --oneline 87edd94 Add read-only persistent cache (flutter/engine#8049) 4c94049 Move android_sdk_downloader so I can more easily deprecate it (flutter/engine#8084) 7cbd9d8 Roll src/third_party/skia 3d1b941f3a7d..bea1f94f341e (7 commits) (flutter/engine#8083) 260669c Fix deleting text when the last character is some special characters on IOS (flutter/engine#7982) The AutoRoll server is located here: https://autoroll.skia.org/r/flutter-engine-flutter-autoroll Documentation for the AutoRoller is here: https://skia.googlesource.com/buildbot/+/master/autoroll/README.md If the roll is causing failures, please contact the current sheriff (mklim@google.com), and stop the roller if necessary.
Some clients (e.g., embedded devices) prefer generating persistent cache files for the specific device beforehand, and ship them as readonly files in OTA packages.