IRC status does not like builders with spaces in them

[TTimo]

buildbot 0.7.7

I have several builders with spaces in their names (for instance 'Windows build'), and the IRC bot doesn't understand statements such as:

bot: watch Windows build or bot: watch 'Windows build'

[warner]

yeah, that's because the irc-commadn parser is splitting on spaces. I'd suggest not using spaces in your builder names, and we should implement the separate "builder description string" (as a space-friendly adjunct to the builder name) described in #248 and #10. At least that description string would let you make the waterfall page look better (by using the description string instead of the builder name as the column heading)

[ashcrow]

I'll see if I can get one space in the name to be valid. To allow for spaces in general we will need to decide to either create a general parser that understands input or settle on a quoting standard for use.

[ashcrow]

After looking a bit more into it I think it might be simpler to allow grouping with " or ' like so

bot: force build "my builder"

or

bot: watch 'some builder'

[dustin]

This sounds good. I want to say there's something in the Python standard library that will split a quoted string like that, but I may be thinking of Amanda::Util::split_quoted_string .. sometimes all of my projects blur together :)

[ashcrow]

I wrote a regex splitter which should do the job ...

rx = re.compile("['"]+(?:[^'"]+)['"]+|(?:[^ '"]+)")

I just need to write it together and test :-)

[dustin]

Shlex is the utility that does this. It's already in use for the 'force' subcommand, so it should be easy enough to add it for the 'watch' subcommand.

[dustin]

commit 0c51ff9e5b3e358d3f2e268f5f3cf81a9ac1eb3f
Author: Dustin J. Mitchell <dustin@zmanda.com>
Date:   Sat Dec 19 09:49:32 2009 -0600

    (fixes #220) use shlex.split everywhere in the IRC bot