Differential D22993

Find aggregation.dbx at its new location
ClosedPublic
Actions

Authored by ngraham on Aug 7 2019, 2:59 PM.

Details

Reviewers
vialav
Group Reviewers
Dolphin
Commits
R449:db88a5696882: Find aggregation.dbx at its new location
Summary

This file hasn't existed at that location for years. This patch updates the logic to find it at one of the several locations where it may live now.

Test Plan

Dolphin no longer shows inotify_add_watch("/home/name/.dropbox/aggregation.dbx") failed: "No such file or directory" in the console output

BUG: 410355
FIXED-IN: 19.08.1

Diff Detail

Repository
R449 Plugins for Dolphin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
ngraham requested review of this revision.Aug 7 2019, 2:59 PM
ngraham created this revision.
ngraham edited the test plan for this revision. (Show Details)Aug 7 2019, 3:00 PM
ngraham updated this revision to Diff 63292.Aug 7 2019, 3:46 PM

Instead, find aggregation.dbx fie correctly

ngraham retitled this revision from Stop looking for ~/.dropbox/aggregation.dbx, which no longer exists to Find aggregation.dbx at its new location.Aug 7 2019, 3:47 PM
ngraham edited the summary of this revision. (Show Details)
vialav accepted this revision.Aug 8 2019, 8:23 AM
vialav added a subscriber: vialav.

I've tested the patch and it has fixed the issue of ~/.dropbox/aggregation.dbx being non-existent on Dropbox nowadays in that location.

Please, consider a merge.

This revision is now accepted and ready to land.Aug 8 2019, 8:23 AM
ngraham edited the test plan for this revision. (Show Details)Aug 8 2019, 6:34 PM
Closed by commit R449:db88a5696882: Find aggregation.dbx at its new location (authored by ngraham). · Explain WhyAug 8 2019, 6:36 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

PathPackages
Mdropbox/fileviewdropboxplugin.cpp (15 lines)

Diff 63372

View Options

dropbox/fileviewdropboxplugin.cpp

Loading...