GNU bug report logs - #4616
23.1.50; empty script files

Please note: This is a static page, with minimal formatting, updated once a day.
Click here to see this page with the latest information and nicer formatting.

Package: emacs; Severity: minor; Reported by: Markus Rost <rost <at> math.uni-bielefeld.de>; dated Fri, 2 Oct 2009 14:10:06 UTC; Maintainer for emacs is bug-gnu-emacs <at> gnu.org.

Message received at 4616 <at> emacsbugs.donarmstrong.com:

Received: (at 4616) by emacsbugs.donarmstrong.com; 8 Oct 2009 17:43:43 +0000
From rost <at> math.uni-bielefeld.de Thu Oct  8 10:43:42 2009
X-Spam-Checker-Version: SpamAssassin 3.2.5-bugs.debian.org_2005_01_02
	(2008-06-10) on rzlab.ucr.edu
X-Spam-Level: 
X-Spam-Bayes: score:0.5 Bayes not run. spammytokens:Tokens not available.
	hammytokens:Tokens not available.
X-Spam-Status: No, score=-2.2 required=4.0 tests=AWL,HAS_BUG_NUMBER,
	MURPHY_DRUGS_REL8 autolearn=ham version=3.2.5-bugs.debian.org_2005_01_02
Received: from mail2.math.uni-bielefeld.de (mail2.math.uni-bielefeld.de [129.70.14.10])
	by rzlab.ucr.edu (8.14.3/8.14.3/Debian-5) with ESMTP id n98HhfpE028944
	for <4616 <at> emacsbugs.donarmstrong.com>; Thu, 8 Oct 2009 10:43:42 -0700
Received: from sonic02.math.uni-bielefeld.de (sonic02.math.uni-bielefeld.de [129.70.14.134])
	by mail2.math.uni-bielefeld.de (Postfix) with ESMTP id A42B57304B;
	Thu,  8 Oct 2009 19:43:35 +0200 (CEST)
Received: by sonic02.math.uni-bielefeld.de (Postfix, from userid 3710)
	id 11AC9EFE6B; Thu,  8 Oct 2009 19:43:33 +0200 (CEST)
From: Markus Rost <rost <at> math.uni-bielefeld.de>
To: 4616 <at> debbugs.gnu.org
Subject: bug#4616: 23.1.50; empty script files
X-GNU-Emacs-Mailer: GNU Emacs 22.3
X-Home-Page: http://www.math.uni-bielefeld.de/~rost
Message-Id: <20091008174333.11AC9EFE6B <at> sonic02.math.uni-bielefeld.de>
Date: Thu,  8 Oct 2009 19:43:32 +0200 (CEST)

What is the best way to implement --script?  Let me discuss some
possibilities:

1. The current implementation.  It consists of the lines

      if (c == '!')
	{
	  /* #! appears at the beginning of an executable file.
	     Skip the first line.  */
	  while (c != '\n' && c >= 0)
	    c = READCHAR;
	  goto retry;
	}

   in function read1 in lread.c.  This is appealing by its brevity.
   However it has the disadvantage that the basic read function has a
   *general* feature in order to get --script and the "#!"-syntax
   working.  Further it leads to the bug originally posted:  Execution
   of an empty script file like

   ===File ~/x.el==============================================
   #!/home/rost/build/cvs/emacs/src/emacs --script
   ============================================================

   yields an "End of file during parsing" error and returns with positive
   exit status.

2. One may think of handling --script by first removing a first
   #!-line before passing the file to load.  This seems to be very
   hard to implement because the load routine is complicated
   (switching back and forth between C and Elisp functions).  Besides
   this practical problem, there is the following disadvantage:  In
   some circumstances one may want to load a script file directly, as
   in

   emacs -batch ... -l ~/x.el ...

   Moreover, when editing the file one may want to test it with
   eval-buffer.

3. This leaves the possibility to handle #!-lines when running
   eval-region.  It should skip a #!-line at the beginning of the
   region.  It could skip any #!-line in the region, which is what
   read1 does right now, but I see no necessity for that.  This would
   be implemented by my two previous patches

   <URL:http://lists.gnu.org/archive/html/bug-gnu-emacs/2009-10/msg00082.html>
   <URL:http://lists.gnu.org/archive/html/bug-gnu-emacs/2009-10/msg00124.html>

   Of course making a change in readevalloop/read1 needs a careful
   check, maybe by several people.  However, the patch is a very
   "local" one around the variable unread_char and there are only a
   few places where unread_char appears, so checking the patch is not
   too much work.  Moreover, if something would be wrong with the
   change, very likely emacs would not even compile as many files are
   loaded during the build process.

Acknowledgement sent to Markus Rost <rost <at> math.uni-bielefeld.de>:
Extra info received and forwarded to list. Copy sent to Emacs Bugs <bug-gnu-emacs <at> gnu.org>. Full text available.