[133098] contrib/pypi2port/pypi2port.py

gaurav at macports.org gaurav at macports.org
Fri Feb 20 02:56:41 PST 2015


Revision: 133098
          https://trac.macports.org/changeset/133098
Author:   gaurav at macports.org
Date:     2015-02-20 02:56:41 -0800 (Fri, 20 Feb 2015)
Log Message:
-----------
Converted from python 2 to python 3

Modified Paths:
--------------
    contrib/pypi2port/pypi2port.py

Modified: contrib/pypi2port/pypi2port.py
===================================================================
--- contrib/pypi2port/pypi2port.py	2015-02-20 09:40:12 UTC (rev 133097)
+++ contrib/pypi2port/pypi2port.py	2015-02-20 10:56:41 UTC (rev 133098)
@@ -11,12 +11,12 @@
 import argparse
 import sys
 import os
-import urllib2
+import urllib.request, urllib.error, urllib.parse
 import hashlib
 import zipfile
 from progressbar import *
 try:
-    import xmlrpclib
+    import xmlrpc.client
 except ImportError:
     import xmlrpc.client as xmlrpclib
 import textwrap
@@ -29,7 +29,7 @@
 import shlex
 import getpass
 
-client = xmlrpclib.ServerProxy('http://pypi.python.org/pypi')
+client = xmlrpc.client.ServerProxy('http://pypi.python.org/pypi')
 
 
 """ Lists all packages available in pypi database """
@@ -38,7 +38,7 @@
 def list_all():
     list_packages = client.list_packages()
     for package in list_packages:
-        print package
+        print(package)
 
 
 """ Searches for a particular package by the name classifier """
@@ -47,8 +47,8 @@
 def search(pkg_name):
     values = client.search({'name': pkg_name})
     for value in values:
-        for key in value.keys():
-            print key, '-->', value[key]
+        for key in list(value.keys()):
+            print((key, '-->', value[key]))
 
 
 """ Fetches the release data for a paticular package based on
@@ -59,11 +59,11 @@
     if pkg_version:
         values = client.release_data(pkg_name, pkg_version)
         if values:
-            for key in values.keys():
-                print key, '-->', values[key]
+            for key in list(values.keys()):
+                print((key, '-->', values[key]))
         else:
-            print "No such package found."
-            print "Please specify the exact package name."
+            print("No such package found.")
+            print("Please specify the exact package name.")
         return
     return
 
@@ -72,7 +72,7 @@
 
 
 def fetch(pkg_name, dict):
-    print "Fetching distfiles..."
+    print("Fetching distfiles...")
     checksum_md5 = dict['md5_digest']
     parent_dir = './sources'
     home_dir = parent_dir + '/' + 'python'
@@ -87,7 +87,7 @@
     url = dict['url']
     file_name = src_dir + '/' + dict['filename']
 
-    u = urllib2.urlopen(url)
+    u = urllib.request.urlopen(url)
     with open(file_name, 'wb') as f:
         meta = u.info()
         file_size = int(meta.getheaders("Content-Length")[0])
@@ -113,7 +113,7 @@
 
     checksum_md5_calc = hashlib.md5(open(file_name).read()).hexdigest()
     if str(checksum_md5) == str(checksum_md5_calc):
-        print 'Successfully fetched'
+        print('Successfully fetched')
         ext = file_name.split('.')[-1]
         if ext == 'egg':
             zip = zipfile.ZipFile(file_name)
@@ -122,11 +122,11 @@
                     zip.extract(name, src_dir)
         return file_name
     else:
-        print 'Aborting due to inconsistency on checksums\n'
+        print('Aborting due to inconsistency on checksums\n')
         try:
             os.remove(file_name)
-        except OSError, e:
-            print "Error: %s - %s." % (e.filename, e.strerror)
+        except OSError as e:
+            print(("Error: %s - %s." % (e.filename, e.strerror)))
         return False
 
 
@@ -210,7 +210,7 @@
     with open(diff_file, 'w') as d:
         try:
             while 1:
-                d.write(diff_string.next())
+                d.write(next(diff_string))
         except:
             pass
 
@@ -225,7 +225,7 @@
         existing_portfile = \
             subprocess.check_output(command, stderr=subprocess.STDOUT).strip()
         return existing_portfile
-    except Exception, e:
+    except Exception as e:
         return False
 
 
@@ -235,12 +235,12 @@
 
 def checksums(pkg_name, pkg_version):
     flag = False
-    print "Attempting to fetch distfiles..."
+    print("Attempting to fetch distfiles...")
     file_name = fetch_url(pkg_name, pkg_version, True)
     if file_name:
         checksums = []
         try:
-            print "Generating checksums..."
+            print("Generating checksums...")
             command = "openssl rmd160 "+file_name
             command = command.split()
             rmd160 = subprocess.check_output(command, stderr=subprocess.STDOUT)
@@ -263,7 +263,7 @@
                 pass
             return checksums
         except:
-            print "Error\n"
+            print("Error\n")
             return
 
 
@@ -277,8 +277,8 @@
         if not r.status_code == 200:
             raise Error('No distfile')
     except:
-        print "No distfile found"
-        print "Please set a DISTFILE env var before generating the portfile"
+        print("No distfile found")
+        print("Please set a DISTFILE env var before generating the portfile")
         sys.exit(0)
 
 
@@ -316,14 +316,14 @@
 
     for phase in [port_fetch, port_checksum, port_extract, port_configure,
                   port_build, port_destroot, port_clean]:
-        print phase.__name__
+        print((phase.__name__))
         phase_output = phase(name, portv)
         if phase_output:
-            print phase.__name__ + " - SUCCESS"
+            print((phase.__name__ + " - SUCCESS"))
         else:
-            print phase.__name__ + " FAILED"
+            print((phase.__name__ + " FAILED"))
             port_clean(name, portv)
-            print "Exiting"
+            print("Exiting")
             sys.exit(1)
 
         euid = os.geteuid()
@@ -450,7 +450,7 @@
 
 def create_portfile(dict, file_name, dict2):
     search_distfile(dict['name'], dict['version'])
-    print "Creating Portfile for pypi package " + dict['name'] + "..."
+    print(("Creating Portfile for pypi package " + dict['name'] + "..."))
     with open(file_name, 'w') as file:
         file.write('# -*- coding: utf-8; mode: tcl; tab-width: 4; ')
         file.write('indent-tabs-mode: nil; c-basic-offset: 4 ')
@@ -475,8 +475,8 @@
                 file.write('maintainers         {0}\n\n'.format(
                            os.getenv('maintainer', 'nomaintainer')))
         else:
-            print "No maintainers found..."
-            print "Looking for maintainers in environment variables..."
+            print("No maintainers found...")
+            print("Looking for maintainers in environment variables...")
             file.write('maintainers         {0}\n\n'.format(
                        os.getenv('maintainer', 'nomaintainer')))
 
@@ -486,7 +486,7 @@
                              ' ', summary)
             summary = re.sub(r'\s(\s)+', ' ', summary)
             summary = summary.encode('utf-8')
-            summary = filter(lambda x: x in string.printable, summary)
+            summary = [x for x in summary if x in string.printable]
             sum_lines = textwrap.wrap(summary)
             file.write('description         ')
             for sum_line in sum_lines:
@@ -526,8 +526,8 @@
         if home_page and not home_page == 'UNKNOWN':
             file.write('homepage            {0}\n'.format(home_page))
         else:
-            print "No homepage found..."
-            print "Looking for homepage in environment variables..."
+            print("No homepage found...")
+            print("Looking for homepage in environment variables...")
             file.write('homepage            {0}\n'.format(
                        os.getenv('home_page', '')))
 
@@ -560,8 +560,8 @@
                 master_site = dict['release_url']
 #                print master_site
             else:
-                print "No master site found..."
-                print "Looking for master site in environment variables..."
+                print("No master site found...")
+                print("Looking for master site in environment variables...")
                 master_site = os.getenv('master_site', '')
         if master_site:
             file.write('master_sites        {0}\n'.format(master_site))
@@ -576,7 +576,7 @@
         file.write('distname            {0}-{1}\n\n'.format(
                    dict['name'], dict['version']))
 
-        print "Attempting to generate checksums for " + dict['name'] + "..."
+        print(("Attempting to generate checksums for " + dict['name'] + "..."))
         checksums_values = checksums(dict['name'], dict['version'])
         if checksums_values:
             file.write('checksums           rmd160  {0} \\\n'.format(
@@ -591,7 +591,7 @@
         else:
             file.write('python.versions     25 26 27 32 33 34\n\n')
 
-        print "Finding dependencies..."
+        print("Finding dependencies...")
         file.write('if {${name} ne ${subport}} {\n')
         file.write('    depends_build-append \\\n')
         file.write('                        ' +
@@ -639,22 +639,22 @@
             file.write('}\n')
         else:
             file.write('}\n')
-    print "Searching for existent port..."
+    print("Searching for existent port...")
     port_exists = search_port(dict['name'])
     if port_exists:
-        print "Creating diff..."
+        print("Creating diff...")
         old_file = port_exists
         new_file = './dports/python/py-'+dict['name']+'/Portfile'
         diff_file = './dports/python/py-'+dict['name']+'/patch.Portfile.diff'
         create_diff(old_file, new_file, diff_file)
-        print str(os.path.abspath(diff_file))+"\n"
+        print((str(os.path.abspath(diff_file))+"\n"))
 #        with open(diff_file) as diff:
 #            print diff.read()
-        print "\nIf you want to open a new ticket. Please visit"
-        print "https://trac.macports.org/auth/login/?next=/newticket"
-        print "to open a new ticket after logging in with your credentials."
+        print("\nIf you want to open a new ticket. Please visit")
+        print("https://trac.macports.org/auth/login/?next=/newticket")
+        print("to open a new ticket after logging in with your credentials.")
     else:
-        print "No port found."
+        print("No port found.")
 
 
 """ Creates the directories and other commands necessary
@@ -678,22 +678,22 @@
     if not os.path.exists(home_dir):
         os.makedirs(home_dir)
 
-    print "Attempting to fetch data from pypi..."
+    print("Attempting to fetch data from pypi...")
 
     dict = client.release_data(pkg_name, pkg_version)
     dict2 = client.release_urls(pkg_name, pkg_version)
     if dict and dict2:
-        print "Data fetched successfully."
+        print("Data fetched successfully.")
     elif dict:
-        print "Release Data fetched successfully."
+        print("Release Data fetched successfully.")
     elif dict2:
-        print "Release url fetched successfully."
+        print("Release url fetched successfully.")
     else:
-        print "No data found."
+        print("No data found.")
 
     file_name = os.path.join(home_dir, "Portfile")
     create_portfile(dict, file_name, dict2)
-    print "SUCCESS\n"
+    print("SUCCESS\n")
 
 
 """ Main function - Argument Parser """
@@ -748,7 +748,7 @@
                 pkg_version = client.package_releases(pkg_name)[0]
                 release_data(pkg_name, pkg_version)
             else:
-                print "No release found\n"
+                print("No release found\n")
         return
 
 
@@ -763,7 +763,7 @@
                 pkg_version = releases[0]
                 fetch_url(pkg_name, pkg_version)
             else:
-                print "No release found\n"
+                print("No release found\n")
         return
 
 
@@ -778,7 +778,7 @@
                 pkg_version = vers[0]
                 print_portfile(pkg_name, pkg_version)
             else:
-                print "No release found\n"
+                print("No release found\n")
         return
 
 
@@ -787,7 +787,7 @@
             pkg_name = options.package_test[0]
             port_testing(pkg_name)
         else:
-            print "No package name specified\n"
+            print("No package name specified\n")
         return
 
     parser.print_help()
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <https://lists.macosforge.org/pipermail/macports-changes/attachments/20150220/fcb319ab/attachment-0001.html>


More information about the macports-changes mailing list