[Merge] lp:~gandelman-a/ubuntu/lucid/facter/lp732953_876130 into lp:ubuntu/lucid-updates/facter

Adam Gandelman adam.gandelman at canonical.com
Tue Oct 25 17:41:33 UTC 2011


Adam Gandelman has proposed merging lp:~gandelman-a/ubuntu/lucid/facter/lp732953_876130 into lp:ubuntu/lucid-updates/facter.

Requested reviews:
  Ubuntu Development Team (ubuntu-dev)
Related bugs:
  Bug #732953 in facter (Ubuntu Lucid): "can_connect function inside ec2.rb always return false"
  https://bugs.launchpad.net/ubuntu/lucid/+source/facter/+bug/732953
  Bug #876130 in facter (Ubuntu Lucid): "Incorrect operatingsystem reported in puppet run"
  https://bugs.launchpad.net/ubuntu/lucid/+source/facter/+bug/876130

For more details, see:
https://code.launchpad.net/~gandelman-a/ubuntu/lucid/facter/lp732953_876130/+merge/80366

Rebased a previous MP on lucid-updates. Properly versioned and applied patches as per feedback from James Page on original MP:

A few comments that need resolution before we can upload:

1) The branch is currently based on 1.5.6-2ubuntu2 from the lucid release pocket; it needs to be based on 1.5.6-2ubuntu2.1 from lucid-updates - this is why the merge shows a conflict in debian/changelog

2) This package in Lucid uses an earlier version of debhelper and is source format 1.0; so patches don't get automatically applied.  The correct course of action (as one patch has already be directly applied in 1.5.6-2ubuntu2.1) is to directly apply the patches to the upstream source tree.  Ugly but its the most minimal change for the SRU and does not impact the update that has already been made
-- 
https://code.launchpad.net/~gandelman-a/ubuntu/lucid/facter/lp732953_876130/+merge/80366
Your team Ubuntu Development Team is requested to review the proposed merge of lp:~gandelman-a/ubuntu/lucid/facter/lp732953_876130 into lp:ubuntu/lucid-updates/facter.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: review-diff.txt
Type: text/x-diff
Size: 1646 bytes
Desc: not available
URL: <https://lists.ubuntu.com/archives/ubuntu-reviews/attachments/20111025/6f7d577c/attachment.diff>


More information about the Ubuntu-reviews mailing list