Re: compile error in Git v2.7.0-rc0
- From
Jeff King <peff@peff.net>
- Date
- Dec 14, 2015, 20:46 UTC
- Message-ID
- <20151214204624.GA14788@sigill.intra.peff.net>
- In-Reply-To
- <xmqqvb80ix9u.fsf@gitster.mtv.corp.google.com>
On Mon, Dec 14, 2015 at 12:35:25PM -0800, Junio C Hamano wrote:
Show 29 quoted lines
> johan defries <johandefries@gmail.com> writes:
>
> > Probably because I have NO_IPV6 defined.
> >
> > ident.c: In function ‘canonical_name’:
> > ident.c:89:37: error: ‘buf’ undeclared (first use in this function)
> > struct hostent *he = gethostbyname(buf);
> > ^
> > ident.c:89:37: note: each undeclared identifier is reported only once
> > for each function it appears in
> > make: *** [ident.o] Fout 1
>
> Thanks. This should perhaps do?
>
> ident.c | 1 +
> 1 file changed, 1 insertion(+)
>
> diff --git a/ident.c b/ident.c
> index 4e7f99d..2900879 100644
> --- a/ident.c
> +++ b/ident.c
> @@ -86,6 +86,7 @@ static int canonical_name(const char *host, struct strbuf *out)
> freeaddrinfo(ai);
> }
> #else
> + char buf[1024];
> struct hostent *he = gethostbyname(buf);
> if (he && strchr(he->h_name, '.')) {
> strbuf_addstr(out, he->h_name);Whoops. Looks like we didn't test the NO_IPV6 code path.
I don't think that fix is right, though. We should be passing "host" to gethostbyname.
-Peff