[PATCH] Fixed typo in Doc/gitrepository-layout

From: LDVSOFT gitrepository-layout.txt: In description of `info' directory, note about `$GIT_COMMON_DIR' was referencing `index' instead of `info'. Signed-off-by: LDVSOFT --- Documentation/gitrepository-layout.txt | 2 +- 1 file changed, 1

Re: [PATCH v8 4/4] worktree: add 'list' command

Michael Rappazzo writes: > > +--porcelain:: > + With `list`, output in an easy-to-parse format for scripts. > + This format will remain stable across Git versions and regardless of > user > + configuration. ... and exactly what does it output? That would be

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Tue, Sep 22, 2015 at 12:45:51PM -0700, Junio C Hamano wrote: > Jacob Keller writes: > > > I don't think this patch actually changes behavior as it stands now. I > > think Junio's suggestion does. Personally, I'd prefer some sort of > > warning when you use xread and

Re: Unable to create temporary file '/var/git/tmv3-target-overlay.git/shallow_Un8ZOR': Permission denied

Hi Joakim, On 2015-09-21 19:08, Joakim Tjernlund wrote: > On Mon, 2015-09-21 at 09:48 -0700, Junio C Hamano wrote: >> Duy Nguyen writes: >> >> > Is it really necessary to remove write access in $GIT_DIR? Do we (git >> > devs) have some guidelines about things in $GIT_DIR? >>

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

Lars Schneider writes: > This works. OK, and thanks; as I don't do perforce, the squash was without any testing. > Do we need the “-e” option? In syntactic sense, no, but our codebase tends to prefer to have one, because it is easier to spot which ones are the

Re: [PATCH] filter-branch: add passed/remaining seconds on progress

Gabor Bernat writes: > On Mon, Sep 21, 2015 at 11:24 PM, Gábor Bernát > ... >> Agreed, :) did not abandoned this, just got caught up with many stuff. >> Thanks for the help, > > So do I need to do anything else with this? :) If you can fetch from me to see if the output

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On 22.09.15 08:23, Jacob Keller wrote: > On Mon, Sep 21, 2015 at 9:55 PM, Torsten Bögershausen wrote: >> But in any case I suggest to xread() as it is, and not to change the >> functionality >> behind the back of the users. >> >> > > I don't think this patch actually changes

Re: Specifying N revisions after the initial commit

On Tue, 22 Sep 2015 14:32:19 -0400 Josh Boyer wrote: > Please CC me as I'm not subscribed. > > I was hoping someone could help me with the revision shorthand to get > the commit sha of a commit N commits after the initial commit. What happens if right after the initial

Re: [PATCH] hooks/update: Add a hooks.denyunsignedtags option

On Sat, Sep 12, 2015 at 12:37:33PM +0200, Julian Andres Klode wrote: > Introduce an option to deny unsigned tags from entering > a repository. This is useful in teams where members forget > to sign their release tags. > > It does not actually check whether the signature is actually > complete or

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

On Tue, Sep 22, 2015 at 11:37 AM, Junio C Hamano wrote: > Jacob Keller writes: > >> The other issue here is that expand_notes_ref is called on the --ref >> argument waaay before the current code even decides if the operation >> is "read" or "write".

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

Stefan Beller writes: > So how would you find out when we are done? while (1) { if (we have available slot) ask to start a new one; if (nobody is running anymore) break;

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

Jacob Keller writes: > The other issue here is that expand_notes_ref is called on the --ref > argument waaay before the current code even decides if the operation > is "read" or "write". Thus we'd have to break this out and handle > things very differently. I think you

git-svn aborts with "Use of uninitialized value $u" when a non-svn-backed branch is present in remote

I found a specific case in which git-svn improperly aborts: 1. I created a git-svn repository, named "git-svn repo", by cloned an svn repository via the git-svn tool. 2. I created a normal git repository, named "configuration repo". I renamed the master branch to "configuration". The initial

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

Yup, this was privately reported and I just squashed a fix in right now ;-) Thanks. "cd t && make test-lint" would have caught it. On Tue, Sep 22, 2015 at 12:11 PM, Michael Blume wrote: > I'm seeing test failures > > non-executable tests:

Re: Specifying N revisions after the initial commit

Josh Boyer writes: > On Tue, Sep 22, 2015 at 2:40 PM, Konstantin Khomoutov > ... >> Hence, given any particular commit, you're able to trace all of its >> ancestry, but the reverse is not possible. > > That makes sense. I suppose I will have to resort to parsing output > of

Re: [PATCHv3 02/13] xread: poll on non blocking fds

Jeff King writes: > On Tue, Sep 22, 2015 at 12:45:51PM -0700, Junio C Hamano wrote: > >> One caveat is that the caller may not know in the first place. >> >> The last time I checked the existing callers of xread(), there were >> a few that read from a file descriptor they did not

Specifying N revisions after the initial commit

Hi All, Please CC me as I'm not subscribed. I was hoping someone could help me with the revision shorthand to get the commit sha of a commit N commits after the initial commit. Thus far I've figured out that to get the initial commit in a repository, you can use: git rev-list --max-parents=0

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Tue, Sep 22, 2015 at 11:21 AM, Junio C Hamano wrote: > Stefan Beller writes: > >> On Tue, Sep 22, 2015 at 8:58 AM, Junio C Hamano wrote: >>> Eric Sunshine writes: >>> > while (1) { >

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

On Mon, Sep 21, 2015 at 6:08 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> +void default_start_failure(void *data, >> +struct child_process *cp, >> +struct strbuf *err) >> +{ >> + int i; >> +

Re: Specifying N revisions after the initial commit

On Tue, Sep 22, 2015 at 2:40 PM, Konstantin Khomoutov wrote: > On Tue, 22 Sep 2015 14:32:19 -0400 > Josh Boyer wrote: > >> Please CC me as I'm not subscribed. >> >> I was hoping someone could help me with the revision shorthand to get >> the commit sha of

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

I'm seeing test failures non-executable tests: t9825-git-p4-handle-utf16-without-bom.sh ls -l shows that all the other tests are executable but t9825 isn't. On Tue, Sep 22, 2015 at 9:02 AM, Junio C Hamano wrote: > Lars Schneider writes: > >> This

Re: [PATCHv3 02/13] xread: poll on non blocking fds

Jacob Keller writes: > I don't think this patch actually changes behavior as it stands now. I > think Junio's suggestion does. Personally, I'd prefer some sort of > warning when you use xread and get EAGAIN or EWOULDBLOCK. I'd rather > see it somehow warn so that we can

Re: [PATCHv3 03/13] xread_nonblock: add functionality to read from fds nonblockingly

On Mon, Sep 21, 2015 at 3:39 PM, Stefan Beller wrote: Maybe change the title to "without blocking" instead of "nonblockingly"? Regards, Jake -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message to majord...@vger.kernel.org More majordomo

Re: [PATCHv3 03/13] xread_nonblock: add functionality to read from fds nonblockingly

On Mon, Sep 21, 2015 at 5:10 PM, Junio C Hamano wrote: >> This wrapper just restarts on EINTR, but not on EAGAIN like xread >> does. This gives less guarantees on the actual reading rather than >> on returning fast. > > The last sentence is a bit hard to parse, by the way. >

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

On Wed, Sep 16, 2015 at 3:34 PM, Junio C Hamano wrote: > Jacob Keller writes: > >> From: Jacob Keller >> >> The documentation for --refs says that it will treat unqualified refs as >> under refs/notes. Current behavior is to

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Mon, Sep 21, 2015 at 9:55 PM, Torsten Bögershausen wrote: > But in any case I suggest to xread() as it is, and not to change the > functionality > behind the back of the users. > > I don't think this patch actually changes behavior as it stands now. I think Junio's suggestion

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

On Tue, Sep 22, 2015 at 7:17 AM, Junio C Hamano wrote: > Calls expand_notes_ref() made on a command line argument that > specifies the source (which I think is similar to what the other > recent topic calls "read-only") should be made to calls to a more > lenient version (and

Re: [PATCHv3 07/13] fetch_populated_submodules: use new parallel job processing

Stefan Beller writes: > In a later patch we enable parallel processing of submodules, this > only adds the possibility for it. So this change should not change > any user facing behavior. > > Signed-off-by: Stefan Beller > --- > builtin/fetch.c | 3 +-

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Tue, Sep 22, 2015 at 8:58 AM, Junio C Hamano wrote: > Eric Sunshine writes: > >>> while (1) { >>> nr = read(fd, buf, len); >>> - if ((nr < 0) && (errno == EAGAIN || errno == EINTR)) >>> -

Re: [PATCHv3 02/13] xread: poll on non blocking fds

Stefan Beller writes: > On Tue, Sep 22, 2015 at 8:58 AM, Junio C Hamano wrote: >> Eric Sunshine writes: >> while (1) { nr = read(fd, buf, len); - if ((nr < 0) && (errno ==

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

Lars Schneider writes: > If it is no extra work for you, you can remove the quotes around > “db”. I can also create a new patch roll including the sed change > and the quote change if it is easier for you. Now you've tested the SQUASH??? for me, I can just squash that

Re: [PATCH v8 1/4] worktree: add top-level worktree.c

Michael Rappazzo writes: > worktree.c contains functions to work with and get information from > worktrees. This introduction moves functions related to worktrees > from branch.c into worktree.c > > Signed-off-by: Michael Rappazzo > --- > Makefile

Re: [PATCH v8 2/4] worktree: refactor find_linked_symref function

Michael Rappazzo writes: > Refactoring will help transition this code to provide additional useful > worktree functions. > > Signed-off-by: Michael Rappazzo > --- > worktree.c | 86 > ++ > 1

Re: [PATCH v8 3/4] worktree: add functions to get worktree details

Michael Rappazzo writes: > +/** > + * get the main worktree > + */ > +static struct worktree *get_main_worktree() static struct worktree *get_main_worktree(void) > +{ > + struct worktree *worktree = NULL; > struct strbuf path = STRBUF_INIT; > + struct strbuf

Re: Specifying N revisions after the initial commit

Theodore Ts'o writes: > On Tue, Sep 22, 2015 at 04:11:23PM -0400, Josh Boyer wrote: >> Oh, context would help, yes. In the case of the tree I'm parsing, I >> know for a fact that the commit history is entirely linear and will >> (should) always remain so. E.g. >> >> A - B - C -

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

On Tue, Sep 22, 2015 at 12:53 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> So how would you find out when we are done? > > while (1) { > if (we have available slot) > ask to start a new one; >

Re: Unable to create temporary file '/var/git/tmv3-target-overlay.git/shallow_Un8ZOR': Permission denied

On Tue, 2015-09-22 at 22:00 +0200, Johannes Schindelin wrote: > Hi Joakim, > > On 2015-09-21 19:08, Joakim Tjernlund wrote: > > On Mon, 2015-09-21 at 09:48 -0700, Junio C Hamano wrote: > > > Duy Nguyen writes: > > > > > > > Is it really necessary to remove write access in

[PATCHv4 04/14] strbuf: add strbuf_read_once to read without blocking

The new call will read from a file descriptor into a strbuf once. The underlying call xread_nonblock is meant to execute without blocking if the file descriptor is set to O_NONBLOCK. It is a bug to call strbuf_read_once on a file descriptor which would block. Signed-off-by: Stefan Beller

[PATCHv4 01/14] submodule: Send "Fetching submodule " to standard error

From: Jonathan Nieder The "Pushing submodule " progress output correctly goes to stderr, but "Fetching submodule " is going to stdout by mistake. Fix it to write to stderr. Noticed while trying to implement a parallel submodule fetch. When this particular output line went

[PATCHv4 03/14] xread_nonblock: add functionality to read from fds without blocking

Provide a wrapper to read(), similar to xread(), that restarts on EINTR but not EAGAIN (or EWOULDBLOCK). This enables the caller to handle polling itself, possibly polling multiple sockets or performing some other action. Helped-by: Jacob Keller Helped-by: Jeff King

[PATCHv4 08/14] submodules: allow parallel fetching, add tests and documentation

This enables the work of the previous patches. Signed-off-by: Stefan Beller --- Documentation/fetch-options.txt | 7 +++ builtin/fetch.c | 6 +- builtin/pull.c | 6 ++ submodule.c | 3 +-- submodule.h

[PATCHv4 00/14] fetch submodules in parallel and a preview on parallel "submodule update"

Today there was lots of discussion on the correct way of reading the strbufs as well as some discussion on the structure of the asynchronous parallel process loop. Patches 1-8 bring parallel fetching of submodules, and have had some good exposure to review and feedback is incorporated. Patches

[PATCHv4 12/14] git submodule update: cmd_update_clone

Split the cloning part out to its own function, this allow us in a later patch to convert cmd_update in C. Signed-off-by: Stefan Beller --- git-submodule.sh | 31 --- 1 file changed, 20 insertions(+), 11 deletions(-) diff --git a/git-submodule.sh

[PATCHv4 02/14] xread: poll on non blocking fds

>From the man page: EAGAIN The file descriptor fd refers to a file other than a socket and has been marked nonblocking (O_NONBLOCK), and the read would block. EAGAIN or EWOULDBLOCK The file descriptor fd refers to a socket and has been marked nonblocking (O_NONBLOCK),

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Tue, Sep 22, 2015 at 05:14:42PM -0700, Stefan Beller wrote: > We should not care if the call to poll failed, as we're in an infinite loop > and > can only get out with the correct read(..). So maybe an implementation like > this > would already suffice: > > ssize_t xread(int fd, void *buf,

[PATCHv4 09/14] submodule-config: Untangle logic in parse_config

CC: Eric Sunshine Signed-off-by: Stefan Beller --- submodule-config.c | 74 +- 1 file changed, 29 insertions(+), 45 deletions(-) diff --git a/submodule-config.c b/submodule-config.c index

[PATCHv4 14/14] Rewrite submodule update in C

This will make parallelisation easier in a followup patch. This is just a translation from shell to C, hopefully not introducing any bugs. Signed-off-by: Stefan Beller --- builtin/submodule--helper.c | 251 git-submodule.sh

[PATCHv4 06/14] run-command: add an asynchronous parallel child processor

This allows to run external commands in parallel with ordered output on stderr. If we run external commands in parallel we cannot pipe the output directly to the our stdout/err as it would mix up. So each process's output will flow through a pipe, which we buffer. One subprocess can be directly

[PATCHv4 11/14] git submodule update: cmd_update_recursive

Split the recursion part out to its own function, this allow us in a later patch to convert cmd_update in C. Signed-off-by: Stefan Beller --- git-submodule.sh | 47 ++- 1 file changed, 26 insertions(+), 21 deletions(-) diff --git

[PATCHv4 13/14] git submodule update: cmd_update_fetch

Split the fetching part out to its own function, this allow us in a later patch to convert cmd_update in C. Signed-off-by: Stefan Beller --- git-submodule.sh | 164 --- 1 file changed, 84 insertions(+), 80 deletions(-)

[PATCHv4 05/14] run-command: factor out return value computation

We will need computing the return value in a later patch without the wait. Signed-off-by: Stefan Beller --- run-command.c | 56 ++-- 1 file changed, 34 insertions(+), 22 deletions(-) diff --git a/run-command.c

[PATCHv4 10/14] submodule config: keep update strategy around

We need the submodule update strategies in a later patch. Signed-off-by: Stefan Beller --- submodule-config.c | 11 +++ submodule-config.h | 1 + 2 files changed, 12 insertions(+) diff --git a/submodule-config.c b/submodule-config.c index afe0ea8..8b8c7d1 100644

[PATCHv4 07/14] fetch_populated_submodules: use new parallel job processing

In a later patch we enable parallel processing of submodules, this only adds the possibility for it. So this change should not change any user facing behavior. Signed-off-by: Stefan Beller --- submodule.c | 119 +++- 1

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

Eric Wong wrote: > Victor Leschuk wrote: > > The thing is that git-cat-file keeps growing during work when running > > in "batch" mode. See the figure attached: it is for cloning a rather > > small repo (1 hour to clone about ~14000 revisions).

Re: [PATCHv3 02/13] xread: poll on non blocking fds

Stefan Beller writes: > We should not care if the call to poll failed, as we're in an infinite loop > and > can only get out with the correct read(..). I think I agree with that reasoning. The only thing we want out of this call to poll(2) is to delay calling read(2) again

[PATCH] notes: document behavior of --ref and --notes DWIMery

From: Jacob Keller The --notes and --ref parameter for selecting which notes ref to operate on are based off of expand_notes_ref functionality. The documentation mentioned that an unqualified ref argument would be taken as under `refs/notes/`. However, this does not

Re: [PATCH] notes: document behavior of --ref and --notes DWIMery

Jacob Keller writes: > From: Jacob Keller > > The --notes and --ref parameter for selecting which notes ref to operate > on are based off of expand_notes_ref functionality. The documentation > mentioned that an unqualified ref argument would be

Re: Specifying N revisions after the initial commit

On Tue, Sep 22, 2015 at 04:11:23PM -0400, Josh Boyer wrote: > Oh, context would help, yes. In the case of the tree I'm parsing, I > know for a fact that the commit history is entirely linear and will > (should) always remain so. E.g. > > A - B - C - D - E - F ... {N} > > So yes, finding e.g.

Re: [PATCH] notes: document behavior of --ref and --notes DWIMery

On Tue, 2015-09-22 at 13:40 -0700, Junio C Hamano wrote: > How about phrasing it totally differently? > > The ref specifies the full refname when it begins with > `refs/notes/`; otherwise `ref/notes/` is prefixed to form a > full name of the ref. > > I think that would remove

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

On Tue, Sep 22, 2015 at 2:41 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> On Tue, Sep 22, 2015 at 12:53 PM, Junio C Hamano wrote: >>> Stefan Beller writes: >>> So how would you find out when we are

Re: [PATCH v7 0/7] git-p4: add support for large file systems

larsxschnei...@gmail.com writes: > From: Lars Schneider > > diff to v6: > * fix spaces in .gitattributes for Git-LFS files (old "[:space:]", new > "[[:space:]]") > * generated patch on top of next (95c4325) to resolve merge conflicts > > I am sorry about the

Re: Specifying N revisions after the initial commit

On Tue, Sep 22, 2015 at 3:55 PM, Junio C Hamano wrote: > Josh Boyer writes: > >> On Tue, Sep 22, 2015 at 2:40 PM, Konstantin Khomoutov >> ... >>> Hence, given any particular commit, you're able to trace all of its >>> ancestry, but the reverse is not

Re: [PATCH] Makefile: Change library order for static linking

On Mon, Sep 21, 2015 at 10:09:52AM -0700, Junio C Hamano wrote: > I think I said this already, but I found these bits from your patch > > - $(LIBS) $(CURL_LIBCURL) $(EXPAT_LIBEXPAT) > + $(CURL_LIBCURL) $(EXPAT_LIBEXPAT) $(LIBS) > > that first uses libcurl and then libs

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

Stefan Beller writes: > On Tue, Sep 22, 2015 at 12:53 PM, Junio C Hamano wrote: >> Stefan Beller writes: >> >>> So how would you find out when we are done? >> >> while (1) { >> if (we have available slot) >>

Re: [PATCH v7 1/7] git-p4: add optional type specifier to gitConfig reader

larsxschnei...@gmail.com writes: > From: Lars Schneider > > The functions “gitConfig” and “gitConfigBool” are almost identical. Make > “gitConfig” more generic by adding an optional type specifier. Use the type > specifier “—bool” with “gitConfig” to implement

[PATCH v2] notes: correct documentation of DWIMery for notes references

From: Jacob Keller expand_notes_ref is used by --ref from git-notes(1) and --notes from the git log to find the full refname of a notes reference. Previously the documentation of these options was not clear about what sorts of expansions would be performed. Fix the

Re: [PATCH] notes: document behavior of --ref and --notes DWIMery

On Tue, 2015-09-22 at 13:40 -0700, Junio C Hamano wrote: > Jacob Keller writes: > > > From: Jacob Keller > > > > The --notes and --ref parameter for selecting which notes ref to > > operate > > on are based off of expand_notes_ref

Re: [PATCH] notes: document behavior of --ref and --notes DWIMery

"Keller, Jacob E" writes: > On Tue, 2015-09-22 at 13:40 -0700, Junio C Hamano wrote: >> How about phrasing it totally differently? >> >> The ref specifies the full refname when it begins with >> `refs/notes/`; otherwise `ref/notes/` is prefixed to form a >>

Re: [PATCH] Makefile: Change library order for static linking

Remi Pommarel writes: > I have one last question thought. Wouldn't it be nice if we had > configure to autodetect the need for -lssl with libcurl? Sure. I do not think anybody tried, but it would be nice. > We could make > configure to check for Curl_ssl_init function

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

Victor Leschuk wrote: > The thing is that git-cat-file keeps growing during work when running > in "batch" mode. See the figure attached: it is for cloning a rather > small repo (1 hour to clone about ~14000 revisions). However the clone > of a large repo (~28

t5561 failing after make PROFILE=GEN

Hi, I noticed that t5561 fails on my machine when compiling with "make PROFILE=GEN". Luckily, the reason seems to be the test only, not the tool it is testing. I tracked it down that far that log_div() (defined in t/t5561-http-backend.sh but used in t/t556x_common) appends the given text to the

Re: [PATCHv3 02/13] xread: poll on non blocking fds

On Tue, Sep 22, 2015 at 1:00 PM, Junio C Hamano wrote: > Jeff King writes: > >> On Tue, Sep 22, 2015 at 12:45:51PM -0700, Junio C Hamano wrote: >> >>> One caveat is that the caller may not know in the first place. >>> >>> The last time I checked the existing

Re: [PATCHv3 06/13] run-command: add an asynchronous parallel child processor

Stefan Beller writes: > On Tue, Sep 22, 2015 at 2:41 PM, Junio C Hamano wrote: >> Stefan Beller writes: >> >>> On Tue, Sep 22, 2015 at 12:53 PM, Junio C Hamano wrote: Stefan Beller writes:

Re: [PATCH] filter-branch: add passed/remaining seconds on progress

On Mon, Sep 21, 2015 at 11:24 PM, Gábor Bernát wrote: > On Mon, Sep 21, 2015 at 11:22 PM, Eric Sunshine > wrote: >> >> On Mon, Sep 21, 2015 at 3:52 PM, Junio C Hamano wrote: >> > Eric Sunshine

Re: [PATCHv3 03/13] xread_nonblock: add functionality to read from fds nonblockingly

Jacob Keller writes: > On Mon, Sep 21, 2015 at 3:39 PM, Stefan Beller wrote: > > Maybe change the title to "without blocking" instead of "nonblockingly"? Both suggestions make sense ;-) -- To unsubscribe from this list: send the line "unsubscribe

Re: [PATCHv3 02/13] xread: poll on non blocking fds

Eric Sunshine writes: >> while (1) { >> nr = read(fd, buf, len); >> - if ((nr < 0) && (errno == EAGAIN || errno == EINTR)) >> - continue; >> + if (nr < 0) { >> + if (errno ==

Re: git-svn: Why not use git-fast-import?

On Tue, Sep 22, 2015 at 02:12:59AM -0700, Victor Leschuk wrote: > I've been playing with git-svn for some time already and as it seems > to me there are two most important problems which make it hard to use > in production for large repositories. Very low speed and large memory > footprint of

Small mistakes in translation

Hi, On page => https://git-scm.com/book/pl/v1/Rozproszony-Git-Rozproszone-przep%C5%82ywy-pracy < I found typographical error in first line (Is: powala => Should be: pozwala) in third part of text entitle "Przepływ pracy z osobą integrującą zmiany". And next line, it means third line at the bottom

git-svn: Why not use git-fast-import?

Hello all, I've been playing with git-svn for some time already and as it seems to me there are two most important problems which make it hard to use in production for large repositories. Very low speed and large memory footprint of synchronization with SVN repos (I am talking about clone and

Re: [PATCHv3 04/13] strbuf: add strbuf_read_once to read without blocking

On Mon, Sep 21, 2015 at 5:17 PM, Junio C Hamano wrote: > Stefan Beller writes: > >> The new call will read a fd into a strbuf once. The underlying call > > "read from a fd" > >> xread_nonblock is meant to execute non blockingly if the fd is set to >>

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

On 22 Sep 2015, at 03:10, Junio C Hamano wrote: > Eric Sunshine writes: > >> Yes, it's because $d is a variable reference, even within double >> quotes. > > s/even/especially/ ;-) > > Here is what I queued as SQUASH??? > > diff --git

Re: Small mistakes in translation

Thanks Ray, Ray Chen gmail.com> writes: > > On Tue, 22 Sep 2015, Przemysław Skrzyniarz wrote: > > The issues you found should better be reported to the progit community > on GitHub: https://github.com/progit/progit Next message, it means fourth I will send correctly group (of course and I'll

Re: Small mistakes in translation

On Tue, 22 Sep 2015, Przemysław Skrzyniarz wrote: > Hi, > > On page => > https://git-scm.com/book/pl/v1/Rozproszony-Git-Rozproszone-przep%C5%82ywy-pracy > < I found typographical error in first line (Is: powala => Should be: > pozwala) in third part of text entitle "Przepływ pracy z osobą

Re: [PATCH v4 1/2] git-p4: add test case for "Translation of file content failed" error

On 22 Sep 2015, at 01:54, Eric Sunshine wrote: > On Mon, Sep 21, 2015 at 7:03 PM, Lars Schneider > wrote: >> On 21 Sep 2015, at 20:09, Junio C Hamano wrote: >>> larsxschnei...@gmail.com writes: +test_expect_success

Re: Small mistakes in translation

I'm found one more mistake on page => https://git-scm.com/book/pl/v1/Rozproszony-Git-Rozproszone-przep%C5%82ywy-pracy < In third line, before next to last line (Is: tryby => Should be: trybu) in fourth part of text entitle "Przepływ pracy z dyktatorem i porucznikami"(place in line - cit.: "... go

RE: [PATCH] git-svn: make batch mode optional for git-cat-file

As for your remark regarding the option naming: > An option whose name begins with no- looks somewhat strange. You can even say --no-no-cat-file-batch from the command line, I suspect. We already do have some of these: 'no-metadata', 'no-checkout', 'no-auth-cache'. So I was just following the

Re: [PATCH] git-svn: make batch mode optional for git-cat-file

Victor Leschuk writes: > We already do have some of these: 'no-metadata', 'no-checkout', > no-auth-cache'. So I was just following the existing convention. Do > you think we need to change it and stick with > --catch-file-batch=1/--cat-file-batch=0 ? Inventing a new

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

Jacob Keller writes: > I never got any better suggestion on how to allow the behavior > desired, which is to enable merging from a non-notes location, in > order to provide a standard location for remote notes, ie: > refs/remote-notes// Step back a bit and think again.

Re: [PATCH 1/2] notes: don't expand qualified refs in expand_notes_ref

On Tue, Sep 22, 2015 at 7:17 AM, Junio C Hamano wrote: > The current code before your patch limits the allowed pair of notes > trees by insisting that both appear as the tips of refs somewhere in > refs/notes/*. For allowing to merge from outside refs/notes/, you > need to

Re: [PATCHv3 09/13] submodule config: keep update strategy around

On Mon, Sep 21, 2015 at 5:56 PM, Eric Sunshine wrote: > On Mon, Sep 21, 2015 at 6:39 PM, Stefan Beller wrote: >> We need the submodule update strategies in a later patch. >> >> Signed-off-by: Stefan Beller >> --- >> diff --git