cygwin dbus port patches

[Thiago Macieira]

Em Segunda-feira 14. Junho 2010, às 04.15.34, Yaakov (Cygwin/X) escreveu:
> On Sun, 2010-06-13 at 20:38 +0200, Thiago Macieira wrote:
> > > There are some flags which GCC allows but have no effect on Cygwin and
> > > therefore generate warnings as such.  This includes -fPIC and
> > > -fvisibility*.  Adding the -Werror to cc_supports_flag makes sure that
> > > the flag is truly supported and necessary.
> > 
> > The problem is that this will cause false-negatives when building code
> > that is valid.
> > 
> > I've seen this before, that in maintainer mode there are a couple of
> > extra warnings enabled like the prototype warning.
> 
> My patch doesn't add -Werror to CFLAGS, it just adds it to
> cc_supports_flag's testing of the flag itself.  So this will not cause
> any problems with the code itself.

It causes false-positives in cc_supports_flag because testing the option causes 
a warning because of the source code.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
  Senior Product Manager - Nokia, Qt Development Frameworks
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.freedesktop.org/archives/dbus/attachments/20100614/f268d998/attachment.pgp>