[PATCH] Beautified MacroLogFeature

[Sebastian Trüg]

And I forgot to show an example again:


-----------------------------------------------------------------------------
-- 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.         

-----------------------------------------------------------------------------
-- The following dependencies were found.                                    
-----------------------------------------------------------------------------
   * libDVDRead                                                              
     Libdvdread provides a simple foundation for reading DVD video disks.    
   * Taglib                                                                  
     Read and write tags in audio files                                      
   * FFmpeg                                                                  
     Needed for the K3b FFmpeg decoder plugin which can decode virtually all
     audio types.
   * Flac                                                                                
     Needed for the Flac audio decoder plugin.                                           
   * Flac++                                                                              
     Needed for the Flac audio decoder plugin.                                           
   * Mad mp3                                                                             
     Needed for the mp3 audio decoder plugin.                                            
   * Muse                                                                                
     Needed for the Musepack audio decoder plugin                                        
   * Sndfile                                                                             
     Needed for the libsndfile audio decoder plugin.                                     
   * Lame mp3 encoder                                                                    
     Needed for the lame mpf encoder encoder plugin.                                     
   * Ogg Vorbis                                                                          
     Needed for the K3b Ogg Vorbis decoder and encoder plugins.                          

-----------------------------------------------------------------------------
-- The following OPTIONAL dependencies could NOT be found.                   
-----------------------------------------------------------------------------
   * Musicbrainz (1.1 or higher)  <http://musicbrainz.org/>                  
     Provide information about the CD, about the artist or about related
     information

-----------------------------------------------------------------------------



On Thursday 19 February 2009 23:57:54 Loïc Corbasson wrote:
> On Thu, Feb 19, 2009 at 21:24, Alexander Neundorf <neundorf at kde.org> wrote:
> > On Thursday 19 February 2009, Sebastian Trüg wrote:
> > > On Thursday 19 February 2009 18:23:01 Sebastian Trüg wrote:
> > > > > I agree with Allen -- when I compile a module, the information "you
> >
> > are
> >
> > > > > missing this and that" is much more important than the information
> > > > > "I found this and that". For the missing stuff I might have to take
> > > > > action. For the found stuff there is no problem, so there is little
> > > > > point in filling 3 screens with "I found this and that, so you'll
> > > > > get such and such features"... we don't list the features that one
> > > > > gets
> >
> > in
> >
> > > > > all cases, either
> > > > >
> > > > > :-) I think all that's needed is a short confirmation that "foo"
> > > > > : was found
> > > > >
> > > > > after you installed it.
> > > >
> > > > well, OK then. I will just use my own local copy for K3b then,
> > > > because frankly, I really don't like the output as it is now.
> > >
> > > Sorry, this sounds way to aggressive. And that was not my intention.
> > > What
> >
> > I
> >
> > > should have written is:
> >
> > No problem.
> > Actually I agree with you, and also also agree that your version looks
> > nicer.
> > But one of the things was that there should be only one line per package,
> > so
> > it's easy to grep for stuff.
> > And it's also correct that the things which were found shouldn't take too
> > much
> > space.
> >
> > Alex
>
> How about making this an option then, that you can switch on and off, and
> which is non-verbose like today as a default? I too like Sebastian's work
> and find it very useful -- but I understand that it doesn't fit all use
> cases, e.g. making grepping difficult.
>
> Cheers,
> Loïc