Merge lp://staging/~laney/bamf/multiple-xdg-current-desktop-values into lp://staging/bamf

Proposed by Iain Lane
Status: Merged
Approved by: Marco Trevisan (Treviño)
Approved revision: 650
Merged at revision: 649
Proposed branch: lp://staging/~laney/bamf/multiple-xdg-current-desktop-values
Merge into: lp://staging/bamf
Diff against target: 135 lines (+27/-15)
3 files modified
src/bamf-application.c (+5/-3)
src/bamf-legacy-screen.c (+10/-1)
src/bamf-matcher.c (+12/-11)
To merge this branch: bzr merge lp://staging/~laney/bamf/multiple-xdg-current-desktop-values
Reviewer Review Type Date Requested Status
Marco Trevisan (Treviño) Approve
Unity Team Pending
Review via email: mp+307189@code.staging.launchpad.net

Commit message

Adjust for XDG_CURRENT_DESKTOP containing multiple colon-separated values

Description of the change

XDG_CURRENT_DESKTOP for Unity 7 now contains multiple values.

bamf needs to get along with this.

To post a comment you must log in.
Revision history for this message
Iain Lane (laney) wrote :

This bug makes new nautilus windows not match the launcher icon in Yakkety currently

650. By Iain Lane

Don't leak current_desktops

Revision history for this message
Marco Trevisan (Treviño) (3v1n0) wrote :

Looks good, thanks!

review: Approve

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: