pkgsrc-WIP-discuss archive

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

Re: compiler error? (RE: please review wip/cc65)



In Message <60cb6cc6424c8344ab18d0ff0961bf4d.squirrel%www.duzan.org@localhost> ,
   gary%duzan.org@localhost wrote:

=>=> gary%duzan.org@localhost <mailto:gary%duzan.org@localhost> wrote:
=>=>> In file included from strbuf.c:42:
=>=>> va_copy.h:48:2: error: #error "The compiler is broken!"
=>=>> In file included from xsprintf.c:46:
=>=>> va_copy.h:48:2: error: #error "The compiler is broken!"
=>=>> gmake[2]: *** [depend] Error 1
=>=>> gmake[2]: Leaving directory
=>=>> `/usr/pkgsrc/wip/cc65/work.x86_64/cc65-2.12.0/src/common'
=>=>> gmake[1]: *** [all] Error 2
=>=>
=>=> (Changing group. Please remove pkgsrc-wip-review from replies.)
=>=>
=>=> The following comment was the response I received from the developer:
=>=>
=>=> {{{ This means that the compiler claims to be compliant to an ANSI or ISO
=>=> standard before C99 but defines va_copy anyway. If the former is true,
=>=> va_copy is no reserved identifier and must not be defined by the compiler.
=>=> At least this is my interpretation of the standard is how other compilers
=>=> implement it. Maybe someone active in -BSD development can comment on it.
=>=> }}}
=>
=>   A quick search shows that NetBSD is setting the following in stdarg.h:
=>
=>#if !defined(_ANSI_SOURCE) &&                                           \
=>    (defined(_ISOC99_SOURCE) || (__STDC_VERSION__ - 0) >= 199901L ||    \
=>     defined(_NETBSD_SOURCE))
=>#define va_copy(dest, src)      __va_copy((dest), (src))
=>#endif
=>
=>   I haven't yet checked to see if this is actually included, or which
=>macros were set.

   For completeness, yes, stdarg.h is included. _NETBSD_SOURCE is
being defined by default since none of { _ANSI_SOURCE, _POSIX_C_SOURCE,
_XOPEN_SOURCE } are being defined. So defining _ANSI_SOURCE would
do what the source expects. Of course, as Joerg stated, using the
system va_copy() if defined might be better, and from the looks of
things might be an easier patch.

                                        Gary Duzan



-------------------------------------------------------------------------
Sponsored by: SourceForge.net Community Choice Awards: VOTE NOW!
Studies have shown that voting for your favorite open source project,
along with a healthy diet, reduces your potential for chronic lameness
and boredom. Vote Now at http://www.sourceforge.net/community/cca08
_______________________________________________
pkgsrc-wip-discuss mailing list
pkgsrc-wip-discuss%lists.sourceforge.net@localhost
https://lists.sourceforge.net/lists/listinfo/pkgsrc-wip-discuss


Home | Main Index | Thread Index | Old Index