Rev 6038: Cleanup transport deprecations, register_transport's override parameter have been ignored for years. in file:///home/vila/src/bzr/cleanup/deprecations/

[Vincent Ladeuil]

At file:///home/vila/src/bzr/cleanup/deprecations/

------------------------------------------------------------
revno: 6038
revision-id: v.ladeuil+lp at free.fr-20110715161130-yy9cr5xj8y2frx2i
parent: v.ladeuil+lp at free.fr-20110715160931-d40sr1hbb1tj57q9
committer: Vincent Ladeuil <v.ladeuil+lp at free.fr>
branch nick: deprecations
timestamp: Fri 2011-07-15 18:11:30 +0200
message:
  Cleanup transport deprecations, register_transport's override parameter have been ignored for years.
-------------- next part --------------
=== modified file 'bzrlib/transport/__init__.py'
--- a/bzrlib/transport/__init__.py	2011-05-18 19:22:46 +0000
+++ b/bzrlib/transport/__init__.py	2011-07-15 16:11:30 +0000
@@ -138,10 +138,11 @@
 def register_lazy_transport(prefix, module, classname):
     if not prefix in transport_list_registry:
         register_transport_proto(prefix)
-    transport_list_registry.register_lazy_transport_provider(prefix, module, classname)
-
-
-def register_transport(prefix, klass, override=DEPRECATED_PARAMETER):
+    transport_list_registry.register_lazy_transport_provider(
+        prefix, module, classname)
+
+
+def register_transport(prefix, klass):
     if not prefix in transport_list_registry:
         register_transport_proto(prefix)
     transport_list_registry.register_transport_provider(prefix, klass)

=== modified file 'bzrlib/transport/local.py'
--- a/bzrlib/transport/local.py	2011-04-07 10:36:24 +0000
+++ b/bzrlib/transport/local.py	2011-07-15 16:11:30 +0000
@@ -50,15 +50,7 @@
     def __init__(self, base):
         """Set the base path where files will be stored."""
         if not base.startswith('file://'):
-            symbol_versioning.warn(
-                "Instantiating LocalTransport with a filesystem path"
-                " is deprecated as of bzr 0.8."
-                " Please use bzrlib.transport.get_transport()"
-                " or pass in a file:// url.",
-                 DeprecationWarning,
-                 stacklevel=2
-                 )
-            base = urlutils.local_path_to_url(base)
+            raise AssertionError("not a file:// url: %r" % base)
         if base[-1] != '/':
             base = base + '/'