Differential D17573

Fix the X11TimestampUpdateTest
ClosedPublic
Actions

Authored by graesslin on Dec 14 2018, 6:10 AM.

Details

Reviewers
zzag
Group Reviewers
KWin
Commits
R108:c3e712e967bd: Fix the X11TimestampUpdateTest
Summary

It also searches for the platform plugin, so we need to ensure it finds
it in the build directory.

This was a regression caused by building all platform plugins in the
correct location.

Test Plan

strace on the failing test, verified correct plugin is loaded

Diff Detail

Repository
R108 KWin
Lint
Automatic diff as part of commit; lint not applicable.
Unit
Automatic diff as part of commit; unit tests not applicable.
graesslin created this revision.Dec 14 2018, 6:10 AM
Restricted Application added a project: KWin. · View Herald TranscriptDec 14 2018, 6:10 AM
Restricted Application added a subscriber: kwin. · View Herald Transcript
graesslin requested review of this revision.Dec 14 2018, 6:10 AM
zzag accepted this revision.Dec 14 2018, 8:31 AM
zzag added a subscriber: zzag.
zzag added inline comments.
autotests/test_x11_timestamp_update.cpp
51

Please add an explanatory comment.

This revision is now accepted and ready to land.Dec 14 2018, 8:31 AM
Closed by commit R108:c3e712e967bd: Fix the X11TimestampUpdateTest (authored by graesslin). · Explain WhyDec 14 2018, 4:12 PM
This revision was automatically updated to reflect the committed changes.

Revision Contents
Changeset List

Diff 47569

View Options

autotests/test_x11_timestamp_update.cpp

Loading...