[PATCH] Beautified MacroLogFeature

Thiago Macieira thiago at kde.org
Fri Feb 20 12:50:58 CET 2009


Em Sexta-feira 20 Fevereiro 2009, às 12:01:15, David Faure escreveu:
> On Friday 20 February 2009, Sebastian Trüg wrote:
> > -------------------------------------------------------------------------
> >---- -- The following REQUIRED dependencies could NOT be found.
> > -------------------------------------------------------------------------
> >---- * KCddb  <http://multimedia.kde.org> KCddb is used to retrieve audio
> > CD meta data from the internet.
>
> I like this part - it takes as much space as the current stuff but
> the additional "-----------" makes it easier to read and the indentation
> of the output is better than the current macro.
>
> > -------------------------------------------------------------------------
> >---- -- The following dependencies were found.
> > -------------------------------------------------------------------------
> >---- * libDVDRead Libdvdread provides a simple foundation for reading DVD
> > video disks.
>
> If this could be on a single line, then it would be perfect ;-)

One more thing: reverse the order. Please list the dependencies like this:

 - stuff we found (abbreviated like David asked)
 - optional stuff we did not find
 - required stuff we did not find

The required not found are the most important to be seen, so they should be 
close to the bottom.

-- 
Thiago Macieira - thiago (AT) macieira.info - thiago (AT) kde.org
  Senior Product Manager - Nokia, Qt Software
      PGP/GPG: 0x6EF45358; fingerprint:
      E067 918B B660 DBD1 105C  966C 33F5 F005 6EF4 5358


More information about the Kde-buildsystem mailing list