[merge] NewPack constructor takes a PackCollection rather than a bunch of it's attributes

[John Arbash Meinel]

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA1

Martin Pool wrote:
> Just a refactoring on the way to http://launchpad.net/bugs/288751/
> 
> This is api-incompatible but it's a very internal class.
> 
> 

BB:tweak

This looks good to me, though Robert is the only person who might be
using NewPack outside of bzrlib, so we should probably check to see if
the api break is reasonable for him.

John
=:->
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (Cygwin)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEARECAAYFAkkan2sACgkQJdeBCYSNAANIhwCfar7CUqctkQka8bUtoxHpLAXu
ETQAoLZWVLMN7Yug8pTtxDLHfeGUTQNp
=I5/f
-----END PGP SIGNATURE-----