Update for fabric3.0

Author: LecrisUT

pyproject.toml

@@ -0,0 +1,62 @@
+[build-system]
+requires = ["setuptools"]
+build-backend = "setuptools.build_meta"
+
+[project]
+name = "patchwork"
+version = "2.0.0"
+description = "Deployment/sysadmin operations, powered by Fabric"
+authors = [{ name = "Jeff Forcier", email = "jeff@bitprophet.org" }]
+maintainers = [{ name = "Jeff Forcier", email = "jeff@bitprophet.org" }]
+requires-python = ">=3.6.2"
+readme = "README.rst"
+license = { file = "LICENSE" }
+classifiers = [
+    "Development Status :: 5 - Production/Stable",
+    "Environment :: Console",
+    "Intended Audience :: Developers",
+    "Intended Audience :: System Administrators",
+    "License :: OSI Approved :: BSD License",
+    "Operating System :: POSIX",
+    "Operating System :: Unix",
+    "Operating System :: MacOS :: MacOS X",
+    "Operating System :: Microsoft :: Windows",
+    "Programming Language :: Python",
+    "Programming Language :: Python :: 2",
+    "Programming Language :: Python :: 2.7",
+    "Programming Language :: Python :: 3",
+    "Programming Language :: Python :: 3.4",
+    "Programming Language :: Python :: 3.5",
+    "Programming Language :: Python :: 3.6",
+    "Topic :: Software Development",
+    "Topic :: Software Development :: Build Tools",
+    "Topic :: Software Development :: Libraries",
+    "Topic :: Software Development :: Libraries :: Python Modules",
+    "Topic :: System :: Software Distribution",
+    "Topic :: System :: Systems Administration",
+]
+
+dependencies = [
+    "fabric>=3.0",
+]
+
+[project.optional-dependencies]
+dev = [
+]
+doc = [
+    "releases",
+    "alabaster",
+    "sphinx",
+]
+test = [
+    "invocations",
+    "pytest",
+    "pytest-cov",
+    "mock",
+]
+
+[project.urls]
+homepage = "https://www.fabfile.org/"
+
+[tool.pytest.ini_options]
+testpaths = ["tests"]

setup.py

@@ -7,4 +7,4 @@ def have_program(c, name):
     """
     Returns whether connected user has program ``name`` in their ``$PATH``.
     """
-    return c.run("which {}".format(name), hide=True, warn=True)
+    return c.run(f"which {name}", hide=True, warn=True)

patchwork/__init__.py src/__init__.pypatchwork/__init__.py renamed to src/__init__.py

@@ -4,8 +4,6 @@
 
 import re
 
-from invoke.vendor import six
-
 from .util import set_runner
 
 
@@ -25,12 +23,12 @@ def directory(c, runner, path, user=None, group=None, mode=None):
     :param str mode:
         ``chmod`` compatible mode string to apply to the directory.
     """
-    runner("mkdir -p {}".format(path))
+    runner(f"mkdir -p {path}")
     if user is not None:
         group = group or user
-        runner("chown {}:{} {}".format(user, group, path))
+        runner(f"chown {user}:{group} {path}")
     if mode is not None:
-        runner("chmod {} {}".format(mode, path))
+        runner(f"chmod {mode} {path}")
 
 
 @set_runner
@@ -78,8 +76,8 @@ def contains(c, runner, filename, text, exact=False, escape=True):
     if escape:
         text = _escape_for_regex(text)
         if exact:
-            text = "^{}$".format(text)
-    egrep_cmd = 'egrep "{}" "{}"'.format(text, filename)
+            text = f"^{text}$"
+    egrep_cmd = f'egrep "{text}" "{filename}"'
     return runner(egrep_cmd, hide=True, warn=True).ok
 
 
@@ -116,7 +114,7 @@ def append(c, runner, filename, text, partial=False, escape=True):
         Whether to perform regex-oriented escaping on ``text``.
     """
     # Normalize non-list input to be a list
-    if isinstance(text, six.string_types):
+    if isinstance(text, str):
         text = [text]
     for line in text:
         regex = "^" + _escape_for_regex(line) + ("" if partial else "$")
@@ -127,7 +125,7 @@ def append(c, runner, filename, text, partial=False, escape=True):
         ):
             continue
         line = line.replace("'", r"'\\''") if escape else line
-        runner("echo '{}' >> {}".format(line, filename))
+        runner(f"echo '{line}' >> {filename}")
 
 
 def _escape_for_regex(text):

src/patchwork/__init__.py

@@ -29,7 +29,7 @@ def distro_name(c):
     }
     for name, sentinels in sentinel_files.items():
         for sentinel in sentinels:
-            if exists(c, "/etc/{}".format(sentinel)):
+            if exists(c, f"/etc/{sentinel}"):
                 return name
     return "other"
 

patchwork/_version.py src/patchwork/_version.pypatchwork/_version.py renamed to src/patchwork/_version.py

@@ -6,7 +6,7 @@
 # apt/deb, rpm/yum/dnf, arch/pacman, etc etc etc.
 
 
-from patchwork.info import distro_family
+from ..info import distro_family
 
 
 def package(c, *packages):
@@ -29,4 +29,4 @@ def rubygem(c, gem):
     """
     Install a Ruby gem.
     """
-    return c.sudo("gem install -b --no-rdoc --no-ri {}".format(gem))
+    return c.sudo(f"gem install -b --no-rdoc --no-ri {gem}")

patchwork/environment.py src/patchwork/environment.pypatchwork/environment.py renamed to src/patchwork/environment.py

@@ -2,8 +2,6 @@
 File transfer functionality above and beyond basic ``put``/``get``.
 """
 
-from invoke.vendor import six
-
 
 def rsync(
     c,
@@ -79,7 +77,7 @@ def rsync(
         (rsync's ``--rsh`` flag.)
     """
     # Turn single-string exclude into a one-item list for consistency
-    if isinstance(exclude, six.string_types):
+    if isinstance(exclude, str):
         exclude = [exclude]
     # Create --exclude options from exclude list
     exclude_opts = ' --exclude "{}"' * len(exclude)
@@ -97,30 +95,30 @@ def rsync(
     # always-a-list, always-up-to-date-from-all-sources attribute to save us
     # from having to do this sort of thing. (may want to wait for Paramiko auth
     # overhaul tho!)
-    if isinstance(keys, six.string_types):
+    if isinstance(keys, str):
         keys = [keys]
     if keys:
         key_string = "-i " + " -i ".join(keys)
     # Get base cxn params
     user, host, port = c.user, c.host, c.port
-    port_string = "-p {}".format(port)
+    port_string = f"-p {port}"
     # Remote shell (SSH) options
     rsh_string = ""
     # Strict host key checking
     disable_keys = "-o StrictHostKeyChecking=no"
     if not strict_host_keys and disable_keys not in ssh_opts:
-        ssh_opts += " {}".format(disable_keys)
+        ssh_opts += f" {disable_keys}"
     rsh_parts = [key_string, port_string, ssh_opts]
     if any(rsh_parts):
-        rsh_string = "--rsh='ssh {}'".format(" ".join(rsh_parts))
+        rsh_string = f"--rsh='ssh {' '.join(rsh_parts)}'"
     # Set up options part of string
     options_map = {
         "delete": "--delete" if delete else "",
         "exclude": exclude_opts.format(*exclusions),
         "rsh": rsh_string,
         "extra": rsync_opts,
     }
-    options = "{delete}{exclude} -pthrvz {extra} {rsh}".format(**options_map)
+    options = f"{options_map['delete']}{options_map['exclude']} -pthrvz {options_map['extra']} {options_map['rsh']}"
     # Create and run final command string
     # TODO: richer host object exposing stuff like .address_is_ipv6 or whatever
     if host.count(":") > 1: