From 19f83a4a16e30478b891ee0a9f3d8542cc6bba8c Mon Sep 17 00:00:00 2001
From: Thomas Lake <tswsl1989@sucs.org>
Date: Tue, 16 Aug 2011 13:02:09 +0100
Subject: [PATCH] Edit scripts to move overviewer.dat to render dir
 Overviewer.dat is moved on first run and a warning is issued

---
 contrib/findSigns.py     | 22 +++++++++++++++-------
 overviewer.py            |  4 ++--
 overviewer_core/world.py | 17 +++++++++++++++--
 3 files changed, 32 insertions(+), 11 deletions(-)

diff --git a/contrib/findSigns.py b/contrib/findSigns.py
index 9e3b621..3213b35 100644
--- a/contrib/findSigns.py
+++ b/contrib/findSigns.py
@@ -5,17 +5,15 @@ This script will scan through every chunk looking for signs and write out an
 updated overviewer.dat file.  This can be useful if your overviewer.dat file
 is either out-of-date or non-existant.  
 
-To run, simply give a path to your world directory, for example:
+To run, simply give a path to your world directory and the path to your
+output directory. For example:
 
-    python contrib/findSigns.py ../world.test/
+    python contrib/findSigns.py ../world.test/ output_dir/
 
 Once that is done, simply re-run the overviewer to generate markers.js:
 
     python overviewer.py ../world.test/ output_dir/
 
-Note: if your cachedir is not the same as your world-dir, you'll need to manually
-move overviewer.dat into the correct location.
-
 '''
 import sys
 import re
@@ -23,16 +21,22 @@ import os
 import cPickle
 
 sys.path.append(".")
-import nbt
+from overviewer_core import nbt
 
 from pprint import pprint
 
 worlddir = sys.argv[1]
+outputdir = sys.argv[2]
 if os.path.exists(worlddir):
     print "Scanning chunks in ", worlddir
 else:
     sys.exit("Bad WorldDir")
 
+if os.path.exists(outputdir):
+    print "Output directory is ", outputdir
+else:
+    sys.exit("Bad OutputDir")
+
 matcher = re.compile(r"^r\..*\.mcr$")
 
 POI = []
@@ -63,8 +67,12 @@ for dirpath, dirnames, filenames in os.walk(worlddir):
                             print "Found sign at (%d, %d, %d): %r" % (newPOI['x'], newPOI['y'], newPOI['z'], newPOI['msg'])
 
 
+if os.path.isfile(os.path.join(worlddir, "overviewer.dat")):
+    print "Overviewer.dat detected in WorldDir - this is no longer the correct location\n"
+    print "You may wish to delete the old file. A new overviewer.dat will be created\n"
+    print "Old file: ", os.path.join(worlddir, "overviewer.dat")
 
-pickleFile = os.path.join(worlddir,"overviewer.dat")
+pickleFile = os.path.join(outputdir,"overviewer.dat")
 with open(pickleFile,"wb") as f:
     cPickle.dump(dict(POI=POI), f)
 
diff --git a/overviewer.py b/overviewer.py
index 4c20613..8138ca9 100755
--- a/overviewer.py
+++ b/overviewer.py
@@ -241,7 +241,7 @@ def main():
         logging.info("Notice: Not using biome data for tinting")
     
     # First do world-level preprocessing
-    w = world.World(worlddir, useBiomeData=useBiomeData, regionlist=regionlist)
+    w = world.World(worlddir, destdir, useBiomeData=useBiomeData, regionlist=regionlist)
     w.go(options.procs)
 
     logging.info("Rending the following tilesets: %s", ",".join(options.rendermode))
@@ -281,7 +281,7 @@ def delete_all(worlddir, tiledir):
     # TODO should we delete tiledir here too?
     
     # delete the overviewer.dat persistant data file
-    datfile = os.path.join(worlddir,"overviewer.dat")
+    datfile = os.path.join(tiledir,"overviewer.dat")
     if os.path.exists(datfile):
         os.unlink(datfile)
         logging.info("Deleting {0}".format(datfile))
diff --git a/overviewer_core/world.py b/overviewer_core/world.py
index aece3fa..a18b86a 100644
--- a/overviewer_core/world.py
+++ b/overviewer_core/world.py
@@ -69,8 +69,9 @@ class World(object):
 
     mincol = maxcol = minrow = maxrow = 0
     
-    def __init__(self, worlddir, useBiomeData=False,regionlist=None):
+    def __init__(self, worlddir, outputdir, useBiomeData=False,regionlist=None):
         self.worlddir = worlddir
+        self.outputdir = outputdir
         self.useBiomeData = useBiomeData
                 
         #find region files, or load the region list
@@ -111,8 +112,20 @@ class World(object):
         # info self.persistentData.  This dictionary can hold any information
         # that may be needed between runs.
         # Currently only holds into about POIs (more more details, see quadtree)
-        # TODO maybe store this with the tiles, not with the world?
+        
         self.pickleFile = os.path.join(self.worlddir, "overviewer.dat")
+        if os.path.exists(self.pickleFile):
+            logging.warning("overviewer.dat detected in WorldDir - this is no longer the correct location")
+            logging.warning("Moving overviewer.dat to OutputDir")
+            import shutil
+            try:
+                shutil.move(self.pickleFile, self.outputdir)
+                logging.info("overviewer.dat moved")
+            except BaseException as ex:
+                logging.error("Unable to move overviewer.dat")
+                logging.debug(ex.str())
+
+        self.pickleFile = os.path.join(self.outputdir, "overviewer.dat")    
         if os.path.exists(self.pickleFile):
             with open(self.pickleFile,"rb") as p:
                 self.persistentData = cPickle.load(p)