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.