[gnu.bash.bug] "bash" checks for mail even when interactive

guy@bootme.auspex.com (Guy Harris) (06/13/89)

Which isn't too cool, since that causes messages to get printed even if
the shell was fired up by "rsh", and that causes things like "rcp" to
fall flat on their face....

*** parse.y.orig	Mon Jun 12 16:45:34 1989
--- parse.y	Mon Jun 12 16:46:27 1989
***************
*** 890,895 ****
--- 890,896 ----
     the last two tokens read, and calls read_token.  */
  yylex ()
  {
+   extern int interactive;
    extern int prompt_printable;
    extern char *decode_prompt_string ();
  
***************
*** 901,907 ****
  	 We do this only if it is time to do so. Notice that only here
  	 is the mail alarm reset; nothing takes place in check_mail ()
  	 except the checking of mail.  Please don't change this. */
!       if (time_to_check_mail ())
  	{
  	  check_mail ();
  	  reset_mail_timer ();
--- 902,908 ----
  	 We do this only if it is time to do so. Notice that only here
  	 is the mail alarm reset; nothing takes place in check_mail ()
  	 except the checking of mail.  Please don't change this. */
!       if (interactive && time_to_check_mail ())
  	{
  	  check_mail ();
  	  reset_mail_timer ();

bfox@AUREL.CALTECH.EDU (Brian Fox) (06/13/89)

   Date: Mon, 12 Jun 89 16:52:19 PDT
   From: guy@bootme.auspex.com (Guy Harris)

  "bash" checks for mail even when interactive Which isn't too cool,
  since that causes messages to get printed even if the shell was fired
  up by "rsh", and that causes things like "rcp" to fall flat on their
  face....

Yes, that was true, but I have fixed it in the upcoming 1.00 release.
The fix does NOT belong in parse.y, so remove it.

   *** parse.y.orig	Mon Jun 12 16:45:34 1989
   --- parse.y	Mon Jun 12 16:46:27 1989
   ***************
   *** 890,895 ****
   --- 890,896 ----
	the last two tokens read, and calls read_token.  */
     yylex ()
     {
   +   extern int interactive;
       extern int prompt_printable;
       extern char *decode_prompt_string ();

   ***************
   *** 901,907 ****
	    We do this only if it is time to do so. Notice that only here
	    is the mail alarm reset; nothing takes place in check_mail ()
	    except the checking of mail.  Please don't change this. */
   !       if (time_to_check_mail ())
	   {
	     check_mail ();
	     reset_mail_timer ();
   --- 902,908 ----
	    We do this only if it is time to do so. Notice that only here
	    is the mail alarm reset; nothing takes place in check_mail ()
	    except the checking of mail.  Please don't change this. */
   !       if (interactive && time_to_check_mail ())
	   {
	     check_mail ();
	     reset_mail_timer ();