Pack's constructor now takes resolve_ext_ref.

dulwich/pack.py

@@ -1749,7 +1749,7 @@ def write_pack_index_v2(f, entries, pack_checksum):
 class Pack(object):
     """A Git pack object."""
 
-    def __init__(self, basename):
+    def __init__(self, basename, resolve_ext_ref=None):
         self._basename = basename
         self._data = None
         self._idx = None
@@ -1757,7 +1757,7 @@ class Pack(object):
         self._data_path = self._basename + '.pack'
         self._data_load = lambda: PackData(self._data_path)
         self._idx_load = lambda: load_pack_index(self._idx_path)
-        self.resolve_ext_ref = None
+        self.resolve_ext_ref = resolve_ext_ref
 
     @classmethod
     def from_lazy_objects(self, data_fn, idx_fn):

dulwich/tests/test_pack.py

@@ -444,7 +444,7 @@ class TestThinPack(PackTests):
                 store=self.store)
 
         # Index the new pack.
-        pack = self.make_pack(resolve_ext_ref=True)
+        pack = self.make_pack(True)
         data = PackData(pack._data_path)
         data.pack = pack
         data.create_index(self.pack_prefix + '.idx')
@@ -452,10 +452,9 @@ class TestThinPack(PackTests):
         del self.store[self.blobs['bar'].id]
 
     def make_pack(self, resolve_ext_ref):
-        pack = Pack(self.pack_prefix)
-        if resolve_ext_ref:
-            pack.resolve_ext_ref = self.store.get_raw
-        return pack
+        return Pack(
+            self.pack_prefix,
+            resolve_ext_ref=self.store.get_raw if resolve_ext_ref else None)
 
     def test_get_raw(self):
         self.assertRaises(