according to perldoc -f filehandle you can pass the filehandle to the subroutine

open my $sendmail, "|/path/to/sendmail -t $to" or die("couldn't open p
+ipe to sendmail");
print_details($sendmail);
[download]

BTW: you don't have to use the & to call a subroutine, and, IMVHO, should be aware of it's meaning before doing so.

regards,
tomte

Thanks Tomte, I tried your way...but it does not work. I have posted the same question again along with a piece of my code. May be u can check it and then give me a suitable suggestion. I would appreciate if u do so.

edited: Wed Mar 12 16:42:02 2003 by jeffa - i moved your code to this thread, the id is still the same 242417

sub print_details{
print "\nWe have recieved your order for a $pizza_size pizza with ";
&pizza_toppings;
}
[download]

# send e-mail
my $sendmail;
$sendmail="/usr/sbin/sendmail -t";
open(SENDMAIL , "| $sendmail") ||die("unable to open");
print SENDMAIL  "From: $txt_email\n";
print SENDMAIL  "To: abc123@hotmail.com\n";
print SENDMAIL  "Reply-To: abc123@hotmail.com \n"; 
print SENDMAIL  "Subject: YOUR PIZZA ORDER\n" ;
print SENDMAIL &print_details;
close (SENDMAIL);
[download]

Since you didn't escape the @ in your email addresses (and since you didn't use -w which would have told you this), the string @hotmail is being seen as an array, so your email is getting sent out with these headers:

From: somebody@somewhere.com
To: abc123.com
Reply-To abc123.com
Subject: YOUR PIZZA ORDER

---

We're not surrounded, we're in a target-rich environment!

Thanks jasonk, I corrected those two lines with

print SENDMAIL  "To: abc123\@hotmail.com\n";
print SENDMAIL  "Reply-To: abc123\@hotmail.com \n";
[download]

Even with those changes, it does not work...

Well, something like this should do the trick:

sub print_details {
  my $fh = shift;
  print $fh "We have recieved your order for a $pizza_size pizza with 
+";
  pizza_toppings();
 } 

open my $sendmail, "| /usr/sbin/sendmail -t" or die ("no sendmail??");
[...]
print $sendmail "Subject: ...\n\n";
print_details($sendmail);
close $sendmail;
[download]

Just make sure you call print_details everywhere with a filehandle as argument.
If this doesn't help you, read the docs I pointed you to, it's all there. ( and here too: open, perlfaq filehandle )

regards,
tomte

---