Page MenuHomePhorge
Differential D189

Use a temp file to save session
ClosedPublic
Actions

Authored by tusooa on Sun, Oct 13, 7:29 PM.
Tags
None
Referenced Files
F48076: D189.1729309078.diff
Thu, Oct 17, 8:37 PM
F47223: D189.1729261436.diff
Thu, Oct 17, 7:23 AM
F46985: D189.1729239894.diff
Thu, Oct 17, 1:24 AM
Subscribers
None

Details

Reviewers
nannanko
Group Reviewers
O1: the Kazv Project
Commits
rKc3a37a47cb61: Use a temp file to save session
Summary

To avoid losing data when saving session, we write to a temp file first, and then move it into place.

Type: add

BUG: https://iron.lily-is.land/T143

Test Plan

Verify unit tests pass. Open kazv and verify it can save data properly.

Diff Detail

Repository
rK kazv
Branch
tusooa/stacked
Lint
No Lint Coverage
Unit
No Test Coverage
Build Status
Buildable 301
Build 590: GitLab CI for kazv
Build 589: arc lint + arc unit

Event Timeline

tusooa created this revision.Sun, Oct 13, 7:29 PM
Owners added a reviewer: O1: the Kazv Project.Sun, Oct 13, 7:29 PM
Harbormaster returned this revision to the author for changes because remote builds failed.Sun, Oct 13, 7:44 PM
Harbormaster failed remote builds in B297: Diff 496!
tusooa updated this revision to Diff 497.Mon, Oct 14, 8:47 AM

Use an unprivileged user to build and test

Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Oct 14, 8:51 AM
Harbormaster failed remote builds in B298: Diff 497!
tusooa updated this revision to Diff 498.Mon, Oct 14, 9:03 AM

Fix calling build-unprivileged.sh

Harbormaster returned this revision to the author for changes because remote builds failed.Mon, Oct 14, 9:16 AM
Harbormaster failed remote builds in B299: Diff 498!
tusooa updated this revision to Diff 499.Mon, Oct 14, 10:03 AM

Change to build dir ahead of time

tusooa updated this revision to Diff 500.Mon, Oct 14, 10:24 AM

Fix release build

Harbormaster failed remote builds in B300: Diff 499!Mon, Oct 14, 10:31 AM
Harbormaster completed remote builds in B301: Diff 500.Mon, Oct 14, 10:41 AM
tusooa requested review of this revision.Mon, Oct 14, 10:41 AM
nannanko accepted this revision.Wed, Oct 16, 6:13 AM
This revision is now accepted and ready to land.Wed, Oct 16, 6:13 AM
Closed by commit rKc3a37a47cb61: Use a temp file to save session. · Explain WhyWed, Oct 16, 3:25 PM
This revision was automatically updated to reflect the committed changes.
tusooa added a commit: rKc3a37a47cb61: Use a temp file to save session.

Revision Contents
Changeset List

Diff 500

View Options

packaging/GNU-Linux/appimage/build-unprivileged.sh

Loading...
View Options

packaging/GNU-Linux/appimage/build.sh

Loading...
View Options

src/matrix-sdk.hpp

Loading...
View Options

src/matrix-sdk.cpp

Loading...
View Options

src/tests/matrix-sdk-sessions-test.hpp

Loading...
View Options

src/tests/matrix-sdk-sessions-test.cpp

Loading...