packages: secpanel/secpanel.spec, secpanel/TODO (NEW) - TODO

[Pawel Golaszewski]

On Fri, 6 Nov 2009, Jakub Bogusz wrote:
> > Author: blues                        Date: Thu Oct 29 08:15:20 2009 GMT
> > Module: packages                      Tag: HEAD
> > ---- Log message:
> > - TODO
> > 
> > ---- Files affected:
> > packages/secpanel:
> >    secpanel.spec (1.31 -> 1.32) , TODO (NONE -> 1.1)  (NEW)
> > 
> > ---- Diffs:
> > 
> > ================================================================
> > Index: packages/secpanel/secpanel.spec
> > diff -u packages/secpanel/secpanel.spec:1.31 packages/secpanel/secpanel.spec:1.32
> > --- packages/secpanel/secpanel.spec:1.31	Fri Apr 10 10:39:10 2009
> > +++ packages/secpanel/secpanel.spec	Thu Oct 29 09:15:14 2009
> > @@ -1,6 +1,4 @@
> >  # $Revision$, $Date$
> > -# TODO:
> > -# - some icon for desktop
> >  Summary:	Visual management of SSH connections
> >  Summary(pl.UTF-8):	Wizualna nakładka na klienta SSH
> >  Name:		secpanel
> 
> > ================================================================
> > Index: packages/secpanel/TODO
> > diff -u /dev/null packages/secpanel/TODO:1.1
> > --- /dev/null	Thu Oct 29 09:15:20 2009
> > +++ packages/secpanel/TODO	Thu Oct 29 09:15:14 2009
> > @@ -0,0 +1,2 @@
> > +# TODO:
> > +- some icon for desktop
> > ================================================================
> I don't find such way useful:

I do.
You can put there a lot more info that in spec header, without any "^#". 
Compilation errors, whatever.

I never liked long TODO's in spec headers. It's only noise.

> - when I download package by builder, I don't have ANY idea, that there
>   even is any TODO

True - it should be fixed.

> - TODO is not available in .src.rpm

It can be packaged as doc. Easy to do. Packaging it and putting as Source 
will fix previous issue...

-- 
pozdr.  Paweł Gołaszewski          jid:blues<at>jabber<dot>gda<dot>pl
--------------------------------------------------------------------------
If you think of MS-DOS as mono, and Windows as stereo, then Linux is Dolby
Pro-Logic Surround Sound with Bass Boost and all the music is free.