I finally got around to syncing with cvs and rebuilding the patch.

> 
> >   2) "@Echo off" needs to go after the initial comment block in the
> >      file; otherwise, install.pl will miss the "OPTIONAL" line when
> >      offering which scripts to install.  This is arguably a bug in
> >      install.pl...  But I kind of like having the top-level
> >      description of the script appear first in the file.  I am pretty
> >      sure the "::" lines do not echo anyway.
> 
> True, the "::" lines do not echo.
> I have no problem with moving the @Echo off to under the 1st comment block.
> 
> Should I resubmit the patch with the Echo statements moved? I'll look into doing 
> this sometime
> tomorrow if need be.
> 
> >
> > Other than these minor things, the question is, what do other people
> > think?  You have a good point about todo.pl already being fairly
> > verbose about what it is doing...  Do people like the current output,
> > which prints something whenever a command gets pushed on the to-do
> > list, as well as when it gets popped off?  Or are "quieter" scripts
> > better?
> 
> > I am leaning toward applying these "quieting" patches unless somebody
> > wants to talk me out of it.
> >
> >  - Pat




-------------------------------------------------------
This SF.Net email is sponsored by: IBM Linux Tutorials
Free Linux tutorial presented by Daniel Robbins, President and CEO of
GenToo technologies. Learn everything from fundamentals to system
administration.http://ads.osdn.com/?ad_id=1470&alloc_id=3638&op=click
_______________________________________________
unattended-devel mailing list
[EMAIL PROTECTED]
https://lists.sourceforge.net/lists/listinfo/unattended-devel

Reply via email to