git/list[1] front-page[2] threads[3] people[4] search[5] about
 

[PATCH v4 21/21] remote-hg: activate graphlog extension for hg_log()

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Apr 11, 2013, 12:23 UTC
Message-ID
<1365682997-11329-22-git-send-email-felipe.contreras@gmail.com>
In-Reply-To
<1365682997-11329-1-git-send-email-felipe.contreras@gmail.com>
From: Antoine Pelisse <apelisse@gmail.com>

The hg_log() test helper uses the "--graph" parameter that is implemented by the GraphLog extension. If the extension is not activated by the user, the parameter is not available. Activate the extension in setup().

Also changes the way we grep the output in hg_log(). The pipe operator can hide the return code of hg command. As a matter of fact, if log fails because it doesn't know about "--graph", it doesn't report any failure and let's you think everything worked.

Signed-off-by: Antoine Pelisse <apelisse@gmail.com>
Signed-off-by: Felipe Contreras <felipe.contreras@gmail.com>
---
 contrib/remote-helpers/test-hg-bidi.sh   | 5 ++++-
 contrib/remote-helpers/test-hg-hg-git.sh | 4 +++-
 2 files changed, 7 insertions(+), 2 deletions(-)
diff --git a/contrib/remote-helpers/test-hg-bidi.sh b/contrib/remote-helpers/test-hg-bidi.sh
index a3c88f6..f368953 100755
--- a/contrib/remote-helpers/test-hg-bidi.sh
+++ b/contrib/remote-helpers/test-hg-bidi.sh
@@ -50,7 +50,8 @@ hg_push () {
 }
 
 hg_log () {
-	hg -R $1 log --graph --debug | grep -v 'tag: *default/'
+	hg -R $1 log --graph --debug >log &&
+	grep -v 'tag: *default/' log
 }
 
 setup () {
@@ -62,6 +63,8 @@ setup () {
 	echo "commit = -d \"0 0\""
 	echo "debugrawcommit = -d \"0 0\""
 	echo "tag = -d \"0 0\""
+	echo "[extensions]"
+	echo "graphlog ="
 	) >> "$HOME"/.hgrc &&
 	git config --global remote-hg.hg-git-compat true
 
diff --git a/contrib/remote-helpers/test-hg-hg-git.sh b/contrib/remote-helpers/test-hg-hg-git.sh
index 73ae18d..253e65a 100755
--- a/contrib/remote-helpers/test-hg-hg-git.sh
+++ b/contrib/remote-helpers/test-hg-hg-git.sh
@@ -78,7 +78,8 @@ hg_push_hg () {
 }
 
 hg_log () {
-	hg -R $1 log --graph --debug | grep -v 'tag: *default/'
+	hg -R $1 log --graph --debug >log &&
+	grep -v 'tag: *default/' log
 }
 
 git_log () {
@@ -97,6 +98,7 @@ setup () {
 	echo "[extensions]"
 	echo "hgext.bookmarks ="
 	echo "hggit ="
+	echo "graphlog ="
 	) >> "$HOME"/.hgrc &&
 	git config --global receive.denycurrentbranch warn
 	git config --global remote-hg.hg-git-compat true
-- 
1.8.2.1
Previous: Felipe ContrerasNext: Junio C Hamano
Message 24 of 25 in “remote-hg: general updates”
  1. 00/21 remote-hg: general updatesFelipe Contreras, Apr 11, 2013
  2. 01/21 remote-hg: trivial cleanupsFelipe Contreras, Apr 11, 2013
  3. 02/21 remote-hg: add missing config variable in docFelipe Contreras, Apr 11, 2013
  4. 03/21 remote-hg: properly report errors on bookmark pushesFelipe Contreras, Apr 11, 2013
  5. 04/21 remote-hg: fix for files with spacesFelipe Contreras, Apr 11, 2013
  6. 05/21 remote-hg: make sure fake bookmarks are updatedFelipe Contreras, Apr 11, 2013
  7. 06/21 remote-hg: trivial test cleanupsFelipe Contreras, Apr 11, 2013
  8. 07/21 remote-hg: redirect buggy mercurial outputFelipe Contreras, Apr 11, 2013
  9. 08/21 remote-hg: split bookmark handlingFelipe Contreras, Apr 11, 2013
  10. 09/21 remote-hg: refactor exportFelipe Contreras, Apr 11, 2013
  11. 10/21 remote-hg: update remote bookmarksFelipe Contreras, Apr 11, 2013
  12. 11/21 remote-hg: update tags globallyFelipe Contreras, Apr 11, 2013
  13. 12/21 remote-hg: push to the appropriate branchFelipe Contreras, Apr 11, 2013
  14. 13/21 remote-hg: force remote pushFelipe Contreras, Apr 11, 2013
  15. 14/21 remote-hg: show more proper errorsFelipe Contreras, Apr 11, 2013
  16. Antoine PelisseMay 27, 2013
  17. Felipe ContrerasMay 27, 2013
  18. 15/21 remote-hg: add basic author testsFelipe Contreras, Apr 11, 2013
  19. 16/21 remote-hg: add simple mail testFelipe Contreras, Apr 11, 2013
  20. 17/21 remote-hg: add 'insecure' optionFelipe Contreras, Apr 11, 2013
  21. 18/21 remote-hg: fix bad state issueFelipe Contreras, Apr 11, 2013
  22. 19/21 remote-hg: document location of stored hg repositoryFelipe Contreras, Apr 11, 2013
  23. 20/21 remote-hg: fix bad file pathsFelipe Contreras, Apr 11, 2013
  24. 21/21 remote-hg: activate graphlog extension for hg_log()Felipe Contreras, Apr 11, 2013
  25. Junio C HamanoApr 11, 2013

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.