Reverting 1a1ab844f03e due to issue 250

--HG--
branch : distribute
extra : rebase_source : 0dca5e604c96429f07c4d7786d3a0991c42c129d

3 files changed, 6 insertions, 134 deletions

diff --git a/setuptools/command/__init__.py b/setuptools/command/__init__.py
index a601e2d3..b063fa19 100644
--- a/setuptools/command/__init__.py
+++ b/setuptools/command/__init__.py
@@ -8,6 +8,10 @@ __all__ = [
 from setuptools.command import install_scripts
 import sys
 
+if sys.version>='2.5':
+    # In Python 2.5 and above, distutils includes its own upload command
+    __all__.remove('upload')
+
 from distutils.command.bdist import bdist
 
 if 'egg' not in bdist.format_commands:
diff --git a/setuptools/command/register.py b/setuptools/command/register.py
index 874401a5..3b2e0859 100755
--- a/setuptools/command/register.py
+++ b/setuptools/command/register.py
@@ -1,6 +1,4 @@
 from distutils.command.register import register as _register
-from ConfigParser import ConfigParser
-import os
 
 class register(_register):
     __doc__ = _register.__doc__
@@ -8,63 +6,5 @@ class register(_register):
     def run(self):
         # Make sure that we are using valid current name/version info
         self.run_command('egg_info')
-        self._section_name = self.repository
         _register.run(self)
 
-    def _read_pypirc(self):
-        """Reads the .pypirc file."""
-        rc = self._get_rc_file()
-        if os.path.exists(rc):
-            repository = self.repository
-            config = ConfigParser()
-            config.read(rc)
-            sections = config.sections()
-            if 'distutils' in sections:
-                # let's get the list of servers
-                index_servers = config.get('distutils', 'index-servers')
-                _servers = [server.strip() for server in
-                            index_servers.split('\n')
-                            if server.strip() != '']
-                if _servers == []:
-                    # nothing set, let's try to get the default pypi
-                    if 'pypi' in sections:
-                        _servers = ['pypi']
-                    else:
-                        # the file is not properly defined, returning
-                        # an empty dict
-                        return {}
-                for server in _servers:
-                    current = {'server': server}
-                    current['username'] = config.get(server, 'username')
-
-                    # optional params
-                    for key, default in (('repository',
-                                          None),
-                                         ('realm', self.DEFAULT_REALM),
-                                         ('password', None)):
-                        if config.has_option(server, key):
-                            current[key] = config.get(server, key)
-                        else:
-                            current[key] = default
-                    if (current['server'] == repository or
-                        current['repository'] == repository):
-                        return current
-            elif 'server-login' in sections:
-                # old format
-                server = 'server-login'
-                if config.has_option(server, 'repository'):
-                    repository = config.get(server, 'repository')
-                else:
-                    repository = None
-                return {'username': config.get(server, 'username'),
-                        'password': config.get(server, 'password'),
-                        'repository': repository,
-                        'server': server,
-                        'realm': self.DEFAULT_REALM}
-
-        return {}
-
-    def _set_config(self):
-        _register._set_config(self)
-        if self.repository is None:
-                raise ValueError('%s is missing a repository value in .pypirc' % self._section_name)
diff --git a/setuptools/command/upload.py b/setuptools/command/upload.py
index f2e0b2e1..4bd6021d 100755
--- a/setuptools/command/upload.py
+++ b/setuptools/command/upload.py
@@ -2,7 +2,6 @@
 
 Implements the Distutils 'upload' subcommand (upload package to PyPI)."""
 
-
 from distutils.errors import *
 from distutils.core import Command
 from distutils.spawn import spawn
@@ -12,7 +11,6 @@ try:
 except ImportError:
     from md5 import md5
 import os
-import sys
 import socket
 import platform
 import ConfigParser
@@ -21,7 +19,7 @@ import base64
 import urlparse
 import cStringIO as StringIO
 
-class _upload(Command):
+class upload(Command):
 
     description = "upload binary package to PyPI"
 
@@ -67,7 +65,7 @@ class _upload(Command):
                 if not self.password:
                     self.password = config.get('server-login', 'password')
         if not self.repository:
-            raise ValueError('%s is missing a repository value in .pypirc' % self._section_name)
+            self.repository = self.DEFAULT_REPOSITORY
 
     def run(self):
         if not self.distribution.dist_files:
@@ -184,73 +182,3 @@ class _upload(Command):
         if self.show_response:
             print '-'*75, r.read(), '-'*75
 
-if sys.version >= "2.5":
-    from distutils.command.upload import upload as distutils_upload
-    class upload(distutils_upload):
-
-        def run(self):
-            self._section_name = self.repository
-            distutils_upload.run(self)
-
-        def _read_pypirc(self):
-            """Reads the .pypirc file."""
-            self._section_name = self.repository
-            rc = self._get_rc_file()
-            if os.path.exists(rc):
-                repository = self.repository
-                config = ConfigParser.ConfigParser()
-                config.read(rc)
-                sections = config.sections()
-                if 'distutils' in sections:
-                    # let's get the list of servers
-                    index_servers = config.get('distutils', 'index-servers')
-                    _servers = [server.strip() for server in
-                                index_servers.split('\n')
-                                if server.strip() != '']
-                    if _servers == []:
-                        # nothing set, let's try to get the default pypi
-                        if 'pypi' in sections:
-                            _servers = ['pypi']
-                        else:
-                            # the file is not properly defined, returning
-                            # an empty dict
-                            return {}
-                    for server in _servers:
-                        current = {'server': server}
-                        current['username'] = config.get(server, 'username')
-
-                        # optional params
-                        for key, default in (('repository',
-                                              None),
-                                             ('realm', self.DEFAULT_REALM),
-                                             ('password', None)):
-                            if config.has_option(server, key):
-                                current[key] = config.get(server, key)
-                            else:
-                                current[key] = default
-                                if (current['server'] == repository:
-                            return current
-                elif 'server-login' in sections:
-                    # old format
-                    server = 'server-login'
-                    if config.has_option(server, 'repository'):
-                        repository = config.get(server, 'repository')
-                    else:
-                        repository = None
-                    return {'username': config.get(server, 'username'),
-                            'password': config.get(server, 'password'),
-                            'repository': repository,
-                            'server': server,
-                            'realm': self.DEFAULT_REALM}
-
-            return {}
-
-        def finalize_options(self):
-            distutils_upload.finalize_options(self)
-            if not self.repository:
-                raise ValueError('%s is missing a repository value in .pypirc' % self._section_name)
-
-
-else:
-    upload = _upload
-