[Development] Cleaner code base patches

Thiago Macieira thiago.macieira at intel.com
Mon Jan 28 17:15:06 CET 2013


On segunda-feira, 28 de janeiro de 2013 16.06.25, Knoll Lars wrote:
> On Jan 28, 2013, at 10:01 AM, Tomasz Siekierda <sierdzio at gmail.com> wrote:
> > On 28 January 2013 09:57, Thiago Macieira <thiago.macieira at intel.com> 
wrote:
> >> Options are:
> >> 
> >> 1) always with -developer-build
> >> 2) on by default on -developer-build, but the CI disables it
> >> 3) completely optional, CI enables it
> >> 4) completely optional, not enabled by the CI
> >> 
> >> I favour options 1 or 2.
> > 
> > I would vote for 1), but it can cause problems in bigger releases
> > (remember amount of warnings before Qt5 alpha?).
> 
> I'd favor (2) for now, and move over to (1) once we gained a little
> experience with it for the compilation of the whole module. The header
> compilation test should probably move to (1) immediately.

Ok, thanks.

I'll modify the patch to introduce a configure-time option.
-- 
Thiago Macieira - thiago.macieira (AT) intel.com
  Software Architect - Intel Open Source Technology Center
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 190 bytes
Desc: This is a digitally signed message part.
URL: <http://lists.qt-project.org/pipermail/development/attachments/20130128/d718d960/attachment.sig>


More information about the Development mailing list