On 2014-03-23 01.44, David Cowden wrote:
> http://stackoverflow.com/questions/22585091/git-hooks-pre-push-script-does-not-receive-input-via-stdin
>
> Is this a bug in git? Or am I just a shell noob?
>
> Thanks in advance,
>
> David
I assume that you have the right version of Git ?
You can look
Thanks for the resubmission. Comments below...
On Sat, Mar 22, 2014 at 11:12 AM, Ashwin Jha wrote:
> Subject: [PATCH] Modify fsck_commit. Replace memcmp by skip_prefix
In his review, Tanay already mentioned indicating a resubmission via
[PATCH vN]. Additionally, specify the module or function be
In addition to the valuable review comments already provided by
Alexandru and David, see a few more below.
On Sat, Mar 22, 2014 at 5:25 PM, Mustafa Orkun Acar
wrote:
> Subject: [PATCH v2] Rewrite strbuf.c:strbuf_cmp() replace memcmp() with
> starts_with()
This isn't actually v2. It would have b
On Sat, Mar 22, 2014 at 5:33 AM, George Papanikolaou
wrote:
> On Sat, Mar 22, 2014 at 12:46 AM, Eric Sunshine
> wrote:
>> Because it's unnecessary and invites confusion from people reading the
>> code since they now have to wonder if there is something unusual and
>> non-obvious going. Worse, th
Thanks for your comments, Tanay and Eric.
Regarding the [PATCH v2], I will keep this in mind for the subsequent
patches.
On 03/23/2014 02:10 PM, Eric Sunshine wrote:
Thanks for the resubmission. Comments below...
On Sat, Mar 22, 2014 at 11:12 AM, Ashwin Jha wrote:
Subject: [PATCH] Modify
On Fri, Mar 21, 2014 at 08:31:42AM +0100, Per Cederqvist wrote:
> Signed-off-by: Per Cederqvist
> ---
> guilt-import-commit | 6 +++---
> 1 file changed, 3 insertions(+), 3 deletions(-)
>
> diff --git a/guilt-import-commit b/guilt-import-commit
> index 20dcee2..9488ded 100755
> --- a/guilt-impor
On Fri, Mar 21, 2014 at 08:31:44AM +0100, Per Cederqvist wrote:
> When extracting the patch, we only want the actual patches. We don't
> want the "---" delimiter. Simplify the extraction by simply deleting
> everything before the first "diff " line. (Use sed instead of awk to
> simplify the code
On Tue, Mar 11, 2014 at 11:37 PM, Andrew Keller wrote:
> I am considering developing a new feature, and I'd like to poll the group for
> opinions.
>
> Background: A couple years ago, I wrote a set of scripts that speed up
> cloning of frequently used repositories. The scripts utilize a bare Git
The documentation as-is does not mention that the pre-push hook is
executed even when there is nothing to push. This can lead a new
reader to beilieve there will always be lines fed to the script's
standerd input and cause minor confusion as to what is happening
when there are no lines provided to
The documentation as-is does not mention that the pre-push hook is
executed even when there is nothing to push. This can lead a new
reader to beilieve there will always be lines fed to the script's
standerd input and cause minor confusion as to what is happening
when there are no lines provided to
The documentation as-is does not mention that the pre-push hook is
executed even when there is nothing to push. This can lead a new
reader to beilieve there will always be lines fed to the script's
standerd input and cause minor confusion as to what is happening
when there are no lines provided to
You are correct, it was a misunderstanding on my part. I have
suggested a patch to clarify the pre-push documentation in a separate
thread.
Thanks for pointing that out!
On Sun, Mar 23, 2014 at 12:19 AM, Torsten Bögershausen wrote:
> On 2014-03-23 01.44, David Cowden wrote:
>> http://stackoverf
Johannes Sixt writes:
> I see what you mean. Notice, however, that we also test whether
> the regular expressions can be compiled successfully, and for this
> it is desirable to have the complete list of userdiff drivers.
> Until we have at least one test-case for each driver, we wouldn't
> get t
On Sun, Mar 23, 2014 at 6:04 PM, Jeff Sipek wrote:
> On Fri, Mar 21, 2014 at 08:31:42AM +0100, Per Cederqvist wrote:
>
>> Signed-off-by: Per Cederqvist
>> ---
>> guilt-import-commit | 6 +++---
>> 1 file changed, 3 insertions(+), 3 deletions(-)
>>
>> diff --git a/guilt-import-commit b/guilt-imp
On Sun, Mar 23, 2014 at 08:57:08PM +0100, Per Cederqvist wrote:
> On Sun, Mar 23, 2014 at 6:04 PM, Jeff Sipek wrote:
>
> > On Fri, Mar 21, 2014 at 08:31:42AM +0100, Per Cederqvist wrote:
> >
> >> Signed-off-by: Per Cederqvist
> >> ---
> >> guilt-import-commit | 6 +++---
> >> 1 file changed, 3
David Cowden writes:
> The documentation as-is does not mention that the pre-push hook is
> executed even when there is nothing to push. This can lead a new
> reader to beilieve there will always be lines fed to the script's
> standerd input and cause minor confusion as to what is happening
> wh
On Sun, Mar 23, 2014 at 6:09 PM, Jeff Sipek wrote:
> On Fri, Mar 21, 2014 at 08:31:44AM +0100, Per Cederqvist wrote:
>> When extracting the patch, we only want the actual patches. We don't
>> want the "---" delimiter. Simplify the extraction by simply deleting
>> everything before the first "di
On Sun, Mar 23, 2014 at 9:07 PM, Jeff Sipek wrote:
> On Sun, Mar 23, 2014 at 08:57:08PM +0100, Per Cederqvist wrote:
>> On Sun, Mar 23, 2014 at 6:04 PM, Jeff Sipek wrote:
>>
>> > On Fri, Mar 21, 2014 at 08:31:42AM +0100, Per Cederqvist wrote:
>> >
>> >> Signed-off-by: Per Cederqvist
>> >> ---
>>
On 03/22/2014 05:56 AM, Jeff King wrote:
On Fri, Mar 21, 2014 at 07:58:55PM -0700, Siddharth Agarwal wrote:
Is it also reproducible just with the tip of "next"? Note that the
patches in jk/bitmap-reuse-delta have not been widely deployed (in
particular, we are not yet using them at GitHub, and w
On Sun, Mar 23, 2014 at 9:10 AM, Ashwin Jha wrote:
> On 03/23/2014 02:10 PM, Eric Sunshine wrote:
>> On Sat, Mar 22, 2014 at 11:12 AM, Ashwin Jha wrote:
>>> - if (memcmp(buffer, "tree ", 5))
>>> + buffer = (char *)skip_prefix(buffer, "tree ");
>>
>> These casts are ugly. It should be
On Sun, Mar 23, 2014 at 3:01 PM, David Cowden wrote:
> The documentation as-is does not mention that the pre-push hook is
> executed even when there is nothing to push. This can lead a new
> reader to beilieve there will always be lines fed to the script's
> standerd input and cause minor confusi
21 matches
Mail list logo