bundle2: equate 'UN' with no compression
authorGregory Szorc <gregory.szorc@gmail.com>
Thu, 10 Nov 2016 23:29:01 -0800
changeset 30439 71b368e3b590
parent 30438 90933e4e44fd
child 30440 c3944ab1443a
bundle2: equate 'UN' with no compression An upcoming patch will change the "alg" argument passed to this function from None to "UN" when no compression is wanted. The existing implementation of bundle2 does not set a "Compression" parameter if no compression is used. In theory, setting "Compression=UN" should work. But I haven't audited the code to see if all client versions supporting bundle2 will accept this. Rather than take the risk, avoid the BC breakage and treat "UN" the same as None.
mercurial/bundle2.py
--- a/mercurial/bundle2.py	Thu Nov 10 23:15:02 2016 -0800
+++ b/mercurial/bundle2.py	Thu Nov 10 23:29:01 2016 -0800
@@ -515,7 +515,7 @@
 
     def setcompression(self, alg):
         """setup core part compression to <alg>"""
-        if alg is None:
+        if alg in (None, 'UN'):
             return
         assert not any(n.lower() == 'Compression' for n, v in self._params)
         self.addparam('Compression', alg)