summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
Diffstat (limited to '')
-rwxr-xr-xsrc/vgstash_cli.py27
1 files changed, 22 insertions, 5 deletions
diff --git a/src/vgstash_cli.py b/src/vgstash_cli.py
index d00f339..1967458 100755
--- a/src/vgstash_cli.py
+++ b/src/vgstash_cli.py
@@ -194,18 +194,35 @@ def notes(title, system, edit):
with tempfile.NamedTemporaryFile() as tmpfile:
tmpfile.write(target_game.notes.encode("UTF-8"))
tmpfile.flush()
- process = subprocess.run([os.getenv("EDITOR", "vim"), tmpfile.name])
+ pre_stat = os.stat(tmpfile.name)
+ o_mtime = pre_stat.st_mtime
+ o_size = pre_stat.st_size
+ try:
+ process = subprocess.run([os.getenv("EDITOR", "vim"), tmpfile.name])
+ except:
+ click.echo("Could not run editor. Is it set correctly?")
+ return
tmpfile.flush()
tmpfile.seek(0)
+ post_stat = os.stat(tmpfile.name)
+ n_mtime = post_stat.st_mtime
+ n_size = post_stat.st_size
note_arr = []
for line in tmpfile:
note_arr.append(line.decode("UTF-8").rstrip("\r\n"))
target_game.notes = "\n".join(note_arr)
db.update_game(target_game, target_game)
- if process.returncode == 0:
- click.echo("Notes for {} on {} have been updated!".format(target_game.title, target_game.system))
- else:
- click.echo("Couldn't find an editor for notes. Check the EDITOR environment variable and try again.")
+ if process.returncode == 0:
+ # determine what actually happened
+ if o_mtime == n_mtime:
+ if o_size == n_size:
+ click.echo("Notes for {} on {} left unchanged.".format(target_game.title, target_game.system))
+ elif n_mtime > o_mtime and o_size != n_size:
+ click.echo("Notes for {} on {} have been updated!".format(target_game.title, target_game.system))
+ return
+ else:
+ click.echo("The editor crashed. Please try again.")
+ return
else:
if len(target_game.notes) > 0:
click.echo("Notes for {} on {}:".format(target_game.title, target_game.system))
pan='5' class='logmsg'> This filter is used to get an idea of which games you're currently playing through, so you can prioritize games to play when you're bored and detect it when you've beaten a game but haven't marked it as such. 2018-03-13Update helpers a bitzlg1-2/+9 At present, user modification is needed to make these seamless. vgup() may need to be axed in favor of telling the user to make an alias. 2018-03-13Make VGSTASH_DB_LOCATION point to a filezlg2-21/+20 It used to point to a directory, which would then look for .vgstash.db. This behavior was kind of backwards and I don't remember why I did it that way. This change gives users more control over where they put their DB. Be sure to update your environment variable if you have it set! 2016-11-18Remove settings from helpers.shZe Libertine Gamer1-5/+0 Sourcing them in .bash_profile screws up login if they're set. 2016-11-15Correct phrasing in README.Ze Libertine Gamer1-4/+4 2016-11-13DerpZe Libertine Gamer1-0/+1 2016-11-03Improve error handling in shell scriptsZe Libertine Gamer4-3/+23 2016-10-24Correct run_again, add recursionZe Libertine Gamer1-0/+4 Loops and functions -- oh my, what a useful combination. :) 2016-10-21Add quotes to correct behavior for arglistZe Libertine Gamer1-1/+1 2016-10-14updater.sh: add recursion, error handlingZe Libertine Gamer1-43/+101 2016-10-14Correct pipe-handling behaviorZe Libertine Gamer1-1/+9 2016-10-12Clarify a method to move between platformsZe Libertine Gamer1-2/+5 Also correct a typo.