From cdd255c5bbad8722c92787274e1ac038ef06c13f Mon Sep 17 00:00:00 2001
From: Luke Murphy <lukewm@riseup.net>
Date: Sat, 3 Oct 2020 13:12:56 +0200
Subject: [PATCH] Add more reasoning

---
 etherpump/commands/pull.py | 18 +++++++++++++-----
 1 file changed, 13 insertions(+), 5 deletions(-)

diff --git a/etherpump/commands/pull.py b/etherpump/commands/pull.py
index 67e2678..e49e45e 100644
--- a/etherpump/commands/pull.py
+++ b/etherpump/commands/pull.py
@@ -215,6 +215,7 @@ async def handle_pad(args, padid, data, info, session):
                 revisions = response["data"]["revisions"]
                 if meta["revisions"] == revisions and not args.force:
                     skip = True
+                    reason = "No new revisions, we already have the latest local copy"
                     break
 
             meta["padid"] = padid
@@ -234,6 +235,7 @@ async def handle_pad(args, padid, data, info, session):
 
             if (meta["revisions"] == 0) and (not args.zerorevs):
                 skip = True
+                reason = "0 revisions, this pad was never edited"
                 break
 
             # todo: load more metadata!
@@ -261,6 +263,7 @@ async def handle_pad(args, padid, data, info, session):
                     file=sys.stderr,
                 )
                 skip = True
+                reason = "PANIC, couldn't download the pad contents"
                 break
             else:
                 await trio.sleep(1)
@@ -272,10 +275,11 @@ async def handle_pad(args, padid, data, info, session):
                 file=sys.stderr,
             )
             skip = True
+            reason = "PANIC, couldn't download the pad contents"
             break
 
     if skip:
-        print("[ ] {} (skipped)".format(padid))
+        print("[ ] {} (skipped, reason: {})".format(padid, reason))
         return
 
     if args.output:
@@ -286,7 +290,7 @@ async def handle_pad(args, padid, data, info, session):
             path = trio.Path(os.path.split(metapath)[0])
             await path.mkdir()
         except OSError:
-            pass
+            print("PANIC: unable to create {}".format(str(path)))
 
     if args.all or args.text:
         url = info["localapiurl"] + "getText?" + urlencode(data)
@@ -310,7 +314,11 @@ async def handle_pad(args, padid, data, info, session):
                         p + ".meta.json",
                     )
                 )
-                print("[ ] {} (no publish)".format(padid))
+                print(
+                    "[ ] {} (deleted, reason: explicit __NOPUBLISH__)".format(
+                        padid
+                    )
+                )
                 return
 
             ##########################################
@@ -325,7 +333,7 @@ async def handle_pad(args, padid, data, info, session):
                         p + ".meta.json",
                     )
                 )
-                print("[ ] {} (publish opt out)".format(padid))
+                print("[ ] {} (deleted, reason: publish opt-out)".format(padid))
                 return
 
             ver["path"] = p + raw_ext
@@ -443,7 +451,7 @@ async def handle_pad(args, padid, data, info, session):
         async with await trio.open_file(metapath, "w") as f:
             await f.write(json.dumps(meta))
 
-    print("[x] {}".format(padid))
+    print("[x] {} (saved)".format(padid))
 
 
 async def handle_pads(args):