Merge lp:~sil2100/unity-2d/oneiric-security into lp:unity-2d/4.0

Proposed by Łukasz Zemczak
Status: Merged
Approved by: Gerry Boland
Approved revision: 753
Merged at revision: 751
Proposed branch: lp:~sil2100/unity-2d/oneiric-security
Merge into: lp:unity-2d/4.0
Diff against target: 29 lines (+8/-3)
2 files modified
debian/changelog (+7/-1)
panel/applets/appname/windowhelper.cpp (+1/-2)
To merge this branch: bzr merge lp:~sil2100/unity-2d/oneiric-security
Reviewer Review Type Date Requested Status
Gerry Boland (community) Approve
Review via email:

Commit message

releasing version 4.12.0-0ubuntu1.2
Cherry-picked from unity-2d 5.0 trunk (rev 1119): [panel] Use the correct event timestamp when closing an application window.
Also corrected the versioning history.

Description of the change

Backported a patch from 5.0 for the needs of a -security update.

This batch of changes also corrects the versioning (since ubuntu2 was never released, just ubuntu1.1 in oneiric-updates). I changes the changelog accordingly as well.

It would be best not to merge it in but just pull/push, but I am not a member of unity-2d-team (sadly)...

To post a comment you must log in.
Gerry Boland (gerboland) wrote :

I'm a little worried about your changing the version number. Is 4.12.0-0ubuntu1.2 > 4.12.0-0ubuntu2 ?

Łukasz Zemczak (sil2100) wrote :

No, but ubuntu2 never got released anywhere. It's an unused version. Here's the detailed story:

4.12.0-0ubuntu1.1 and 4.12.0-0ubuntu2 were basically the same thing. The only difference was that ubuntu1.1 was ubuntu1 + quilt patch of ubuntu2 bugfix. But ubuntu1.1 was not versioned anywhere (no bzr branch for that) - since it's just ubuntu1 with this one quilt patch added. ubuntu2, on the other hand, is the very same patch but merged into the source tree directly. And it was not released anywhere, just pushed to the branch. Since this provides a 'conflict' (since ugh, two distinct version numbers with the same fix? That's wrong) me and Didier decided that ubuntu2 is not the right way of versioning.

And since ubuntu2 is not packaged, well, we decided to get rid of it and exchange it to ubuntu1.1 and add our fix as ubuntu1.2.

Gerry Boland (gerboland) wrote :

Ok, that makes sense. Approving! Many thanks

review: Approve

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
=== modified file 'debian/changelog'
--- debian/changelog 2011-11-23 14:07:56 +0000
+++ debian/changelog 2012-06-13 17:25:45 +0000
@@ -1,4 +1,10 @@
1unity-2d (4.12.0-0ubuntu2) precise; urgency=low1unity-2d (4.12.0-0ubuntu1.2) oneiric-security; urgency=low
3 * Fix issue with dropdown boxes on sites stop working (LP: #1010466)
5 -- Łukasz 'sil2100' Zemczak <> Wed, 13 Jun 2012 18:36:00 +0200
7unity-2d (4.12.0-0ubuntu1.1) oneiric-updates; urgency=low
3 * Enable accessibility at startup, disable it for started applications9 * Enable accessibility at startup, disable it for started applications
4 (LP: #877358)10 (LP: #877358)
=== modified file 'panel/applets/appname/windowhelper.cpp'
--- panel/applets/appname/windowhelper.cpp 2011-09-27 11:57:06 +0000
+++ panel/applets/appname/windowhelper.cpp 2012-06-13 17:25:45 +0000
@@ -149,8 +149,7 @@
150void WindowHelper::close()150void WindowHelper::close()
152 guint32 timestamp = QDateTime::currentDateTime().toTime_t();152 wnck_window_close(d->m_window, QX11Info::appTime());
153 wnck_window_close(d->m_window, timestamp);
156void WindowHelper::minimize()155void WindowHelper::minimize()


People subscribed via source and target branches

to all changes: