{"thread":{"id":"33839","subject":"[PATCH] Revert \"remote-hg: update bookmarks when pulling\"","startedAt":"2013-05-16T12:43:15Z","lastAt":"2013-05-16T23:14:40Z","messageCount":6,"participants":["Felipe Contreras","Junio C Hamano"],"isPatch":true,"patchVersion":1,"patchTotal":null},"messages":[{"id":"217586","messageId":"1368708195-1044-1-git-send-email-felipe.contreras@gmail.com","threadId":"33839","inReplyTo":null,"subject":"[PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T12:43:15Z","receivedAt":"2013-05-16T12:43:15Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n\nDifferent versions of Mercurial have different arguments for\nbookmarks.updatefromremote(), while it should be possible to call the\nright function with the right arguments depending on the version, it's\nsafer to restore the old behavior for now.\n\nReported by Rodney Lorrimar.\n\nSigned-off-by: Felipe Contreras \n---\n\nIntended for master (v1.8.3).\n\n contrib/remote-helpers/git-remote-hg | 3 ---\n 1 file changed, 3 deletions(-)\n\ndiff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\nindex dc276af..beb864b 100755\n--- a/contrib/remote-helpers/git-remote-hg\n+++ b/contrib/remote-helpers/git-remote-hg\n@@ -363,9 +363,6 @@ def get_repo(url, alias):\n die('Repository error')\n repo.pull(peer, heads=None, force=True)\n \n- rb = peer.listkeys('bookmarks')\n- bookmarks.updatefromremote(myui, repo, rb, url)\n-\n return repo\n \n def rev_to_mark(rev):\n-- \n1.8.3.rc2.542.g24820ba\n"},{"id":"217625","messageId":"7v4ne27ubp.fsf@alter.siamese.dyndns.org","threadId":"33839","inReplyTo":"1368708195-1044-1-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T16:39:22Z","receivedAt":"2013-05-16T16:39:22Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras writes:\n\n> This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n>\n> Different versions of Mercurial have different arguments for\n> bookmarks.updatefromremote(), while it should be possible to call the\n> right function with the right arguments depending on the version, it's\n> safer to restore the old behavior for now.\n>\n> Reported by Rodney Lorrimar.\n>\n> Signed-off-by: Felipe Contreras \n> ---\n>\n> Intended for master (v1.8.3).\n\nHmm, is this the one we merged yesterday?\n\n>\n> contrib/remote-helpers/git-remote-hg | 3 ---\n> 1 file changed, 3 deletions(-)\n>\n> diff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\n> index dc276af..beb864b 100755\n> --- a/contrib/remote-helpers/git-remote-hg\n> +++ b/contrib/remote-helpers/git-remote-hg\n> @@ -363,9 +363,6 @@ def get_repo(url, alias):\n> die('Repository error')\n> repo.pull(peer, heads=None, force=True)\n> \n> - rb = peer.listkeys('bookmarks')\n> - bookmarks.updatefromremote(myui, repo, rb, url)\n> -\n> return repo\n> \n> def rev_to_mark(rev):\n"},{"id":"217633","messageId":"7vbo8a6b9c.fsf@alter.siamese.dyndns.org","threadId":"33839","inReplyTo":"7v4ne27ubp.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T18:16:31Z","receivedAt":"2013-05-16T18:16:31Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Junio C Hamano writes:\n\n> Felipe Contreras writes:\n>\n>> This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n>>\n>> Different versions of Mercurial have different arguments for\n>> bookmarks.updatefromremote(), while it should be possible to call the\n>> right function with the right arguments depending on the version, it's\n>> safer to restore the old behavior for now.\n>>\n>> Reported by Rodney Lorrimar.\n>>\n>> Signed-off-by: Felipe Contreras \n>> ---\n>>\n>> Intended for master (v1.8.3).\n>\n> Hmm, is this the one we merged yesterday?\n\nJust double-checking.\n\nThis is a lot more urgent request for confirmation than anything\nelse I've been sending today (and receiving responses from you\nabout).\n\n>\n>>\n>> contrib/remote-helpers/git-remote-hg | 3 ---\n>> 1 file changed, 3 deletions(-)\n>>\n>> diff --git a/contrib/remote-helpers/git-remote-hg b/contrib/remote-helpers/git-remote-hg\n>> index dc276af..beb864b 100755\n>> --- a/contrib/remote-helpers/git-remote-hg\n>> +++ b/contrib/remote-helpers/git-remote-hg\n>> @@ -363,9 +363,6 @@ def get_repo(url, alias):\n>> die('Repository error')\n>> repo.pull(peer, heads=None, force=True)\n>> \n>> - rb = peer.listkeys('bookmarks')\n>> - bookmarks.updatefromremote(myui, repo, rb, url)\n>> -\n>> return repo\n>> \n>> def rev_to_mark(rev):\n"},{"id":"217651","messageId":"7v1u964jsn.fsf@alter.siamese.dyndns.org","threadId":"33839","inReplyTo":"1368708195-1044-1-git-send-email-felipe.contreras@gmail.com","subject":"Re: [PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T22:55:04Z","receivedAt":"2013-05-16T22:55:04Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras writes:\n\n> This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n>\n> Different versions of Mercurial have different arguments for\n> bookmarks.updatefromremote(), while it should be possible to call the\n> right function with the right arguments depending on the version, it's\n> safer to restore the old behavior for now.\n>\n> Reported by Rodney Lorrimar.\n>\n> Signed-off-by: Felipe Contreras \n> ---\n>\n> Intended for master (v1.8.3).\n\nOK, will revert before I start today's final integration run.\n\nThanks.\n"},{"id":"217653","messageId":"CAMP44s3HKgWE=8SU+aBMBWzwe-fQ8Rjf0yGmGQxcdedhYCNJmQ@mail.gmail.com","threadId":"33839","inReplyTo":"7vbo8a6b9c.fsf@alter.siamese.dyndns.org","subject":"Re: [PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Felipe Contreras","fromEmail":"felipe.contreras@gmail.com","sentAt":"2013-05-16T23:04:41Z","receivedAt":"2013-05-16T23:04:41Z","isPatch":true,"sender":{"key":"felipe.contreras@gmail.com","avatar":"https://avatars.githubusercontent.com/u/8358?v=4"},"body":"On Thu, May 16, 2013 at 1:16 PM, Junio C Hamano wrote:\n> Junio C Hamano writes:\n>\n>> Felipe Contreras writes:\n>>\n>>> This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n>>>\n>>> Different versions of Mercurial have different arguments for\n>>> bookmarks.updatefromremote(), while it should be possible to call the\n>>> right function with the right arguments depending on the version, it's\n>>> safer to restore the old behavior for now.\n>>>\n>>> Reported by Rodney Lorrimar.\n>>>\n>>> Signed-off-by: Felipe Contreras \n>>> ---\n>>>\n>>> Intended for master (v1.8.3).\n>>\n>> Hmm, is this the one we merged yesterday?\n>\n> Just double-checking.\n\nYes.\n\n-- \nFelipe Contreras\n"},{"id":"217657","messageId":"7vsj1m34bj.fsf@alter.siamese.dyndns.org","threadId":"33839","inReplyTo":"CAMP44s3HKgWE=8SU+aBMBWzwe-fQ8Rjf0yGmGQxcdedhYCNJmQ@mail.gmail.com","subject":"Re: [PATCH] Revert \"remote-hg: update bookmarks when pulling\"","fromName":"Junio C Hamano","fromEmail":"gitster@pobox.com","sentAt":"2013-05-16T23:14:40Z","receivedAt":"2013-05-16T23:14:40Z","isPatch":true,"sender":{"key":"gitster@pobox.com","avatar":"https://avatars.githubusercontent.com/u/54884?v=4"},"body":"Felipe Contreras writes:\n\n> On Thu, May 16, 2013 at 1:16 PM, Junio C Hamano wrote:\n>> Junio C Hamano writes:\n>>\n>>> Felipe Contreras writes:\n>>>\n>>>> This reverts commit 24317ef32ac3111ed00792f9b2921dc19dd28fe2.\n>>>>\n>>>> Different versions of Mercurial have different arguments for\n>>>> bookmarks.updatefromremote(), while it should be possible to call the\n>>>> right function with the right arguments depending on the version, it's\n>>>> safer to restore the old behavior for now.\n>>>>\n>>>> Reported by Rodney Lorrimar.\n>>>>\n>>>> Signed-off-by: Felipe Contreras \n>>>> ---\n>>>>\n>>>> Intended for master (v1.8.3).\n>>>\n>>> Hmm, is this the one we merged yesterday?\n>>\n>> Just double-checking.\n>\n> Yes.\n\nOK. Thanks for keeping an eye on real user reports.\n\nPlease relay our thanks to Rodney as well.\n"}]}