LogalDeveloper/Minecraft-Overviewer
Archived
0
Fork 0
Code Activity

Fixed Python3 error message

Browse Source 
Up to this point, when launching overviewer with python3, overviewer would
(ironic as it is) crash on the line where the error message would be displayed
with a syntax error. I have now fixed this.
This commit is contained in:
Nicolas Frattaroli
2013-03-11 18:06:54 +01:00
parent b8e2003c65
commit 633dcd18c1
1 changed files with 28 additions and 28 deletions
Download Patch File Download Diff File Expand all files Collapse all files

56
overviewer.py
View File

@@ -20,9 +20,9 @@ import sys
# quick version check # quick version check
if not (sys.version_info[0] == 2 and sys.version_info[1] >= 6): if not (sys.version_info[0] == 2 and sys.version_info[1] >= 6):
print "Sorry, the Overviewer requires at least Python 2.6 to run" print("Sorry, the Overviewer requires at least Python 2.6 to run")
if sys.version_info[0] >= 3: if sys.version_info[0] >= 3:
print "and will not run on Python 3.0 or later" print("and will not run on Python 3.0 or later")
sys.exit(1) sys.exit(1)
import os import os
@@ -125,15 +125,15 @@ def main():
# This section of main() runs in response to any one-time options we have, # This section of main() runs in response to any one-time options we have,
# such as -V for version reporting # such as -V for version reporting
if options.version: if options.version:
print "Minecraft Overviewer %s" % util.findGitVersion(), print("Minecraft Overviewer %s" % util.findGitVersion()),
print "(%s)" % util.findGitHash()[:7] print("(%s)" % util.findGitHash()[:7])
try: try:
import overviewer_core.overviewer_version as overviewer_version import overviewer_core.overviewer_version as overviewer_version
print "built on %s" % overviewer_version.BUILD_DATE print("built on %s" % overviewer_version.BUILD_DATE)
if options.verbose > 0: if options.verbose > 0:
print "Build machine: %s %s" % (overviewer_version.BUILD_PLATFORM, overviewer_version.BUILD_OS) print("Build machine: %s %s" % (overviewer_version.BUILD_PLATFORM, overviewer_version.BUILD_OS))
except ImportError: except ImportError:
print "(build info not found)" print("(build info not found)")
return 0 return 0
if options.check_terrain: if options.check_terrain:
@@ -158,13 +158,13 @@ def main():
# first provide an appropriate error for bare-console users # first provide an appropriate error for bare-console users
# that don't provide any options # that don't provide any options
if util.is_bare_console(): if util.is_bare_console():
print "\n" print("\n")
print "The Overviewer is a console program. Please open a Windows command prompt" print("The Overviewer is a console program. Please open a Windows command prompt")
print "first and run Overviewer from there. Further documentation is available at" print("first and run Overviewer from there. Further documentation is available at")
print "http://docs.overviewer.org/\n" print("http://docs.overviewer.org/\n")
print "\n" print("\n")
print "For a quick-start guide on Windows, visit the following URL:\n" print("For a quick-start guide on Windows, visit the following URL:\n")
print "http://docs.overviewer.org/en/latest/win_tut/windowsguide/\n" print("http://docs.overviewer.org/en/latest/win_tut/windowsguide/\n")
else: else:
# more helpful message for users who know what they're doing # more helpful message for users who know what they're doing
@@ -178,13 +178,13 @@ def main():
# in. It checks to see if --config was given that no worldname/destdir were # in. It checks to see if --config was given that no worldname/destdir were
# given, and vice versa # given, and vice versa
if options.config and args: if options.config and args:
print print()
print "If you specify --config, you need to specify the world to render as well as" print("If you specify --config, you need to specify the world to render as well as")
print "the destination in the config file, not on the command line." print("the destination in the config file, not on the command line.")
print "Put something like this in your config file:" print("Put something like this in your config file:")
print "worlds['myworld'] = %r" % args[0] print("worlds['myworld'] = %r" % args[0])
print "outputdir = %r" % (args[1] if len(args) > 1 else "/path/to/output") print("outputdir = %r" % (args[1] if len(args) > 1 else "/path/to/output"))
print print()
logging.error("Cannot specify both --config AND a world + output directory on the command line.") logging.error("Cannot specify both --config AND a world + output directory on the command line.")
parser.print_help() parser.print_help()
return 1 return 1
@@ -471,16 +471,16 @@ def list_worlds():
print print
worlds = world.get_worlds() worlds = world.get_worlds()
if not worlds: if not worlds:
print 'No world saves found in the usual place' print('No world saves found in the usual place')
return return
print "Detected saves:" print("Detected saves:")
# get max length of world name # get max length of world name
worldNameLen = max([len(str(x)) for x in worlds] + [len("World")]) worldNameLen = max([len(str(x)) for x in worlds] + [len("World")])
formatString = "%-" + str(worldNameLen) + "s | %-8s | %-8s | %-16s | %s " formatString = "%-" + str(worldNameLen) + "s | %-8s | %-8s | %-16s | %s "
print formatString % ("World", "Size", "Playtime", "Modified", "Path") print(formatString % ("World", "Size", "Playtime", "Modified", "Path"))
print formatString % ("-"*worldNameLen, "-"*8, "-"*8, '-'*16, '-'*4) print(formatString % ("-"*worldNameLen, "-"*8, "-"*8, '-'*16, '-'*4))
for name, info in sorted(worlds.iteritems()): for name, info in sorted(worlds.iteritems()):
if isinstance(name, basestring) and name.startswith("World") and len(name) == 6: if isinstance(name, basestring) and name.startswith("World") and len(name) == 6:
try: try:
@@ -496,18 +496,18 @@ def list_worlds():
playstamp = '%d:%02d' % (playtime / 3600, playtime / 60 % 60) playstamp = '%d:%02d' % (playtime / 3600, playtime / 60 % 60)
size = "%.2fMB" % (info['SizeOnDisk'] / 1024. / 1024.) size = "%.2fMB" % (info['SizeOnDisk'] / 1024. / 1024.)
path = info['path'] path = info['path']
print formatString % (name, size, playstamp, timestamp, path) print(formatString % (name, size, playstamp, timestamp, path))
if __name__ == "__main__": if __name__ == "__main__":
multiprocessing.freeze_support() multiprocessing.freeze_support()
try: try:
ret = main() ret = main()
util.nice_exit(ret) util.nice_exit(ret)
except textures.TextureException, e: except textures.TextureException as e:
# this isn't a "bug", so don't print scary traceback # this isn't a "bug", so don't print scary traceback
logging.error(str(e)) logging.error(str(e))
util.nice_exit(1) util.nice_exit(1)
except Exception, e: except Exception as e:
logging.exception("""An error has occurred. This may be a bug. Please let us know! logging.exception("""An error has occurred. This may be a bug. Please let us know!
See http://docs.overviewer.org/en/latest/index.html#help See http://docs.overviewer.org/en/latest/index.html#help