1
0
Fork 0

Make sure github driver is used for all github urls

pull/592/merge
Jordi Boggiano 2012-05-12 18:24:07 +02:00
parent 160d2e0847
commit c2933028e8
2 changed files with 9 additions and 2 deletions

View File

@ -129,9 +129,16 @@ class GitDownloader extends VcsDownloader
if (0 === $this->process->execute($command, $handler)) {
return;
}
if (null !== $path) {
$this->filesystem->removeDirectory($path);
}
$retrying = true;
} while (--$retries);
}
if (null !== $path) {
$this->filesystem->removeDirectory($path);
}
$this->throwException('Failed to execute ' . $command . "\n\n" . $this->process->getErrorOutput(), $url);
}
}

View File

@ -43,7 +43,7 @@ class GitHubDriver extends VcsDriver
*/
public function initialize()
{
preg_match('#^(?:https?|git)://github\.com/([^/]+)/(.+?)(?:\.git)?$#', $this->url, $match);
preg_match('#^(?:(?:https?|git)://github\.com/|git@github\.com:)([^/]+)/(.+?)(?:\.git)?$#', $this->url, $match);
$this->owner = $match[1];
$this->repository = $match[2];
$this->originUrl = 'github.com';
@ -177,7 +177,7 @@ class GitHubDriver extends VcsDriver
*/
public static function supports(IOInterface $io, $url, $deep = false)
{
if (!preg_match('#^(?:https?|git)://github\.com/([^/]+)/(.+?)(?:\.git)?$#', $url)) {
if (!preg_match('#^((?:https?|git)://github\.com/|git@github\.com:)([^/]+)/(.+?)(?:\.git)?$#', $url)) {
return false;
}