Re: [PATCH 0/9] transport-helper: updates

2013-09-22 Thread Richard Hansen
On 2013-08-29 11:23, Felipe Contreras wrote:
 Here are the patches that allow transport helpers to be completely 
 transparent;
 renaming branches, deleting them, custom refspecs, --force, --dry-run,
 reporting forced update, everything works.

What is the status of these patches?

I would like to be able to 'git push old:new' and 'git push -f' to a
bzr repo, and these are a prerequisite.  I imagine changes to
git-remote-bzr will also be required; I'm willing write up such
changes to git-remote-bzr, but only if there's interest in picking up
this patch series.

Thanks,
Richard
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/9] transport-helper: updates

2013-09-22 Thread Felipe Contreras
Richard Hansen wrote:
 On 2013-08-29 11:23, Felipe Contreras wrote:
  Here are the patches that allow transport helpers to be completely 
  transparent;
  renaming branches, deleting them, custom refspecs, --force, --dry-run,
  reporting forced update, everything works.
 
 What is the status of these patches?

Ask Junio. All the valid feedback has been addressed, there's no reason not to
apply them.

 I would like to be able to 'git push old:new' and 'git push -f' to a
 bzr repo, and these are a prerequisite.  I imagine changes to
 git-remote-bzr will also be required; I'm willing write up such
 changes to git-remote-bzr, but only if there's interest in picking up
 this patch series.

There's no need to modify git-remote-bzr, if theese patches are applied,
old:new would work for all remote-helpers.

-- 
Felipe Contreras
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


[PATCH 0/9] transport-helper: updates

2013-08-29 Thread Felipe Contreras
Hi,

Here are the patches that allow transport helpers to be completely transparent;
renaming branches, deleting them, custom refspecs, --force, --dry-run,
reporting forced update, everything works.

Some of these were were sent before and rejected without a reason, but here
they are again in case anybody is interested.

Felipe Contreras (9):
  transport-helper: add 'force' to 'export' helpers
  transport-helper: check for 'forced update' message
  fast-export: improve argument parsing
  fast-export: add new --refspec option
  transport-helper: add support for old:new refspec
  fast-import: add support to delete refs
  fast-export: add support to delete refs
  transport-helper: add support to delete branches
  transport-helper: don't update refs in dry-run

 Documentation/git-fast-export.txt |  4 
 Documentation/git-fast-import.txt |  3 +++
 builtin/fast-export.c | 47 ++-
 fast-import.c | 13 ---
 t/t5801-remote-helpers.sh | 10 -
 t/t9300-fast-import.sh| 18 +++
 t/t9350-fast-export.sh| 18 +++
 transport-helper.c| 42 --
 8 files changed, 138 insertions(+), 17 deletions(-)

-- 
1.8.4-fc

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/9] transport-helper: updates

2013-08-29 Thread Junio C Hamano
Felipe Contreras felipe.contre...@gmail.com writes:

 Some of these were were sent before and rejected without a
 reason,...

Earlier, you were asked to leave because many constructive criticism
or suggestion for improvement to your patches in reviews led to
stubborn resistance, ended up wasting everybody's time and left a
bad atmosphere on the list.

It seems that Matthew is trying to see if you can work better with
others than before after a break, but I personally am not hopeful
yet and do not want to waste my/our time on flamewars like we saw in
the past.

So I'll look at these and may pick patches that are obviously and
trivially good, but will refrain from asking clarifications and
suggesting improvements, etc. on other patches that I suspect will
end up wasting the time of people again.  Do not take silence from
me as a silent agreement.

--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html


Re: [PATCH 0/9] transport-helper: updates

2013-08-29 Thread Felipe Contreras
On Thu, Aug 29, 2013 at 2:22 PM, Junio C Hamano gits...@pobox.com wrote:
 Felipe Contreras felipe.contre...@gmail.com writes:

 Some of these were were sent before and rejected without a
 reason,...

 Earlier, you were asked to leave because many constructive criticism
 or suggestion for improvement to your patches in reviews led to
 stubborn resistance, ended up wasting everybody's time and left a
 bad atmosphere on the list.

 It seems that Matthew is trying to see if you can work better with
 others than before after a break, but I personally am not hopeful
 yet and do not want to waste my/our time on flamewars like we saw in
 the past.

 So I'll look at these and may pick patches that are obviously and
 trivially good, but will refrain from asking clarifications and
 suggesting improvements, etc. on other patches that I suspect will
 end up wasting the time of people again.  Do not take silence from
 me as a silent agreement.

If you do not pick these patches, the only one that looses is the Git
project and its users.

-- 
Felipe Contreras
--
To unsubscribe from this list: send the line unsubscribe git in
the body of a message to majord...@vger.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html