Skip Menu |
 

This queue is for tickets about the Bot-BasicBot CPAN distribution.

Report information
The Basics
Id: 59597
Status: resolved
Priority: 0/
Queue: Bot-BasicBot

People
Owner: Nobody in particular
Requestors: hinrik.sig [...] gmail.com
Cc:
AdminCc:

Bug Information
Severity: Critical
Broken in: (no value)
Fixed in: (no value)



Subject: [PATCH] Fixed the reconnect and shutdown code
MIME-Version: 1.0
Content-Type: text/plain; charset="UTF-8"
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 614
Download (untitled) / with headers
text/plain 614b
Bot::BasicBot currently has its own reconnect code from way back, before POE::Component::IRC had the Connector plugin. I removed the old code and loaded the Connector plugin instead, as well as fixing up the shutdown code (currently there's no way to shut down the bot, it will constantly try to reconnect). I added a bot method called "quit" to quit from IRC and shut down gracefully. If you have a policy against naming methods after POE::Component::IRC events (so that those are still accessible via AUTOLOAD), you might want to rename it. http://github.com/hinrik/Bot-BasicBot/compare/aadca7c3f6...0ed4920410
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2841-1280739493-608.59597-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 178
Download (untitled) / with headers
text/plain 178b
My initial patches didn't take care of killing subprocesses created with forkit(), but I've fixed that now: http://github.com/hinrik/Bot-BasicBot/compare/aadca7c3f6...e341ebbd71
MIME-Version: 1.0
X-Mailer: MIME-tools 5.427 (Entity 5.427)
Content-Disposition: inline
Content-Type: text/plain; charset="UTF-8"
Message-ID: <rt-3.8.HEAD-2362-1288782660-1443.59597-0-0 [...] rt.cpan.org>
Content-Transfer-Encoding: binary
X-RT-Original-Encoding: utf-8
Content-Length: 39
This has been fixed as of version 0.82.


This service is sponsored and maintained by Best Practical Solutions and runs on Perl.org infrastructure.

Please report any issues with rt.cpan.org to rt-cpan-admin@bestpractical.com.