diff --git a/master/buildbot/steps/source/oldsource.py b/master/buildbot/steps/source/oldsource.py
index 2c42d94ebc2..4bfe645fc97 100644
--- a/master/buildbot/steps/source/oldsource.py
+++ b/master/buildbot/steps/source/oldsource.py
@@ -172,11 +172,6 @@ def __init__(self, bkurl=None, baseURL=None,
         self.extra_args = extra_args
 
         Source.__init__(self, **kwargs)
-        self.addFactoryArguments(bkurl=bkurl,
-                                 baseURL=baseURL,
-                                 directory=directory,
-                                 extra_args=extra_args,
-                                 )
 
         if bkurl and baseURL:
             raise ValueError("you must use exactly one of bkurl and baseURL")
diff --git a/master/buildbot/steps/source/repo.py b/master/buildbot/steps/source/repo.py
index 26ff0af9e43..29dd41a5be1 100644
--- a/master/buildbot/steps/source/repo.py
+++ b/master/buildbot/steps/source/repo.py
@@ -94,13 +94,6 @@ def copy_callable(param_name,f):
         Source.__init__(self, **kwargs)
 
         assert self.manifest_url is not None
-        self.addFactoryArguments(manifest_url=manifest_url,
-                                 manifest_branch=manifest_branch,
-                                 manifest_file=manifest_file,
-                                 tarball=tarball,
-                                 sync_all_branches=sync_all_branches,
-                                 update_tarball=update_tarball,
-                                 )
 
     def computeSourceRevision(self, changes):
         if not changes: