Merge lp://qastaging/~dannf/ubuntu/saucy/udisks/lp1235051 into lp://qastaging/ubuntu/saucy/udisks

Proposed by dann frazier
Status: Merged
Merge reported by: Sebastien Bacher
Merged at revision: not available
Proposed branch: lp://qastaging/~dannf/ubuntu/saucy/udisks/lp1235051
Merge into: lp://qastaging/ubuntu/saucy/udisks
Diff against target: 45 lines (+11/-2)
3 files modified
debian/changelog (+7/-0)
debian/control (+3/-1)
debian/rules (+1/-1)
To merge this branch: bzr merge lp://qastaging/~dannf/ubuntu/saucy/udisks/lp1235051
Reviewer Review Type Date Requested Status
Sebastien Bacher Approve
Review via email: mp+189203@code.qastaging.launchpad.net
To post a comment you must log in.
Revision history for this message
Jackson Doak (noskcaj) wrote :

Please check that this doesn't break rebuilding by running "debuild && debuild".
If that won't work, you'll need to add debian/source/options with a diff ignore

Revision history for this message
Colin Watson (cjwatson) wrote :

On Fri, Oct 04, 2013 at 02:26:19AM -0000, Jackson Doak wrote:
> Please check that this doesn't break rebuilding by running "debuild && debuild".
> If that won't work, you'll need to add debian/source/options with a diff ignore

dh_autoreconf is no more likely to cause that problem than any other
change; it takes special care to avoid this.

Revision history for this message
dann frazier (dannf) wrote :

On Fri, Oct 4, 2013 at 2:42 AM, Colin Watson <email address hidden> wrote:
> On Fri, Oct 04, 2013 at 02:26:19AM -0000, Jackson Doak wrote:
>> Please check that this doesn't break rebuilding by running "debuild && debuild".
>> If that won't work, you'll need to add debian/source/options with a diff ignore
>
> dh_autoreconf is no more likely to cause that problem than any other
> change; it takes special care to avoid this.

I've confirmed by testing it.

Revision history for this message
Sebastien Bacher (seb128) wrote :

Thanks, we are currently in sync with Debian and it would be nice to stay that way. I don't think this fix is needed for saucy, so let's delay to next cycle and upload then if Debian still didn't get the change in. Setting as "work in progress" to get it out of the sponsoring queue meanwhile, since staled entry makes the queue a bit harder to review

Revision history for this message
Sebastien Bacher (seb128) wrote :

Ok, it seems having those in saucy helps, let's upload, thanks for the work!

review: Approve

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: