Re: [PATCH v3] contrib: add win32 credential-helper

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



Erik Faye-Lund <kusmabite@xxxxxxxxx> writes:

> On Fri, Jul 27, 2012 at 7:10 PM, Erik Faye-Lund <kusmabite@xxxxxxxxx> wrote:
>> Since the Windows port of Git expects binary pipes, we need to make
>> sure the helper-end also sets up binary pipes.
>>
>> Side-step CRLF-issue in test to make it pass.
>>
>> Signed-off-by: Erik Faye-Lund <kusmabite@xxxxxxxxx>
>> ---
>>
>> Sorry for the delay, but here's the third iteration of my credential
>> helper for Windows' credential API.
>>
>> The only change since v2 is that it now supports protocols without
>> a host component, as suggested by Jeff King.
>>
>
> ...aaand just as I sent it out, I realized that my Makefile was a bit
> on the light side. Imagine this patch on top for now, please :)
>
> diff --git a/contrib/credential/wincred/Makefile
> b/contrib/credential/wincred/Makefile
> index b4f098f..bad45ca 100644
> --- a/contrib/credential/wincred/Makefile
> +++ b/contrib/credential/wincred/Makefile
> @@ -4,5 +4,11 @@ CC = gcc
>  RM = rm -f
>  CFLAGS = -O2 -Wall
>
> +-include ../../../config.mak.autogen
> +-include ../../../config.mak
> +
>  git-credential-wincred.exe : git-credential-wincred.c
>  	$(LINK.c) $^ $(LOADLIBES) $(LDLIBS) -o $@
> +
> +clean:
> +	$(RM) git-credential-wincred.exe

I notice that I didn't queue either of these; are you still
polishing them, or are they in good enough shape that it wouldn't
make things harder for you if I queue it, with a plan to merge it
soon after 1.7.12?

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


[Index of Archives]     [Linux Kernel Development]     [Gcc Help]     [IETF Annouce]     [DCCP]     [Netdev]     [Networking]     [Security]     [V4L]     [Bugtraq]     [Yosemite]     [MIPS Linux]     [ARM Linux]     [Linux Security]     [Linux RAID]     [Linux SCSI]     [Fedora Users]