Differential D716

Make sure the migrator saves its settings
ClosedPublic
Actions

Authored by ervin on Dec 28 2015, 7:50 PM.

Details

Reviewers
bensi
dfaure
franckarrecot
Commits
R4:e2b7e9910b22: Make sure the migrator saves its settings
Summary

Found on a pristine test system that the migrator was running the
migration over and over. Apparently sync() isn't called on delete in
that case as one would expect.

Diff Detail

Repository
R4 Zanshin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ervin updated this revision to Diff 1647.Dec 28 2015, 7:50 PM
ervin retitled this revision from to Make sure the migrator saves its settings.
ervin updated this object.
ervin edited the test plan for this revision. (Show Details)
ervin added reviewers: bensi, franckarrecot, dfaure.
dfaure accepted this revision.Dec 28 2015, 8:03 PM
dfaure edited edge metadata.

Strange that it worked for me. Maybe debug vs release (compiler optimizations).

This revision is now accepted and ready to land.Dec 28 2015, 8:03 PM
bensi accepted this revision.Dec 28 2015, 8:10 PM
bensi edited edge metadata.
Closed by commit R4:e2b7e9910b22: Make sure the migrator saves its settings (authored by ervin). · Explain WhyDec 28 2015, 8:59 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Msrc/zanshin/migrator/main.cpp (1 line)

Diff 1658

View Options

src/zanshin/migrator/main.cpp

Loading...