Closed Bug 785146 Opened 9 years ago Closed 9 years ago
mozprofile should set the version for permissions
We currently don't give a schema version to the permissions.sqlite db : https://github.com/mozilla/mozbase/blob/master/mozprofile/mozprofile/permissions.py#L231 We should: https://bugzilla.mozilla.org/show_bug.cgi?id=784875 ABICT, this should be version 3: https://bugzilla.mozilla.org/show_bug.cgi?id=784875#c1
Not sure if there's a better way.
Attachment #709129 - Flags: feedback?(jhammel)
Comment on attachment 709129 [details] [diff] [review] add schema version I ain't a SQLite expert but I would assume this is sufficient. Bonus points for writing a test that fails without this patch and passes with.
Attachment #709129 - Flags: feedback?(jhammel) → feedback+
And, sure enough, fails without the code change, passes with :) Awesome job, mihnea!
pushed: https://github.com/mozilla/mozbase/commit/d9e769d5bcdf5420e92fdf3b43f766e9c9a13520 ...and another bug bites the dust
Status: NEW → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
(In reply to Jeff Hammel [:jhammel] from comment #4) > And, sure enough, fails without the code change, passes with :) Awesome > job, mihnea! Yes, I checked. Thanks!
You need to log in before you can comment on or make changes to this bug.