← Back to branch summary

~azzar1/unity/add-show-desktop-key

« back to all changes in this revision

Viewing changes to ivle/zip.py

  • Committer: David Coles
  • Date: 2010-08-30 03:26:13 UTC
  • Revision ID: coles.david@gmail.com-20100830032613-d14vng0jkelniu3l
python-console: Fix globals broken with new JSON library.

simplejson always returns unicode strings. cJSON would return ordinary strings 
if possible. cPickle.loads() only accepts strings. At present we use pickle 
version 0 so they should all works as ASCII strings. Higher versions of pickle 
are not plain ASCII and are likely to break this and so this should be fixed 
at some point.

Also replaced unconditional exception with one that catches Pickle errors. Not 
sure the best way to report failures of these functions.

Show diffs side-by-side

added added

removed removed

Lines of Context:
ivle/zip.py
27
27
import zipfile
28
28
 
29
29
from ivle import studpath
30
 
from ivle.fileservice_lib.exceptions import WillNotOverwrite
31
30
 
32
31
def make_zip(basepath, paths, file):
33
32
    """Zips up a bunch of files on the student file space and writes it as
95
94
    The path is an absolute path in the current filesystem
96
95
    (if this code is executed inside the jail, then it's inside the jail).
97
96
    """
 
97
    # XXX: Really bad, but circular imports need avoiding.
 
98
    from ivle.fileservice_lib.exceptions import WillNotOverwrite
 
99
 
98
100
    zip = zipfile.ZipFile(file, 'r')
99
101
    # First test the zip file
100
102
    if zip.testzip() is not None: