"bash" checks for mail even when interactive

Thread View: gnu.bash.bug

2 messages

[T] Back to Threads [V] Tree View [G] Group Articles

2 total messages Started by Guy Harris Mon, 12 Jun 1989 23:52

#1017

Author: Guy Harris
Date: Mon, 12 Jun 1989 23:52

33 lines
1541 bytes

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 ();

"bash" checks for mail even when interactive

#1018

Author: Brian Fox
Date: Tue, 13 Jun 1989 00:32

41 lines
1937 bytes

   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 ();

Thread Navigation

This is a paginated view of messages in the thread with full content displayed inline.

Messages are displayed in chronological order, with the original post highlighted in green.

Use pagination controls to navigate through all messages in large threads.

Back to All Threads