1
0
Fork 0

Merge pull request #1986 from xrstf/fixed-hgdownloader

fixed escaping the path (it's no cmdline argument anymore)
pull/1990/merge
Jordi Boggiano 2013-06-10 14:09:33 -07:00
commit ffb9a88f7f
1 changed files with 1 additions and 3 deletions

View File

@ -26,9 +26,8 @@ class HgDownloader extends VcsDownloader
{ {
$url = escapeshellarg($package->getSourceUrl()); $url = escapeshellarg($package->getSourceUrl());
$ref = escapeshellarg($package->getSourceReference()); $ref = escapeshellarg($package->getSourceReference());
$path = escapeshellarg($path);
$this->io->write(" Cloning ".$package->getSourceReference()); $this->io->write(" Cloning ".$package->getSourceReference());
$command = sprintf('hg clone %s %s', $url, $path); $command = sprintf('hg clone %s %s', $url, escapeshellarg($path));
if (0 !== $this->process->execute($command, $ignoredOutput)) { if (0 !== $this->process->execute($command, $ignoredOutput)) {
throw new \RuntimeException('Failed to execute ' . $command . "\n\n" . $this->process->getErrorOutput()); throw new \RuntimeException('Failed to execute ' . $command . "\n\n" . $this->process->getErrorOutput());
} }
@ -45,7 +44,6 @@ class HgDownloader extends VcsDownloader
{ {
$url = escapeshellarg($target->getSourceUrl()); $url = escapeshellarg($target->getSourceUrl());
$ref = escapeshellarg($target->getSourceReference()); $ref = escapeshellarg($target->getSourceReference());
$path = escapeshellarg($path);
$this->io->write(" Updating to ".$target->getSourceReference()); $this->io->write(" Updating to ".$target->getSourceReference());
$command = sprintf('hg pull %s && hg up %s', $url, $ref); $command = sprintf('hg pull %s && hg up %s', $url, $ref);
if (0 !== $this->process->execute($command, $ignoredOutput, $path)) { if (0 !== $this->process->execute($command, $ignoredOutput, $path)) {