# HG changeset patch
# User Pierre-Yves David <pierre-yves.david@octobus.net>
# Date 1618525003 -7200
# Node ID 9cc9b4a25248a5fe8265e7b9659077a9e0f8098a
# Parent  ba673c821b9d107a14b33dea7de6d93fd549dafd
urlutil: provide some information about "bad url" when processing `pushurl`

It appears pushurl only support `<proto>://` entries. This is not obvious and
can lead to obscure error. We make the error less obscure as a start..

Differential Revision: https://phab.mercurial-scm.org/D10455

diff -r ba673c821b9d -r 9cc9b4a25248 mercurial/utils/urlutil.py
--- a/mercurial/utils/urlutil.py	Sun Apr 11 19:19:34 2021 +0200
+++ b/mercurial/utils/urlutil.py	Fri Apr 16 00:16:43 2021 +0200
@@ -742,7 +742,9 @@
     u = url(value)
     # Actually require a URL.
     if not u.scheme:
-        ui.warn(_(b'(paths.%s:pushurl not a URL; ignoring)\n') % path.name)
+        msg = _(b'(paths.%s:pushurl not a URL; ignoring: "%s")\n')
+        msg %= (path.name, value)
+        ui.warn(msg)
         return None
 
     # Don't support the #foo syntax in the push URL to declare branch to
diff -r ba673c821b9d -r 9cc9b4a25248 tests/test-paths.t
--- a/tests/test-paths.t	Sun Apr 11 19:19:34 2021 +0200
+++ b/tests/test-paths.t	Fri Apr 16 00:16:43 2021 +0200
@@ -175,7 +175,7 @@
   > EOF
 
   $ hg paths
-  (paths.default:pushurl not a URL; ignoring)
+  (paths.default:pushurl not a URL; ignoring: "/not/a/url")
   default = /path/to/nothing
 
 #fragment is not allowed in :pushurl