Merge lp://qastaging/~yannick-gicquel/ubuntu/raring/gnome-panel/fix-for-1080375 into lp://qastaging/ubuntu/raring/gnome-panel

Proposed by Y.Gicquel
Status: Merged
Merged at revision: 232
Proposed branch: lp://qastaging/~yannick-gicquel/ubuntu/raring/gnome-panel/fix-for-1080375
Merge into: lp://qastaging/ubuntu/raring/gnome-panel
Diff against target: 44 lines (+24/-0)
3 files modified
debian/changelog (+7/-0)
debian/patches/90-remove_artifact_on_icon_animation.patch (+16/-0)
debian/patches/series (+1/-0)
To merge this branch: bzr merge lp://qastaging/~yannick-gicquel/ubuntu/raring/gnome-panel/fix-for-1080375
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Ubuntu branches Pending
Review via email: mp+134808@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Sebastien Bacher (seb128) wrote :

Thank for your work, it would be nice if you could include the bug informations in the patch description next time, at least a description and replace "Bug: bug URL" to include the bug references, I see that you opened https://bugzilla.gnome.org/show_bug.cgi?id=688618 for that one

review: Approve
Revision history for this message
Y.Gicquel (yannick-gicquel) wrote :

Hi Sebastien,
Ok, sorry for the mistake... i often miss some field to fill with all this
hard process of delivery... I'll be careful next time :-)
Thanks,

2012/11/19 Sebastien Bacher <email address hidden>

> Review: Approve
>
> Thank for your work, it would be nice if you could include the bug
> informations in the patch description next time, at least a description and
> replace "Bug: bug URL" to include the bug references, I see that you opened
> https://bugzilla.gnome.org/show_bug.cgi?id=688618 for that one
> --
>
> https://code.launchpad.net/~yannick-gicquel/ubuntu/raring/gnome-panel/fix-for-1080375/+merge/134808
> You are the owner of
> lp:~yannick-gicquel/ubuntu/raring/gnome-panel/fix-for-1080375.
>

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: