Page MenuHomePhorge
Files F2681397

D25148.1734641215.diff
No OneTemporary
Actions

D25148.1734641215.diff
View Options

diff --git a/src/applications/config/controller/PhabricatorConfigConsoleController.php b/src/applications/config/controller/PhabricatorConfigConsoleController.php
--- a/src/applications/config/controller/PhabricatorConfigConsoleController.php
+++ b/src/applications/config/controller/PhabricatorConfigConsoleController.php
@@ -189,9 +189,10 @@
foreach ($specs as $lib) {
$remote_future = $remote_futures[$lib];
- list($err, $stdout) = $remote_future->resolve();
- if ($err) {
- // If this fails for whatever reason, just move on.
+ try {
+ list($stdout, $err) = $remote_future->resolvex();
+ } catch (CommandException $e) {
+ $this->logGitErrorWithPotentialTips($e, $lib);
continue;
}
@@ -258,13 +259,14 @@
$results = array();
foreach ($log_futures as $lib => $future) {
- list($err, $stdout) = $future->resolve();
- if (!$err) {
+ try {
+ list($stdout, $err) = $future->resolvex();
list($hash, $epoch) = explode(' ', $stdout);
- } else {
+ } catch (CommandException $e) {
$hash = null;
$epoch = null;
- }
+ $this->logGitErrorWithPotentialTips($e, $lib);
+ }
$result = array(
'hash' => $hash,
@@ -275,7 +277,7 @@
$upstream_future = idx($upstream_futures, $lib);
if ($upstream_future) {
- list($err, $stdout) = $upstream_future->resolve();
+ list($stdout, $err) = $upstream_future->resolvex();
if (!$err) {
$branchpoint = trim($stdout);
if (strlen($branchpoint)) {
@@ -340,5 +342,36 @@
->appendChild($table_view);
}
+ /**
+ * Help in better troubleshooting git errors.
+ * @param CommandException $e Exception
+ * @param string $lib Library name involved
+ */
+ private function logGitErrorWithPotentialTips($e, $lib) {
+
+ // First, detect this specific error message related to [safe] stuff.
+ $expected_error_msg_part = 'detected dubious ownership in repository';
+ $stderr = $e->getStderr();
+ if (strpos($stderr, $expected_error_msg_part) !== false) {
+
+ // Found! Let's show a nice resolution tip.
+
+ // Complete path of the problematic repository.
+ $lib_root = dirname(phutil_get_library_root($lib));
+
+ phlog(pht(
+ "Cannot identify the version of the %s repository because ".
+ "the webserver does not trust it (more info on Task %s).\n".
+ "Try this system resolution:\n".
+ "sudo git config --system --add safe.directory %s",
+ $lib,
+ 'https://we.phorge.it/T15282',
+ $lib_root));
+ } else {
+
+ // Otherwise show a generic error message
+ phlog($e);
+ }
+ }
}

File Metadata

Mime Type
text/plain
Expires
Thu, Dec 19, 20:46 (19 h, 18 m)
Storage Engine
blob
Storage Format
Raw Data
Storage Handle
1015566
Default Alt Text
D25148.1734641215.diff (2 KB)

Event Timeline

Content licensed under Creative Commons Attribution-ShareAlike 4.0 (CC-BY-SA) unless otherwise noted; code licensed under Apache 2.0 or other open source licenses. · CC BY-SA 4.0 · Apache 2.0