Re: [PATCH v2 11/11] ref-filter: introduce objectname_atom_parser()

2015-12-17 Thread Eric Sunshine
On Wed, Dec 16, 2015 at 10:30 AM, Karthik Nayak wrote: > Introduce objectname_atom_parser() which will parse the > '%(objectname)' atom and store information into the 'used_atom' > structure based on the modifiers used along with the atom. > > Helped-by: Ramsay Jones > Signed-off-by: Karthik Naya

Re: [PATCH v2 07/11] ref-filter: introduce align_atom_parser()

2015-12-17 Thread Eric Sunshine
On Wed, Dec 16, 2015 at 10:29 AM, Karthik Nayak wrote: > Introduce align_atom_parser() which will parse an "align" atom and > store the required alignment position and width in the "use_atom" > structure for further usage in 'populate_value()'. > > Helped-by: Ramsay Jones > Signed-off-by: Karthik

Re: [PATCH 02/16] refs: add methods for misc ref operations

2015-12-17 Thread Howard Chu
Junio C Hamano pobox.com> writes: > > David Turner twopensource.com> writes: > > > struct ref_be { > > struct ref_be *next; > > const char *name; > > ref_transaction_commit_fn *transaction_commit; > > + > > + pack_refs_fn *pack_refs; > > + peel_ref_fn *peel_ref; > > + create

Re: [RFC] Malicously tampering git metadata?

2015-12-17 Thread Jeff King
On Tue, Dec 15, 2015 at 10:26:39PM -0500, Santiago Torres wrote: > An example of a malicious commit merge follows: > > 1) The attacker controlling or acting as the upstream server identifies > two branches: one in which the unsuspecting developer is working on, and > another in which a vulnerable

Re: [RFC] Malicously tampering git metadata?

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 08:06:36PM -0500, Santiago Torres wrote: > > This is what push certificates ought to solve. > > The server records all pushes and its signed certificates of pushes > > and by the difference of the > > refs (either in packed refs or as a loose ref) to the push certificate >

Re: [PATCHv2] submodule: Port resolve_relative_url from shell to C

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 07:55:43PM +0100, Johannes Sixt wrote: > -static int has_same_dir_prefix(const char *str, const char **out) > +static int starts_with_dot_slash(const char *str) > { > -#ifdef GIT_WINDOWS_NATIVE > - return skip_prefix(str, "./", out) > - || skip_prefix(str,

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 12:51:08PM -0800, Stefan Beller wrote: > > poll() returns -1; errno = EAGAIN /* poll failed. If the fd was OK, the > > failure may be temporaly, > > as much as poll() can see. > > But most probably we

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 09:42:01PM +0100, Torsten Bögershausen wrote: > > Or do you mean to insert another continue in here? > I was thinking that we run into similar loop as before: > read() returns -1; errno = EAGAIN /* No data to read */ > poll() returns -1; errno = EAGAIN /* poll failed. If t

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Jeff King
On Tue, Dec 15, 2015 at 04:04:07PM -0800, Stefan Beller wrote: > If we get an EAGAIN or EWOULDBLOCK the fd must have set O_NONBLOCK. > As the intent of xread is to read as much as possible either until the > fd is EOF or an actual error occurs, we can ease the feeder of the fd > by not spinning th

Re: RefTree: Alternate ref backend

2015-12-17 Thread Mike Hommey
On Thu, Dec 17, 2015 at 02:28:01PM -0800, Shawn Pearce wrote: > On Thu, Dec 17, 2015 at 2:10 PM, Jeff King wrote: > > On Thu, Dec 17, 2015 at 01:02:50PM -0800, Shawn Pearce wrote: > > > >> I started playing around with the idea of storing references directly > >> in Git. Exploiting the GITLINK tre

-C option with pull doesn't seem to be respected in aliases in git 2.6.4.

2015-12-17 Thread Cameron Esfahani
I have git project checked out at ~/llvm. Inside of there, inside of a “tools” directory, I have another project checked out as “lldb”: ~/llvm/tools/lldb I wrote an alias which would help me update all my projects: all = !find . -type d -name .git | sed 's:/.git::' | xargs -I{} -t git

Re: [RFC] Malicously tampering git metadata?

2015-12-17 Thread Santiago Torres
Hi Stefan, thanks for the insight. > This is what push certificates ought to solve. > The server records all pushes and its signed certificates of pushes > and by the difference of the > refs (either in packed refs or as a loose ref) to the push certificate > this tampering of > the server can be

Re: [PATCH v3] git-svn: add support for prefixed globs in config

2015-12-17 Thread Eric Wong
Victor Leschuk wrote: > --- /dev/null > +++ b/t/t9168-git-svn-prefixed-glob.sh > @@ -0,0 +1,136 @@ > +#!/bin/sh > +test_description='git svn globbing refspecs with prefixed globs' > +. ./lib-git-svn.sh > + > +cat >expect.end < +the end > +hi > +start a new branch > +initial > +EOF > + > +test_expe

Re: [PATCH v3 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Philip Oakley
From: "Junio C Hamano" Philip Oakley writes: The gui.recentrepo list may be longer than the maxrecent setting. Use the actual length determined earlier, now saved as $lenrecent. In an ideal world, the git gui would limit the number of entries to the maxrecent setting, however the recentrepo

Re: Git issue report : issue with capital letter in folder name

2015-12-17 Thread PFDuc
Le 2015-12-17 13:29, Stefan Beller a écrit : On Thu, Dec 17, 2015 at 7:45 AM, PFDuc wrote: Hello, first of all thank you for developping git ! I had an issue with a capital block in the folder name inside my git repo. The folder in my local was named "Display" and the one at origin was named

Re: [PATCH v3 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Junio C Hamano
Philip Oakley writes: > The gui.recentrepo list may be longer than the maxrecent setting. > Use the actual length determined earlier, now saved as $lenrecent. > > In an ideal world, the git gui would limit the number of entries > to the maxrecent setting, however the recentrepo config list may >

[PATCH v3 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Philip Oakley
The gui.recentrepo list may be longer than the maxrecent setting. Use the actual length determined earlier, now saved as $lenrecent. In an ideal world, the git gui would limit the number of entries to the maxrecent setting, however the recentrepo config list may have been extended outside of the g

Re: RefTree: Alternate ref backend

2015-12-17 Thread Shawn Pearce
On Thu, Dec 17, 2015 at 2:10 PM, Jeff King wrote: > On Thu, Dec 17, 2015 at 01:02:50PM -0800, Shawn Pearce wrote: > >> I started playing around with the idea of storing references directly >> in Git. Exploiting the GITLINK tree entry, we can associate a name to >> any SHA-1. > > Gitlink entries do

Re: RefTree: Alternate ref backend

2015-12-17 Thread Shawn Pearce
On Thu, Dec 17, 2015 at 1:57 PM, Junio C Hamano wrote: > Shawn Pearce writes: > >> For example, recent git.git has a structure like this: >> >> $ git ls-tree -r refs/txn/committed >> 12 blob 22e42fc826b437033ca444e09368f53a0b169322 ..HEAD >> 16 commit 1ff88560c8d22bcdb528a6629239d63

Re: [PATCH] revision.c: propagate tag names from pending array

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 12:28:48PM -0800, Junio C Hamano wrote: > By the way, a totally unrelated niggle I have with 2073949 is this. > > $ git describe --contains 2073949 > v2.3.1~3^2~4 > > while as you said, this dates back to at least v2.2.0-rc0 > > $ git tag --contains 2073949 >

Re: [PATCH v2 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Philip Oakley
From: "Philip Oakley" From: "Junio C Hamano" , December 17, 2015 7:32 PM Philip Oakley writes: The gui.recentrepo list may be longer than the maxrecent setting. Allow extra space to show any extra entries. In an ideal world, the git gui would limit the number of entries to the maxrecent set

Re: RefTree: Alternate ref backend

2015-12-17 Thread Jeff King
On Thu, Dec 17, 2015 at 01:02:50PM -0800, Shawn Pearce wrote: > I started playing around with the idea of storing references directly > in Git. Exploiting the GITLINK tree entry, we can associate a name to > any SHA-1. Gitlink entries don't imply reachability, though. I guess that doesn't matter

Re: RefTree: Alternate ref backend

2015-12-17 Thread Junio C Hamano
Shawn Pearce writes: > For example, recent git.git has a structure like this: > > $ git ls-tree -r refs/txn/committed > 12 blob 22e42fc826b437033ca444e09368f53a0b169322 ..HEAD > 16 commit 1ff88560c8d22bcdb528a6629239d638f927cb96 heads/maint > 16 commit f3adf457e046f92f03935376

[PATCH v3] git-svn: add support for prefixed globs in config

2015-12-17 Thread Victor Leschuk
Introduce prefixed globs for branches and tags in git-svn. Globs like 'release_*' allow users to avoid long lines in config like: branches = branches/{release_20,release_21,release_22,...} Signed-off-by: Victor Leschuk --- Changes from v1 (in v2 I forgot to switch from `` to $() ): * Joi

RefTree: Alternate ref backend

2015-12-17 Thread Shawn Pearce
I started playing around with the idea of storing references directly in Git. Exploiting the GITLINK tree entry, we can associate a name to any SHA-1. By storing all references in a single tree, atomic transactions are possible. Its a simple compare-and-swap of a single 40 byte SHA-1. This of cour

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Stefan Beller
On Thu, Dec 17, 2015 at 12:42 PM, Torsten Bögershausen wrote: > On 17.12.15 21:22, Stefan Beller wrote: >> On Thu, Dec 17, 2015 at 12:12 PM, Torsten Bögershausen wrote: >>> On 16.12.15 01:04, Stefan Beller wrote: The man page of read(2) says: EAGAIN The file descriptor fd refers

Re: [PATCH 15/16] refs: add LMDB refs backend

2015-12-17 Thread Jonathan Nieder
David Turner wrote: > On Wed, 2015-12-16 at 17:00 -0800, Jonathan Nieder wrote: >> David Turner wrote: >>> +core.refsBackendType:: >>> + Type of refs backend. Default is to use the original files >>> + based backend. Set to 'lmdb' to activate the lmdb database >>> + backend. If you use the

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Torsten Bögershausen
On 17.12.15 21:22, Stefan Beller wrote: > On Thu, Dec 17, 2015 at 12:12 PM, Torsten Bögershausen wrote: >> On 16.12.15 01:04, Stefan Beller wrote: >>> The man page of read(2) says: >>> >>> EAGAIN The file descriptor fd refers to a file other than a socket >>>and has been marked nonblocki

Re: [PATCH] revision.c: propagate tag names from pending array

2015-12-17 Thread Junio C Hamano
Jeff King writes: > When we unwrap a tag to find its commit for a traversal, we > do not propagate the "name" field of the tag in the pending > array (i.e., the ref name the user gave us in the first > place) to the commit (instead, we use an empty string). This > means that "git log --source" wi

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Stefan Beller
On Thu, Dec 17, 2015 at 12:12 PM, Torsten Bögershausen wrote: > On 16.12.15 01:04, Stefan Beller wrote: >> The man page of read(2) says: >> >> EAGAIN The file descriptor fd refers to a file other than a socket >>and has been marked nonblocking (O_NONBLOCK), and the read >>would b

Re: [PATCHv2 2/7] xread: poll on non blocking fds

2015-12-17 Thread Torsten Bögershausen
On 16.12.15 01:04, Stefan Beller wrote: > The man page of read(2) says: > > 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 re

Re: [PATCH v2 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Philip Oakley
From: "Junio C Hamano" , December 17, 2015 7:32 PM Philip Oakley writes: The gui.recentrepo list may be longer than the maxrecent setting. Allow extra space to show any extra entries. In an ideal world, the git gui would limit the number of entries to the maxrecent setting, however the recent

Re: [PATCH v2 4/4] git gui: allow for a long recentrepo list

2015-12-17 Thread Junio C Hamano
Philip Oakley writes: > The gui.recentrepo list may be longer than the maxrecent setting. > Allow extra space to show any extra entries. > > In an ideal world, the git gui would limit the number of entries > to the maxrecent setting, however the recentrepo config list may > have been extended out

Re: [PATCH v2] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Junio C Hamano
Johannes Schindelin writes: > On Windows, when writing to a pipe fails, errno is always > EINVAL. However, Git expects it to be EPIPE. > > According to the documentation, there are two cases in which write() > triggers EINVAL: the buffer is NULL, or the length is odd but the mode > is 16-bit Unic

Re: [PATCHv2] submodule: Port resolve_relative_url from shell to C

2015-12-17 Thread Johannes Sixt
Am 17.12.2015 um 19:55 schrieb Johannes Sixt: As to the implementation, find a patch below that removes the ifdefs and a few other suggestions. The word "offers" is missing in this last half-sentence ;) -- To unsubscribe from this list: send the line "unsubscribe git" in the body of a message

Re: [PATCHv2] submodule: Port resolve_relative_url from shell to C

2015-12-17 Thread Johannes Sixt
Am 17.12.2015 um 01:26 schrieb Stefan Beller: > This reimplements the helper function `resolve_relative_url` in shell > in C. This functionality is needed in C for introducing the groups > feature later on. When using groups, the user should not need to run > `git submodule init`, but it should be

Re: [PATCH v2 10/10] dir: do not use untracked cache ident anymore

2015-12-17 Thread Junio C Hamano
Christian Couder writes: > In the "git worktree" documentation there is: > > "If you move a linked working tree to another file system, or within a > file system that does not support hard links, you need to run at least > one git command inside the linked working tree (e.g. git status) in > orde

Re: Git issue report : issue with capital letter in folder name

2015-12-17 Thread Stefan Beller
On Thu, Dec 17, 2015 at 7:45 AM, PFDuc wrote: > Hello, > > first of all thank you for developping git ! > > I had an issue with a capital block in the folder name inside my git repo. > The folder in my local was named "Display" and the one at origin was named > "display" resulting in error when im

Re: [PATCH] rebase: add --verify-signatures

2015-12-17 Thread Junio C Hamano
Alexander 'z33ky' Hirsch <1ze...@gmail.com> writes: > As I understand it, this is the same reason for the existence of > --verify-signatures for git-merge. Otherwise the same argument could be > made for git-merge I suspect that you are missing the bigger workflow issues, if you think this and m

Git issue report : issue with capital letter in folder name

2015-12-17 Thread PFDuc
Hello, first of all thank you for developping git ! I had an issue with a capital block in the folder name inside my git repo. The folder in my local was named "Display" and the one at origin was named "display" resulting in error when importing python code from this folder for users who got

Re: [PATCH v2 02/10] update-index: add --test-untracked-cache

2015-12-17 Thread David Turner
On Tue, 2015-12-15 at 17:28 +0100, Christian Couder wrote: > +--test-untracked-cache:: > + Only perform tests on the working directory to make sure > + untracked cache can be used. You have to manually enable > + untracked cache using `--force-untracked-cache` (or > + `--untracked-

Re: [PATCH] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Johannes Schindelin
Hi Junio, On Thu, 17 Dec 2015, Junio C Hamano wrote: > Johannes Schindelin writes: > > > My intuition (which I honestly did not verify using performance tests) was > > that write() is called *much* more often than, say, open(),... > > My gut feeling agrees with yours, but I do not think the fr

[PATCH v2] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Johannes Schindelin
On Windows, when writing to a pipe fails, errno is always EINVAL. However, Git expects it to be EPIPE. According to the documentation, there are two cases in which write() triggers EINVAL: the buffer is NULL, or the length is odd but the mode is 16-bit Unicode (the broken pipe is not mentioned as

Re: [PATCH v2 10/10] dir: do not use untracked cache ident anymore

2015-12-17 Thread Christian Couder
On Tue, Dec 15, 2015 at 8:49 PM, Junio C Hamano wrote: > Christian Couder writes: > >> +/* >> + * We used to save the location of the work tree and the kernel version, >> + * but it was not a good idea, so we now just save an empty string. >> + */ > > I do agree that storing the kernel version (o

Re: [PATCH 1/2] push: Fully test --recurse-submodules on command line overrides config

2015-12-17 Thread Junio C Hamano
Stefan Beller writes: >>> This looks good to me. >>> >> Thanks. Does "This" refer to 1/2 alone or the whole series? > > Yes. :) > > "This" is applicable to both patches. We had the discussion on 2/2 about me > misreading a line a few days earlier, but apart from that it looked good, too. Thanks

Re: [PATCH] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Junio C Hamano
Johannes Schindelin writes: > My intuition (which I honestly did not verify using performance tests) was > that write() is called *much* more often than, say, open(),... My gut feeling agrees with yours, but I do not think the frequency at which write() is called should be the primary factor whe

[PATCH v2] git-svn: add support for prefixed globs in config

2015-12-17 Thread Victor Leschuk
Introduce prefixed globs for branches and tags in git-svn. Globs like 'release_*' allow users to avoid long lines in config like: branches = branches/{release_20,release_21,release_22,...} Signed-off-by: Victor Leschuk --- Changes from v1: * Joined implementation and test in one patch

RE: CruiseControl.NET hangs on git-submodule bad file descriptor

2015-12-17 Thread Alexander Skrinnik
OK, no problem I have workaround :) Thanks a lot for your help Best wishes -- Alexander Skrinnik -Original Message- From: Johannes Schindelin [mailto:johannes.schinde...@gmx.de] Sent: Thursday, December 17, 2015 3:55 PM To: Alexander Skrinnik Cc: git@vger.kernel.org Subject: RE: Cruis

RE: CruiseControl.NET hangs on git-submodule bad file descriptor

2015-12-17 Thread Johannes Schindelin
Hi Alexander, you might want to refrain from top-posting on this list in the future. Just sayin' ;-) On Thu, 17 Dec 2015, Alexander Skrinnik wrote: > I found workaround, CC.NET invokes bat-file with command: > git submodule foreach git checkout "myBranch" > It works fine. But looks like the is

Re: [PATCH 7/8] config: add core.untrackedCache

2015-12-17 Thread Duy Nguyen
On Wed, Dec 16, 2015 at 4:53 AM, Ævar Arnfjörð Bjarmason wrote: > On Tue, Dec 15, 2015 at 8:40 PM, Junio C Hamano wrote: >> Ævar Arnfjörð Bjarmason writes: >> I still have a problem with the approach from "design cleanliness" >> point of view[...] >> >> In any case I think we already have agreed

RE: CruiseControl.NET hangs on git-submodule bad file descriptor

2015-12-17 Thread Alexander Skrinnik
Hi Johannes, Thank you for your answer. Yes, this is the same issue. I found workaround, CC.NET invokes bat-file with command: git submodule foreach git checkout "myBranch" mailto:johannes.schinde...@gmx.de] Sent: Thursday, December 17, 2015 11:46 AM To: Alexander Skrinnik Cc: git@vger.kernel.or

Re: [PATCH 7/8] config: add core.untrackedCache

2015-12-17 Thread Duy Nguyen
On Thu, Dec 17, 2015 at 2:44 PM, Jeff King wrote: > I think we may actually be thinking of the same thing. Naively, I would > expect: > > .. > - if there is cache data in the index but that config flag is not set, > presumably we would not update it (we could even explicitly drop it, > b

Re: [PATCH v2 01/10] update-index: use enum for untracked cache options

2015-12-17 Thread Torsten Bögershausen
On 15.12.15 17:28, Christian Couder wrote: > Signed-off-by: Christian Couder > --- > builtin/update-index.c | 18 +- > 1 file changed, 13 insertions(+), 5 deletions(-) > > diff --git a/builtin/update-index.c b/builtin/update-index.c > index 7431938..2430a68 100644 > --- a/builtin

Re: No auto CRLF conversion in Commit Message comments

2015-12-17 Thread Andrew Harfoot
Hi Johannes, On 16/12/2015 16:45, Johannes Schindelin wrote: However, if you have `core.editor = notepad`, it should Just Work because there is a `notepad` helper that performs the LF<->CR/LF translation transparently. The problem in my case was that I had `core.editor = notepad.exe` This wa

Re: CruiseControl.NET hangs on git-submodule bad file descriptor

2015-12-17 Thread Johannes Schindelin
Hi Alexander, On Thu, 17 Dec 2015, Alexander Skrinnik wrote: > I had installed CruiseControl.NET 1.8.5 and Git-1.9.5-preview20150319 > CC.NET invokes bat-file which invokes > git submodule foreach git checkout "myBranch" > > It worked good. > Today I upgraded git to 2.6.4 and CC.NET fails on th

Re: [PATCH v2 10/11] ref-filter: introduce contents_atom_parser()

2015-12-17 Thread Eric Sunshine
On Wed, Dec 16, 2015 at 10:30 AM, Karthik Nayak wrote: > Introduce contents_atom_parser() which will parse the '%(contents)' > atom and store information into the 'used_atom' structure based on the > modifiers used along with the atom. > > Helped-by: Ramsay Jones > Signed-off-by: Karthik Nayak >

Re: [PATCH] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Johannes Schindelin
Hi Junio, On Wed, 16 Dec 2015, Junio C Hamano wrote: > Johannes Schindelin writes: > > > int mingw_fflush(FILE *stream); > > #define fflush mingw_fflush > > > > +static inline ssize_t mingw_write(int fd, const void *buf, size_t len) > > +{ > > + ssize_t result = write(fd, buf, len); > > +

Re: [PATCH] mingw: emulate write(2) that fails with a EPIPE

2015-12-17 Thread Johannes Schindelin
Hi Eric, On Wed, 16 Dec 2015, Eric Sunshine wrote: > On Wednesday, December 16, 2015, Johannes Schindelin > wrote: > > diff --git a/compat/mingw.h b/compat/mingw.h > > @@ -210,6 +210,24 @@ FILE *mingw_freopen (const char *filename, const char > > *otype, FILE *stream); > > +static inline ssize_

Re: [PATCH v2 09/11] ref-filter: introduce remote_ref_atom_parser()

2015-12-17 Thread Eric Sunshine
On Wed, Dec 16, 2015 at 10:30 AM, Karthik Nayak wrote: > Introduce remote_ref_atom_parser() which will parse the '%(upstream)' > and '%(push)' atoms and store information into the 'used_atom' > structure based on the modifiers used along with the corresponding > atom. > > Helped-by: Ramsay Jones

Re: [PATCH v2 08/11] ref-filter: introduce prefixes for the align atom

2015-12-17 Thread Eric Sunshine
On Wed, Dec 16, 2015 at 10:29 AM, Karthik Nayak wrote: > ref-filter: introduce prefixes for the align atom The prefixes are actually for the arguments to the 'align' atom, not for the atom itself. However, it might be better to describe this at a bit higher level. Perhaps: ref-filter: align:

Re: [PATCH v2 01/11] strbuf: introduce strbuf_split_str_without_term()

2015-12-17 Thread Karthik Nayak
On Thu, Dec 17, 2015 at 2:05 AM, Eric Sunshine wrote: > On Wed, Dec 16, 2015 at 10:29 AM, Karthik Nayak wrote: >> strbuf: introduce strbuf_split_str_without_term() > > s/without/omit/ > >> The current implementation of 'strbuf_split_buf()' includes the >> terminator at the end of each strbuf post

CruiseControl.NET hangs on git-submodule bad file descriptor

2015-12-17 Thread Alexander Skrinnik
Hi guys, I had installed CruiseControl.NET 1.8.5 and Git-1.9.5-preview20150319 CC.NET invokes bat-file which invokes git submodule foreach git checkout "myBranch" It worked good. Today I upgraded git to 2.6.4 and CC.NET fails on this command with error: C:\Program Files (x86)\Git\mingw32/libexec

Re: [PATCHv2] submodule: Port resolve_relative_url from shell to C

2015-12-17 Thread Eric Sunshine
A rather superficial review... On Wed, Dec 16, 2015 at 7:26 PM, Stefan Beller wrote: > This reimplements the helper function `resolve_relative_url` in shell s/This reimplements/Reimplement/ > in C. This functionality is needed in C for introducing the groups > feature later on. When using group