git/list[1] front-page[2] threads[3] people[4] search[5] about
 

Re: "trace.c", line 219: error: identifier redeclared: trace_strbuf

From
Ddev <dev@cor0.com>
Date
Sep 7, 2014, 17:19 UTC
Message-ID
<2059998579.17018.1410110359755.JavaMail.vpopmail@webmail2.networksolutionsemail.com>
In-Reply-To
<540B742F.5030700@web.de>
On September 6, 2014 at 4:53 PM "René Scharfe" <l.s.r@web.de> wrote:
> Am 06.09.2014 um 21:26 schrieb dev:
> >
> >
<snip>
Show 37 quoted lines
> >
> > Solaris 10 with Oracle Studio 12.3 compiler tools. A lengthy
> > maillist
> > discussion last week sorted out the previous release of git just
> > fine however this release fails to build.
> >
>
> The issue was introduced with e05bed96 (trace: add 'file:line' to all
> trace output).
>
> -- >8 --
> Subject: [PATCH] trace: correct trace_strbuf() parameter type for
> !HAVE_VARIADIC_MACROS
>
> Reported-by: dev <dev@cor0.com>
> Signed-off-by: Rene Scharfe <l.s.r@web.de>
> ---
>  trace.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/trace.c b/trace.c
> index 54aaee5..b6f25a2 100644
> --- a/trace.c
> +++ b/trace.c
> @@ -216,7 +216,7 @@ void trace_argv_printf(const char **argv, const
> char *format, ...)
>    va_end(ap);
>  }
> 
> -void trace_strbuf(const char *key, const struct strbuf *data)
> +void trace_strbuf(struct trace_key *key, const struct strbuf *data)
>  {
>    trace_strbuf_fl(NULL, 0, key, data);
>  }
> --
> 2.1.0
>

Thank you for the reply and the info. I backed that out and things get worse :

$ diff -h trace.c.backup trace.c 219,221c219,221 < void trace_strbuf(const char *key, const struct strbuf *data) < { < trace_strbuf_fl(NULL, 0, key, data); ---

> void trace_strbuf(struct trace_key *key, const struct strbuf *data)
> {
>         trace_strbuf_fl(NULL, 0, key, data);
$ gmake CFLAGS="$CFLAGS" LDFLAGS="$LD_OPTIONS" NEEDS_LIBICONV=Yes \
Show 10 quoted lines
> SHELL_PATH=/usr/local/bin/bash \
> SANE_TOOL_PATH=/usr/local/bin \
> USE_LIBPCRE=1 LIBPCREDIR=/usr/local CURLDIR=/usr/local \
> EXPATDIR=/usr/local NEEDS_LIBINTL_BEFORE_LIBICONV=1 \
> NEEDS_SOCKET=1 NEEDS_RESOLV=1 USE_NSEC=1 \
> PERL_PATH=/usr/local/bin/perl \
> TAR=/usr/bin/tar \
> NO_PYTHON=1 DEFAULT_PAGER=/usr/xpg4/bin/more \
> DEFAULT_EDITOR=/usr/local/bin/vim DEFAULT_HELP_FORMAT=man \
> prefix=/usr/local
    CC trace.o
"trace.c", line 390: error: reference to static identifier "offset" in
extern inline function
"trace.c", line 392: error: reference to static identifier "offset" in
extern inline function
"trace.c", line 393: error: reference to static identifier "offset" in
extern inline function
"trace.c", line 401: error: reference to static identifier "offset" in
extern inline function
"trace.c", line 403: error: reference to static identifier "offset" in
extern inline function
cc: acomp failed for trace.c
gmake: *** [trace.o] Error 2
$
So is there a possible 2.1.1 coming along shortly ?
dev
Previous: René ScharfeNext: Junio C Hamano
Message 3 of 4 in “"trace.c", line 219: error: identifier redeclared: trace_strbuf”
  1. devSep 6, 2014
  2. René ScharfeSep 6, 2014
  3. devSep 7, 2014
  4. Junio C HamanoSep 8, 2014

Read the whole thread, see it on lore, or plain text.

$ cat FOOTERMessages come from the public archive at lore.kernel.org/git, fetched every hour. The front page is chosen and written each morning by an AI editor and can be wrong; the threads themselves are the record. About and API. For agents: an MCP server at https://gitlist.dev/mcp, and any thread, story or person page as Markdown by adding .md to its URL (or sending Accept: text/markdown). Details in /llms.txt.