Re: Pardon me for being the n00b here...

On Mon, Oct 15, 2001 at 07:40:10PM +0000, Sean Adams wrote:
> 
> Richard,
> 
> Looks good - I went ahead and checked it into CVS.
> 
> Perhaps a good interface for the "second line" feature would be to use
> HTTP.  You could easily update it from external scripts (or remote
> machines) by doing `lynx http://server/secondline?YOUR_TEXT_HERE`
> 
> Sean


Ok.  For giggles, I just checked out the latest code.  Complaints:

1)  overly-verbose stdout.  Almost looks as though debug is now on
    by default, ignoring command-line switches.

2)  streaming is utterly broken.  It seems as though none of my changes
    were ever incorporated into the codebase, and even worse, the 
    playlist code Dan wrote is now gone or seriously broken.

3)  The web code at line 161 hardcodes an RFC1918-compliant address.
    This is broken.


The new functionality is nice, but not at the sacrifice of the old.
I'd be willing to take some time and re-implement the streaming code
that is now several revs back, and add in the code that will correctly
grab the artist/title from a Live365 stream, but not if it continues
to be left out of the CVS tree.  I can't keep re-implementing the
changes I need every time there's another new feature added.  I've
already sold by referral at least one of these devices based on the
presence of this code alone.






-- 
Mark C. Langston
[email protected]
Systems & Network Admin
http://www.bitshift.org

------------------------ Yahoo! Groups Sponsor ---------------------~-->
Pinpoint the right security solution for your company- Learn how to add 128- bit encryption and to authenticate your web site with VeriSign's FREE guide!
http://us.click.yahoo.com/yQix2C/33_CAA/yigFAA/rIp0lB/TM
---------------------------------------------------------------------~->

To unsubscribe from this group, send an email to:
[email protected]

 

Your use of Yahoo! Groups is subject to http://docs.yahoo.com/info/terms/