On Thu, Sep 24, 2015 at 04:23:52pm +0200, Andy Polyakov wrote: > >>> - mingw debug and shared builds in master. > >> > >> While I can confirm problem with shared (fixable with attached patch, > >> please double-check), > > > > Can confirm that your patch works. > > > >> I can't confirm problem with debug (please elaborate). > > > > The travis debug builds now also use --strict-warnings, which enables all > > warnings and treats them as errors, and there's a bunch of warnings when > > building with mingw. > > I think it's appropriate to set pragmatic goals in this case. Fixing > warnings for non-primary[!] platform such as mingw in master is arguably > pragmatic, but not in all versions. So that in this case, i.e. > travis-ci.org, I'd argue for limiting testing to current working > de-facto status for all versions but master. Is it reasonable?
Sure, that seems like a sensible strategy. > > I tried to fix some of them (see [0]) but it's kind of a lost battle. > > What does "kind of a lost battle" mean in the context? Meaning that there actually are a lot of warnings and many of them are very difficult (if not impossible) to fix. Fixing all of them would require quite a bit of work and the result may be very ugly. As an aside, the patch I previously posted is broken, so just ignore it. Cheers
signature.asc
Description: PGP signature
_______________________________________________ openssl-dev mailing list To unsubscribe: https://mta.openssl.org/mailman/listinfo/openssl-dev
