Well, your version comparison logic doesn't seem right, but I guess that depends on what you are wanting.
I would suggest starting as simple as possible with something like this:
Code:
sub EVENT_SAY {
my $ClientVer = $client->GetClientVersion();
quest::say("You are on Client Version: $ClientVer");
}
Then work your way up from there. Most likely something else in your script is breaking the script, which is causing the NPC not to respond.