Merge lp://staging/~deejay1/groundcontrol/fix-for-515303 into lp://staging/groundcontrol

Proposed by Łukasz Jernaś
Status: Merged
Merged at revision: not available
Proposed branch: lp://staging/~deejay1/groundcontrol/fix-for-515303
Merge into: lp://staging/groundcontrol
Diff against target: 12 lines (+2/-1)
1 file modified
groundcontrol.desktop.in (+2/-1)
To merge this branch: bzr merge lp://staging/~deejay1/groundcontrol/fix-for-515303
Reviewer Review Type Date Requested Status
Martin Owens Approve
Review via email: mp+18789@code.staging.launchpad.net
To post a comment you must log in.
Revision history for this message
Łukasz Jernaś (deejay1) wrote :

Marked as "fix committed" but not merged into main repo

Revision history for this message
Martin Owens (doctormo) wrote :

Now it's merged in, technically there was a fix committed ;-)

review: Approve
Revision history for this message
Łukasz Jernaś (deejay1) wrote :

Technically yes, but IMHO let's set statuses to "Fix committed" when it's in the release tree and "Fixed released" when a tarball is out. I know that it seems like a traditional VCS approach, but I think it allows to better keep track on which bugfixes are already in and which not.

Revision history for this message
Martin Owens (doctormo) wrote :

I agree and that's how I'm doing it.

Preview Diff

[H/L] Next/Prev Comment, [J/K] Next/Prev File, [N/P] Next/Prev Hunk
1=== modified file 'groundcontrol.desktop.in'
2--- groundcontrol.desktop.in 2010-02-05 10:05:23 +0000
3+++ groundcontrol.desktop.in 2010-02-07 09:28:13 +0000
4@@ -7,6 +7,7 @@
5 StartupNotify=false
6 Terminal=false
7 Type=Application
8-Categories=GNOME;GTK;Utility;X-OnlineServices;
9+Categories=GNOME;GTK;Settings;X-GNOME-PersonalSettings;
10+OnlyShowIn=GNOME;
11 X-Ubuntu-Gettext-Domain=groundcontrol
12 X-AppInstall-Package=groundcontrol

Subscribers

People subscribed via source and target branches