Re: [ubuntu/saucy-proposed] i7z 0.27.2-2ubuntu1 (Accepted)

2013-08-15 Thread Micah Gersten

-BEGIN PGP SIGNED MESSAGE-
Hash: SHA1

On 08/15/2013 09:58 AM, Jackson Doak wrote:
 i7z (0.27.2-2ubuntu1) saucy; urgency=low

   * Merge from debian unstable. Remaining changes:
 - debian/control: Future-proof by adding x32 to the architecture list

 Date: Thu, 15 Aug 2013 07:44:30 +1000
 Changed-By: Jackson Doak nosk...@ubuntu.com
 Maintainer: Ubuntu Developers ubuntu-devel-disc...@lists.ubuntu.com
 Signed-By: Matthew Fischer matthew.fisc...@canonical.com
 https://launchpad.net/ubuntu/saucy/+source/i7z/0.27.2-2ubuntu1


This could've just been a sync, the changes were in Debian.  Also,
please use -v with the old package version when uploading a merge.

Thanks,
Micah
-BEGIN PGP SIGNATURE-
Version: GnuPG v1.4.11 (GNU/Linux)

iEYEARECAAYFAlIM9JkACgkQTniv4aqX/VnOqQCdFdbC8QADjNk97+FcJ8ubL4Xg
wV0AnA7UMHJVR9UCfV6wzR8+G9eCPHwm
=r46S
-END PGP SIGNATURE-

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: [ubuntu/saucy-proposed] i7z 0.27.2-2ubuntu1 (Accepted)

2013-08-15 Thread Adam Conrad
On Thu, Aug 15, 2013 at 10:32:52AM -0500, Micah Gersten wrote:
 
 On 08/15/2013 09:58 AM, Jackson Doak wrote:
  i7z (0.27.2-2ubuntu1) saucy; urgency=low
 
* Merge from debian unstable. Remaining changes:
  - debian/control: Future-proof by adding x32 to the architecture list
 
 This could've just been a sync, the changes were in Debian.  Also,
 please use -v with the old package version when uploading a merge.

You also refreshed the quilt patches for no discernable reason, which
would make the next merge have a pointless delta to apply.  Good thing
that, in this case, there doesn't need to be a merge at all. ;)

... Adam

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: [ubuntu/saucy-proposed] i7z 0.27.2-2ubuntu1 (Accepted)

2013-08-15 Thread Adam Conrad
On Fri, Aug 16, 2013 at 11:38:19AM +1000, Jackson Doak wrote:
  I hadn't realised it's bad to refresh the patches. sorry.

Generally, anything that increases the size of the delta without
providing an obvious benefit is bad in the sense that it's one
more way the next merge becomes more work due to potential merge
conflicts, etc.

My general rule for Debian-Ubuntu deltas is to try to keep them
as small and self-contained as possible while fixing the bugs or
adding the features that we need.

... Adam

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel


Re: [ubuntu/saucy-proposed] i7z 0.27.2-2ubuntu1 (Accepted)

2013-08-15 Thread Jackson Doak
 I hadn't realised it's bad to refresh the patches. sorry.

On Fri, Aug 16, 2013 at 11:36 AM, Adam Conrad adcon...@ubuntu.com wrote:
 On Thu, Aug 15, 2013 at 10:32:52AM -0500, Micah Gersten wrote:

 On 08/15/2013 09:58 AM, Jackson Doak wrote:
  i7z (0.27.2-2ubuntu1) saucy; urgency=low
 
* Merge from debian unstable. Remaining changes:
  - debian/control: Future-proof by adding x32 to the architecture list

 This could've just been a sync, the changes were in Debian.  Also,
 please use -v with the old package version when uploading a merge.

 You also refreshed the quilt patches for no discernable reason, which
 would make the next merge have a pointless delta to apply.  Good thing
 that, in this case, there doesn't need to be a merge at all. ;)

 ... Adam

-- 
ubuntu-devel mailing list
ubuntu-devel@lists.ubuntu.com
Modify settings or unsubscribe at: 
https://lists.ubuntu.com/mailman/listinfo/ubuntu-devel