Cleanup to get rid of scripts directory

letsencrypt/renewer.py

@@ -2,11 +2,6 @@
 certs within lineages of successor certificates, according to
 configuration."""
 
-# os.path.islink
-# os.readlink
-# os.path.dirname / os.path.basename
-# os.path.join
-
 # TODO: sanity checking consistency, validity, freshness?
 
 # TODO: call new installer API to restart servers after deployment
@@ -87,6 +82,11 @@ def renew(cert, old_version):
 
 def main(config=DEFAULTS):
     """main function for autorenewer script."""
+    # TODO: Distinguish automated invocation from manual invocation,
+    #       perhaps by looking at sys.argv[0] and inhibiting automated
+    #       invocations if /etc/letsencrypt/renewal.conf defaults have
+    #       turned it off. (The boolean parameter should probably be
+    #       called renewer_enabled.)
     for i in os.listdir(config["renewal_configs_dir"]):
         print "Processing", i
         if not i.endswith(".conf"):

letsencrypt/scripts/renewer.py

@@ -1,20 +0,0 @@
-#!/usr/bin/env python
-
-"""Let's Encrypt certificate renewer command-line / cron script."""
-
-import configobj
-
-from letsencrypt.client import renewer
-
-DEFAULTS = configobj.ConfigObj("renewal.conf")
-DEFAULTS["renewal_configs_dir"] = "/tmp/etc/letsencrypt/configs"
-DEFAULTS["official_archive_dir"] = "/tmp/etc/letsencrypt/archive"
-DEFAULTS["live_dir"] = "/tmp/etc/letsencrypt/live"
-
-if __name__ == 'main':
-    if ("renewer_enabled" in DEFAULTS
-            and not DEFAULTS.as_bool("renewer_enabled")):
-        print "Renewer is disabled by configuration!  Exiting."
-        raise SystemExit
-    else:
-        renewer.main()

setup.py

@@ -115,6 +115,7 @@ setup(
     entry_points={
         'console_scripts': [
             'letsencrypt = letsencrypt.cli:main',
+            'letsencrypt-renewer = letsencrypt.renewer:main',
             'jws = letsencrypt.acme.jose.jws:CLI.run',
         ],
         'letsencrypt.plugins': [