Page MenuHomePhorge
Differential D189

Use a temp file to save session
ClosedPublic
Actions

Authored by tusooa on Sun, Oct 13, 7:29 PM.

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
Lint
Lint Not Applicable
Unit
Tests Not Applicable

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 503

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...