Ueland / VikingBot

Vikingbot is yet another simple PHP based IRC bot with support for plugins and secure IRC servers. The bot requires Unix/Linux shell access with PHP support and SSL support in PHP for use against secure IRC servers.
GNU General Public License v3.0
40 stars 6 forks source link

created some new stuff for vikingbot #10

Open coderofsalvation opened 11 years ago

coderofsalvation commented 11 years ago

Like the bot bro, and I found some time to create some fixes & plugins. Feel free to merge stuff from my fork into yours!

The Coder of Salvation

Ueland commented 11 years ago

Hi! Awesome, will have a look and do a merge back into Vikingbot ;)

coderofsalvation commented 11 years ago

Hehe cool :)

Please have a critical look..especially at the system()-calls. For example the 'remindPlugin' (which remembers important links, or mentioning of nicknames):

I chose for system( escapeshellcmd("cat '{$msg}' >> log.txt"); because the >> and grep shellcommands are optimized for superfast dealing with large textfiles.

If I would call file_get_contents() and file_put_contents() instead, it would make the bot very slow for just appending a line to a large textfile.

Today on the way back home, I was thinking..isnt there a way we can achieve the same effect of >> with fopen() and fwrite() ? As in just opening the filepointer (not reading the whole data), and write to the end of the file.

hm lets think about it, rome wasnt build in one day :)

Leon

ps1. here's a 'vikingbot'-extension for the famous roundup issuetracker, feel free to mention it : https://gist.github.com/3295338 ps2. also do not..I repeat not merge the start.sh file. I tell you this because today in the morning I had to call my ISP to restart our webserver :) The problem was that I added this:

while 1: do

start bot

done

Not knowing that you already created a 'restart bot when things go wrong'-mechanism in php. The exec("start.sh &")-call spawned infinite bots with each containing a memory leak (I was working on a new plugin hehe).

On Wed, 8 Aug 2012, Tor Henning Ueland wrote:

Hi! Awesome, will have a look and do a merge back into Vikingbot ;)


Reply to this email directly or view it on GitHub: https://github.com/Ueland/VikingBot/issues/10#issuecomment-7588992

Ueland commented 11 years ago

Hi!

I havent forgotten this yet, i have not had time to go over / merge yet, but i have started now by adding a link to your roundout issue tracker plugin.

hashworks commented 9 years ago

I would suggest to close this. Most of the stuff is kind of specific. Plugins should be posted in own repositories.