diff options
| author | Joe Rayhawk <jrayhawk@omgwallhack.org> | 2011-05-30 23:55:18 -0700 |
|---|---|---|
| committer | Joe Rayhawk <jrayhawk@omgwallhack.org> | 2011-05-30 23:55:18 -0700 |
| commit | ab2df0b7b3ff8020abadd9ad338600e771064961 (patch) | |
| tree | bcb7ff47da4c7ca8b7628ca6fc0b9beebbe2ece4 /libpiny/lib/Piny | |
| parent | cd3ea35e724757aa86f6db16afb16dd280c303f4 (diff) | |
| download | piny-code-ab2df0b7b3ff8020abadd9ad338600e771064961.tar.gz piny-code-ab2df0b7b3ff8020abadd9ad338600e771064961.zip | |
Adding primitive cloning support
Diffstat (limited to 'libpiny/lib/Piny')
| -rw-r--r-- | libpiny/lib/Piny/Repo.pm | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/libpiny/lib/Piny/Repo.pm b/libpiny/lib/Piny/Repo.pm index 82f651d..8a26563 100644 --- a/libpiny/lib/Piny/Repo.pm +++ b/libpiny/lib/Piny/Repo.pm @@ -473,8 +473,8 @@ sub create { chdir( "/srv/git" ); # so git-clone can do the work of resolving local repo names for us if ( defined $source ) { - system( "/bin/chown", $user . "." . $user, $repo->path ) and die "Could not change ownership of repo path!"; # permissions are overridden later - if ( system( "/usr/bin/sudo", "-u", $user, "/usr/bin/git", "clone", "--bare", "--no-hardlinks", "--quiet", $source, $repo->path ) ) { # sudo -u $user needed in order to test user readability of whatever they want to clone. the mode o+rx should probably be masked off if we want private repos to stay private. + system( "/bin/chown", $user->name, $repo->path ) and die "Could not change ownership of repo path!"; # permissions are overridden later + if ( system( "/usr/bin/sudo", "-u", $user->name, "/usr/bin/git", "clone", "--bare", "--no-hardlinks", "--quiet", $source, $repo->path ) ) { # sudo -u $user needed in order to test user readability of whatever they want to clone. the mode o+rx should probably be masked off if we want private repos to stay private. system( "/bin/rm", "-r", $repo->path ) and die "Failed to clean up after failed clone operation!"; die( "Could not clone repository!\n" ); }; |