Merge lp://staging/~attente/indicator-keyboard/gvfs into lp://staging/indicator-keyboard

Proposed by William Hua
Status: Merged
Approved by: Sebastien Bacher
Approved revision: 105
Merged at revision: 105
Proposed branch: lp://staging/~attente/indicator-keyboard/gvfs
Merge into: lp://staging/indicator-keyboard
Diff against target: 14 lines (+6/-1)
1 file modified
tests/indicator-keyboard-test.in (+6/-1)
To merge this branch: bzr merge lp://staging/~attente/indicator-keyboard/gvfs
Reviewer Review Type Date Requested Status
PS Jenkins bot (community) continuous-integration Approve
Indicator Applet Developers Pending
Review via email: mp+181503@code.staging.launchpad.net

Commit message

Kill running gvfs instance after tests finish.

Because the XDG_RUNTIME_DIR was changed to be in the build-tree, a running gvfs instance held a lock on a file in the directory, preventing it from being cleaned properly, and preventing the build from finishing. So we send a SIGTERM to the running gvfs-fused instance, allowing the build to finish.

Description of the change

Kill running gvfs instance after tests finish.

Because the XDG_RUNTIME_DIR was changed to be in the build-tree, a running gvfs instance held a lock on a file in the directory, preventing it from being cleaned properly, and preventing the build from finishing. So we send a SIGTERM to the running gvfs-fused instance, allowing the build to finish.

To post a comment you must log in.
Revision history for this message
PS Jenkins bot (ps-jenkins) wrote :
review: Approve (continuous-integration)

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
The diff is not available at this time. You can reload the page or download it.

Subscribers

People subscribed via source and target branches

to all changes: