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

« back to all changes in this revision

Viewing changes to ivle/makeuser.py

  • Committer: William Grant
  • Date: 2009-04-29 05:20:14 UTC
  • Revision ID: grantw@unimelb.edu.au-20090429052014-5tzyfz97fwt5mn0k
Replace some of ivle.makeuser's os.system calls with subprocess.call.

This makes Subversion repositories with spaces work.

Show diffs side-by-side

added added

removed removed

Lines of Context:
44
44
import warnings
45
45
import filecmp
46
46
import logging
 
47
import subprocess
47
48
import ivle.pulldown_subj
48
49
 
49
50
from ivle.database import ProjectGroup
50
51
 
51
52
def chown_to_webserver(filename):
52
 
    """
53
 
    Chowns a file so the web server user owns it.
54
 
    (This is useful in setting up Subversion conf files).
 
53
    """chown a directory and its contents to the web server.
 
54
 
 
55
    Recursively chowns a file or directory so the web server user owns it.
55
56
    Assumes root.
56
57
    """
57
 
    try:
58
 
        os.system("chown -R www-data:www-data %s" % filename)
59
 
    except:
60
 
        pass
 
58
    subprocess.call(['chown', '-R', 'www-data:www-data', filename])
61
59
 
62
60
def make_svn_repo(path, throw_on_error=True):
63
 
    """Create a Subversion repository at the given path.
64
 
    """
 
61
    """Create a Subversion repository at the given path."""
65
62
    try:
66
 
        res = os.system("svnadmin create '%s'" % path)
 
63
        res = subprocess.call(['svnadmin', 'create', path])
67
64
        if res != 0 and throw_on_error:
68
65
            raise Exception("Cannot create repository: %s" % path)
69
66
    except Exception, exc: