Merge lp://staging/~andy-m-caldwell/ubuntu/trusty/fuse-umfuse-ext2/fix-for-1674868 into lp://staging/ubuntu/trusty/fuse-umfuse-ext2

Proposed by Andy Caldwell
Status: Needs review
Proposed branch: lp://staging/~andy-m-caldwell/ubuntu/trusty/fuse-umfuse-ext2/fix-for-1674868
Merge into: lp://staging/ubuntu/trusty/fuse-umfuse-ext2
Diff against target: 31 lines (+8/-2)
2 files modified
debian/changelog (+6/-0)
fuse-ext2/op_link.c (+2/-2)
To merge this branch: bzr merge lp://staging/~andy-m-caldwell/ubuntu/trusty/fuse-umfuse-ext2/fix-for-1674868
Reviewer Review Type Date Requested Status
Simon Quigley Needs Resubmitting
Ubuntu branches Pending
Review via email: mp+320722@code.staging.launchpad.net

Description of the change

Correctly release locks in op_link (LP: #1674868)

To post a comment you must log in.
Revision history for this message
Simon Quigley (tsimonq2) wrote :

Hello Andy, I apologize for the delay in getting this reviewed.

Ubuntu no longer uses Bazaar for our packaging, and it would be preferred if you were to attach a debdiff (i.e. a diff similar to below) to the bug report. Once that's done, please subscribe ~ubuntu-sponsors to the bug report and someone will promptly look at it.

Here's a couple of packaging changes that I suggest you address when you do that:
 1. Please convert your change to a patch under debian/patches and add a DEP-3 header. More information is available here: https://raphaelhertzog.com/2012/08/08/how-to-use-quilt-to-manage-patches-in-debian-packages/
 2. Since this will need to be a Stable Release Update (SRU), please make sure to modify the bug report to reflect the SRU policy: https://wiki.ubuntu.com/StableReleaseUpdates
If it is a low priority change like you suggest in the changelog, we would like a justification for why you believe it is needed in Trusty.

Again, I apologize for the delay, and thank you for your contribution to Ubuntu and willingness to get this bug fix! I look forward to sponsoring your patch over on the bug report. :)

review: Needs Resubmitting

Unmerged revisions

8. By Andy Caldwell

Correctly release locks in op_link (LP: #1674868)

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: