All of lore.kernel.org
 help / color / mirror / Atom feed
From: Florian Achleitner <florian.achleitner.2.6.31@gmail.com>
To: Junio C Hamano <gitster@pobox.com>
Cc: David Michael Barr <b@rr-dav.id.au>,
	Jonathan Nieder <jrnieder@gmail.com>,
	git@vger.kernel.org,
	Florian Achleitner <florian.achleitner.2.6.31@gmail.com>
Subject: Re: [PATCH/RFC v4 01/16] GSOC remote-svn
Date: Sat, 18 Aug 2012 20:06:14 +0200	[thread overview]
Message-ID: <8572000.QUVXl8yetS@flobuntu> (raw)
In-Reply-To: <7v4no0u950.fsf@alter.siamese.dyndns.org>

On Friday 17 August 2012 21:16:59 Junio C Hamano wrote:
> Comments from mentors and people interested in remote helpers?
> 
> I did minimum line wrapping, typofix and small compilation fixes
> and queued these on 'pu'; I think I saw one commit whose message
> I didn't quite get what it was trying to say, and another that was
> missing S-o-b (I left them untouched).

Should I provide a better version? I found the commit that I forgot to sign-
off, but I'm not sure which message you mean.

> 
> The result merged to 'pu' seems to fail 9020, by the way.

That's because contrib/svn-fe isn't built automatically if you call make in 
the toplevel dir. 
It dies with "fatal: Unable to find remote helper for 'svn'", because the 
helper is not built. We currently need to run make in contrib/svn-fe 
seperately.
That's a bit awkward.

Just checked how it works for svn-fe. It has a seperate test program (test-
svn-fe.c) which is in the toplevel dir and built here, while for svn-fe 
itself, it's the same as for remote-svn. 

Don't know what to do about that.

> 
> Thanks.

  reply	other threads:[~2012-08-18 18:07 UTC|newest]

Thread overview: 26+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2012-08-17 20:25 [PATCH/RFC v4 01/16] GSOC remote-svn Florian Achleitner
2012-08-17 20:25 ` [PATCH/RFC v4 01/16] Implement a remote helper for svn in C Florian Achleitner
2012-08-17 20:25   ` [PATCH/RFC v4 02/16] Integrate remote-svn into svn-fe/Makefile Florian Achleitner
2012-08-17 20:25     ` [PATCH/RFC v4 03/16] Add svndump_init_fd to allow reading dumps from arbitrary FDs Florian Achleitner
2012-08-17 20:25       ` [PATCH/RFC v4 04/16] Add argv_array_detach and argv_array_free_detached Florian Achleitner
2012-08-17 20:25         ` [PATCH/RFC v4 05/16] Connect fast-import to the remote-helper via pipe, adding 'bidi-import' capability Florian Achleitner
2012-08-17 20:25           ` [PATCH/RFC v4 06/16] Add documentation for the 'bidi-import' capability of remote-helpers Florian Achleitner
2012-08-17 20:25             ` [PATCH/RFC v4 07/16] When debug==1, start fast-import with "--stats" instead of "--quiet" Florian Achleitner
2012-08-17 20:25               ` [PATCH/RFC v4 08/16] remote-svn, vcs-svn: Enable fetching to private refs Florian Achleitner
2012-08-17 20:25                 ` [PATCH/RFC v4 09/16] Allow reading svn dumps from files via file:// urls Florian Achleitner
2012-08-17 20:25                   ` [PATCH/RFC v4 10/16] vcs-svn: add fast_export_note to create notes Florian Achleitner
2012-08-17 20:25                     ` [PATCH/RFC v4 11/16] Create a note for every imported commit containing svn metadata Florian Achleitner
2012-08-17 20:25                       ` [PATCH/RFC v4 12/16] remote-svn: Activate import/export-marks for fast-import Florian Achleitner
2012-08-17 20:25                         ` [PATCH/RFC v4 13/16] remote-svn: add incremental import Florian Achleitner
2012-08-17 20:25                           ` [PATCH/RFC v4 14/16] Add a svnrdump-simulator replaying a dump file for testing Florian Achleitner
2012-08-17 20:25                             ` [PATCH/RFC v4 15/16] remote-svn: add marks-file regeneration Florian Achleitner
2012-08-17 20:25                               ` [PATCH/RFC v4 16/16] Add a test script for remote-svn Florian Achleitner
2012-08-18  4:16 ` [PATCH/RFC v4 01/16] GSOC remote-svn Junio C Hamano
2012-08-18 18:06   ` Florian Achleitner [this message]
2012-08-18 20:13     ` Junio C Hamano
2012-08-19  6:35       ` Junio C Hamano
2012-08-20  6:21         ` Florian Achleitner
2012-08-20  6:51           ` Junio C Hamano
2012-08-20 13:52       ` Florian Achleitner
2012-08-20 16:23         ` Junio C Hamano
2012-08-20 16:43         ` Jonathan Nieder

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=8572000.QUVXl8yetS@flobuntu \
    --to=florian.achleitner.2.6.31@gmail.com \
    --cc=b@rr-dav.id.au \
    --cc=git@vger.kernel.org \
    --cc=gitster@pobox.com \
    --cc=jrnieder@gmail.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.