Mail

Change SQL migration code to use GResource too

Bug #1598444 reported by Zisu Andrei
8
This bug affects 1 person
Affects Status Importance Assigned to Milestone
Mail
Confirmed
Low
Unassigned

Bug Description

Corentin made some changes to the client to use GResource for handling of paths. This should be expanded to cover the SQL migration code so we can get rid of the get_resource_directory method.

Zisu Andrei (matzipan)
Changed in pantheon-mail:
importance: Undecided → Wishlist
Revision history for this message
Zisu Andrei (matzipan) wrote :

According to mhall, this would easier packaging as snap, flatpack or appimage.

Revision history for this message
Danielle Foré (danrabbit) wrote : Re: Remove SQL migration code

According to Corentin, this is legacy code leftover from Geary and is useless for us anyways

summary: - Change SQL migration code to use GResource too
+ Remove SQL migration code
Changed in pantheon-mail:
status: New → Confirmed
importance: Wishlist → Low
Revision history for this message
Zisu Andrei (matzipan) wrote :

I believe it is used on install.

Revision history for this message
Zisu Andrei (matzipan) wrote :

As discussed on Slack and confirmed by Ralph, everytime a new account is setup, the SQL files are used to instantiate a new sqlite database.

Revision history for this message
Ralph Plawetzki (purejava) wrote :

> I believe it is used on install.

+1

Zisu Andrei (matzipan)
summary: - Remove SQL migration code
+ Change SQL migration code to use GResource too
To post a comment you must log in.
This report contains Public information  
Everyone can see this information.
Subscribing...

Other bug subscribers

Remote bug watches

Bug watches keep track of this bug in other bug trackers.