Comment 1 for bug 903361

Awesome stuff! Just a few things I noticed:

Overall nice charm!

1) Since Alice takes over the output buffer, having it "start" in the install script blocks the install script and expose never gets run. I'd recommend just removing it from the install hook since you have it in the start hook already

2) It appears to install to /usr/bin/alice and not /bin/alice as is referenced in the start hook, using just `alice` should suffice since it's in the PATH

===Minor nitpicks.===

3) From charm proof:
> W: all charms should provide at least one thing
While it's not vital, charms should provide _something_, not sure if it's scale-safe or not with sessions, but it would be interesting to see if this could operate behind a load balancer or a proxy.

4) There doesn't seem to be any way to configure Alice outside of the defaults it creates, might be beneficial to allow users to configure things like quitmsg, ignore, highlights

5) If you implement configuration items, it might be a good idea to just write an upstart script for alice as well, so you can easily restart/start/stop when configuration changes are made.

Great so far! I know I'll definitely be using this :D