Merge lp:~ivoks/ubuntu/maverick/glib2.0/887946 into lp:ubuntu/maverick-proposed/glib2.0
Proposed by
Ante Karamatić
Status: | Rejected |
---|---|
Rejected by: | Sebastien Bacher |
Proposed branch: | lp:~ivoks/ubuntu/maverick/glib2.0/887946 |
Merge into: | lp:ubuntu/maverick-proposed/glib2.0 |
Diff against target: |
61 lines (+41/-0) 3 files modified
debian/changelog (+7/-0) debian/patches/90-context-unlock.patch (+33/-0) debian/patches/series (+1/-0) |
To merge this branch: | bzr merge lp:~ivoks/ubuntu/maverick/glib2.0/887946 |
Related bugs: |
Reviewer | Review Type | Date Requested | Status |
---|---|---|---|
Stéphane Graber | Approve | ||
Ante Karamatić | Needs Resubmitting | ||
Dave Walker (community) | Needs Fixing | ||
Ubuntu Sponsors | Pending | ||
Review via email: mp+81721@code.launchpad.net |
Description of the change
gmain: move finalization of GSource outside of context lock
(LP: #887946)
To post a comment you must log in.
Unmerged revisions
- 92. By Ante Karamatić
-
Fix debian/changelog
- 91. By Ante Karamatić
-
Add patch debian/
patch90- context- unlock. patch from upstream commit 1f853c75ccb0a42 757c3661ed36427 e69e416af3
(LP: #887946) - 90. By Ante Karamatić
-
Revert changes on glib/gmain.c
- 89. By Ante Karamatić
-
gmain: move finalization of GSource outside of context lock
(LP: #887946)
Hey Ante,
Thanks for the merge proposal. I did have a comment: dep.debian. net/deps/ dep3/)?
- It seems to edit files directly, when the package already makes use of a patching system. Would you be able to convert this to a standalone patch (using DEP-3 headers, http://
Thanks.