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

Re: [PATCH 00/13] remote-hg: general updates

From
Felipe Contreras <felipe.contreras@gmail.com>
Date
Apr 4, 2013, 18:41 UTC
Message-ID
<CAMP44s3zm+-220Xpqv_=JkJ=g2pS79hUNjacVZ=JejitcG=B8A@mail.gmail.com>
In-Reply-To
<871uaqrwrp.fsf@59A2.org>
On Thu, Apr 4, 2013 at 12:11 PM, Jed Brown <jed@59a2.org> wrote:
Show 15 quoted lines
> Felipe Contreras <felipe.contreras@gmail.com> writes:
>
>> I still don't see any good reason why a user might prefer gitifyhg,
>> even more importantly, why gitifyhg developers don't contribute to
>> remote-hg.
>
> Felipe, I read your blog announcement [1] and got the impression that
> remote-hg was ready for daily use.  When I tried to use it, it promptly
> crashed in my first attempt to clone.  I opened up the script, fixed
> whatever caused the first stack trace and made it slighly further before
> it crashed again.  I couldn't tell what was expected to work, what was a
> known problem, and what was an unknown problem.  Many things clearly did
> not work and it had the look of a project that was not getting active
> use.  I felt that it was wildly oversold and that putting it into
> git.git was premature.

Where is the evidence? You say remote-hg doesn't work, I say it does, the difference is that I have evidence to prove it.

There's no bug report, no mail, no log of any kind, no repository to check, no nothing. What are developers supposed to do with this comment? Nothing, it's not constructive.

> I tried gitifyhg later and it basically worked out of the box.  All
> known problems were marked by 'xfail' test cases.  At that time,
> remote-hg failed almost all the gitifyhg tests.
Because gitifyhg tests are testing gitifyhg, not proper behavior. For example:
def test_push_conflict_default(git_dir, hg_repo):
    git_repo = clone_repo(git_dir, hg_repo)
    sh.cd(hg_repo)
    make_hg_commit("b")
    sh.cd(git_repo)
    make_git_commit("c")
    assert sh.git.push(_ok_code=1).stderr.find("master -> master
(non-fast-forward)") > 0

remote-hg doesn't fail with the non-fast-forward error, in fact, it doesn't fail at all, it pushes correctly, and that's reported as a failure.

Show 5 quoted lines
> I contributed a few
> things to gitifyhg, including the notes support (essential when talking
> via email with other people using Mercurial).  Since then, the last
> major project I'm involved with has switched to Git so I rarely need
> gitifyhg or remote-hg any more.

So what is the purpose of this anecdote? Are you going to provide something of value, or is this just an attempt to root for the project you have contributed to?

> FWIW, I also thought Dusty's original announcement oversold gitifyhg, but
> it was closer to the truth and upon cloning the repo, it was more clear
> what didn't work.

This is *one* data-point, you can't make such overreaching generalizations from *one* data-point; you will often be wrong.

Show 7 quoted lines
> The early history of gitifyhg is quite chaotic and I
> didn't realize at first how much code turned out to be borrowed from
> remote-hg.  I don't know whether you wrote all of remote-hg or borrowed
> significant parts from elsewhere.  To be honest, I don't really care,
> but it would be good to coalesce around one project that is well-tested
> and has documented behavior so that the poor folks stuck with Mercurial
> upstreams can have dependable behavior.

Indeed, and mails like this don't help one iota to achieve that. We need data, arguments, and code, and you are not providing any.

Now, if you can go fetch the log of the error you had, that would be great, otherwise I'll turn to something productive, like improving remote-hg.

Cheers.
-- 
Felipe Contreras
Previous: Felipe ContrerasNext: Jed Brown
Message 45 of 48 in “remote-hg: general updates”
  1. 00/13 remote-hg: general updatesFelipe Contreras, Apr 2, 2013
  2. 01/13 remote-hg: trivial cleanupsFelipe Contreras, Apr 2, 2013
  3. 02/13 remote-hg: add missing config variable in docFelipe Contreras, Apr 2, 2013
  4. 03/13 remote-hg: properly report errors on bookmark pushesFelipe Contreras, Apr 2, 2013
  5. 04/13 remote-hg: fix for files with spacesFelipe Contreras, Apr 2, 2013
  6. 05/13 remote-hg: make sure fake bookmarks are updatedFelipe Contreras, Apr 2, 2013
  7. 06/13 remote-hg: trivial test cleanupsFelipe Contreras, Apr 2, 2013
  8. 07/13 remote-hg: redirect buggy mercurial outputFelipe Contreras, Apr 2, 2013
  9. Junio C HamanoApr 2, 2013
  10. Felipe ContrerasApr 2, 2013
  11. Junio C HamanoApr 2, 2013
  12. Felipe ContrerasApr 4, 2013
  13. Junio C HamanoApr 4, 2013
  14. 08/13 remote-hg: split bookmark handlingFelipe Contreras, Apr 2, 2013
  15. 09/13 remote-hg: refactor exportFelipe Contreras, Apr 2, 2013
  16. 10/13 remote-hg: update remote bookmarksFelipe Contreras, Apr 2, 2013
  17. 11/13 remote-hg: force remote pushFelipe Contreras, Apr 2, 2013
  18. 12/13 remote-hg: don't update bookmarks unnecessarilyFelipe Contreras, Apr 2, 2013
  19. 13/13 remote-hg: update tags globallyFelipe Contreras, Apr 2, 2013
  20. Junio C HamanoApr 2, 2013
  21. Felipe ContrerasApr 2, 2013
  22. Junio C HamanoApr 2, 2013
  23. John KeepingApr 2, 2013
  24. Max HornApr 2, 2013
  25. Felipe ContrerasApr 3, 2013
  26. Felipe ContrerasApr 3, 2013
  27. Antoine PelisseApr 3, 2013
  28. Felipe ContrerasApr 5, 2013
  29. Max HornApr 4, 2013
  30. Felipe ContrerasApr 4, 2013
  31. Felipe ContrerasApr 4, 2013
  32. Max HornApr 4, 2013
  33. Felipe ContrerasApr 4, 2013
  34. Max HornApr 5, 2013
  35. Felipe ContrerasApr 6, 2013
  36. Philip OakleyApr 6, 2013
  37. Felipe ContrerasApr 6, 2013
  38. Junio C HamanoApr 6, 2013
  39. Felipe ContrerasApr 6, 2013
  40. Junio C HamanoApr 7, 2013
  41. Jed BrownApr 4, 2013
  42. Junio C HamanoApr 4, 2013
  43. Jed BrownApr 4, 2013
  44. Felipe ContrerasApr 4, 2013
  45. Felipe ContrerasApr 4, 2013
  46. Jed BrownApr 4, 2013
  47. Felipe ContrerasApr 4, 2013
  48. Felipe ContrerasApr 5, 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.