[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