bug-gnu-emacs
[Top][All Lists]
Advanced

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

bug#1099: marked as done (23.0.60; Various eshell files need cl at comp


From: Emacs bug Tracking System
Subject: bug#1099: marked as done (23.0.60; Various eshell files need cl at compile time for `assert' )
Date: Wed, 8 Oct 2008 00:50:03 -0700

Your message dated Wed, 08 Oct 2008 03:42:03 -0400
with message-id <qmljwz8rec.fsf@fencepost.gnu.org>
and subject line Re: bug#1099: 23.0.60; Various eshell files need cl at compile 
time for `assert'
has caused the Emacs bug report #1099,
regarding 23.0.60; Various eshell files need cl at compile time for `assert' 
to be marked as done.

This means that you claim that the problem has been dealt with.
If this is not the case it is now your responsibility to reopen the
bug report if necessary, and/or fix the problem forthwith.

(NB: If you are a system administrator and have no idea what this
message is talking about, this may indicate a serious mail system
misconfiguration somewhere. Please contact don@donarmstrong.com
immediately.)


-- 
1099: http://emacsbugs.donarmstrong.com/cgi-bin/bugreport.cgi?bug=1099
Emacs Bug Tracking System
Contact don@donarmstrong.com with problems
--- Begin Message --- Subject: 23.0.60; Various eshell files need cl at compile time for `assert' Date: Mon, 06 Oct 2008 11:01:22 +0200 User-agent: Gnus/5.13 (Gnus v5.13) Emacs/23.0.60 (gnu/linux)
Package: emacs
Version: 23.0.60
Tags: patch

A recent message¹ on help-gnu-emacs showed eshell failing because
`assert' is not defined.  There are no less than six eshell files which
use this macro without requiring cl at compile time.  The following
patch takes care of this:


--8<---------------cut here---------------start------------->8---
Index: lisp/eshell/em-cmpl.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/em-cmpl.el,v
retrieving revision 1.27
diff -c -r1.27 em-cmpl.el
*** lisp/eshell/em-cmpl.el      21 May 2008 03:51:06 -0000      1.27
--- lisp/eshell/em-cmpl.el      6 Oct 2008 08:25:44 -0000
***************
*** 71,76 ****
--- 71,77 ----
  ;;; Code:
  
  (eval-when-compile
+   (require 'cl)
    (require 'eshell))
  (require 'esh-util)
  
Index: lisp/eshell/em-hist.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/em-hist.el,v
retrieving revision 1.29
diff -c -r1.29 em-hist.el
*** lisp/eshell/em-hist.el      21 May 2008 03:51:06 -0000      1.29
--- lisp/eshell/em-hist.el      6 Oct 2008 08:25:44 -0000
***************
*** 55,60 ****
--- 55,63 ----
  
  ;;; Code:
  
+ (eval-when-compile
+   (require 'cl))
+ 
  (require 'ring)
  (require 'esh-opt)
  (require 'em-pred)
Index: lisp/eshell/em-ls.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/em-ls.el,v
retrieving revision 1.34
diff -c -r1.34 em-ls.el
*** lisp/eshell/em-ls.el        15 Aug 2008 06:55:37 -0000      1.34
--- lisp/eshell/em-ls.el        6 Oct 2008 08:25:44 -0000
***************
*** 27,33 ****
  
  ;;; Code:
  
! (eval-when-compile (require 'eshell))
  (require 'esh-util)
  (require 'esh-opt)
  
--- 27,35 ----
  
  ;;; Code:
  
! (eval-when-compile
!   (require 'cl)
!   (require 'eshell))
  (require 'esh-util)
  (require 'esh-opt)
  
Index: lisp/eshell/esh-cmd.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/esh-cmd.el,v
retrieving revision 1.43
diff -c -r1.43 esh-cmd.el
*** lisp/eshell/esh-cmd.el      15 Aug 2008 06:55:11 -0000      1.43
--- lisp/eshell/esh-cmd.el      6 Oct 2008 08:25:45 -0000
***************
*** 109,114 ****
--- 109,115 ----
  (require 'esh-ext)
  
  (eval-when-compile
+   (require 'cl)
    (require 'pcomplete))
  
  
Index: lisp/eshell/esh-ext.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/esh-ext.el,v
retrieving revision 1.29
diff -c -r1.29 esh-ext.el
*** lisp/eshell/esh-ext.el      6 May 2008 03:36:20 -0000       1.29
--- lisp/eshell/esh-ext.el      6 Oct 2008 08:25:45 -0000
***************
*** 33,38 ****
--- 33,39 ----
  (provide 'esh-ext)
  
  (eval-when-compile
+   (require 'cl)
    (require 'esh-cmd))
  (require 'esh-util)
  
Index: lisp/eshell/esh-io.el
===================================================================
RCS file: /sources/emacs/emacs/lisp/eshell/esh-io.el,v
retrieving revision 1.23
diff -c -r1.23 esh-io.el
*** lisp/eshell/esh-io.el       7 Jun 2008 02:37:13 -0000       1.23
--- lisp/eshell/esh-io.el       6 Oct 2008 08:25:45 -0000
***************
*** 57,63 ****
  
  (provide 'esh-io)
  
! (eval-when-compile (require 'eshell))
  
  (defgroup eshell-io nil
    "Eshell's I/O management code provides a scheme for treating many
--- 57,65 ----
  
  (provide 'esh-io)
  
! (eval-when-compile
!   (require 'cl)
!   (require 'eshell))
  
  (defgroup eshell-io nil
    "Eshell's I/O management code provides a scheme for treating many
--8<---------------cut here---------------end--------------->8---


Suggested Changelog entry:

--8<---------------cut here---------------start------------->8---
2008-10-06  Sven Joachim  <svenjoac@gmx.de>

        * eshell/em-cmpl.el (require):
        * eshell/em-hist.el (require):
        * eshell/em-ls.el (require):
        * eshell/esh-cmd.el (require):
        * eshell/esh-ext.el (require):
        * eshell/esh-io.el (require): Require cl at compile time because
        `assert' needs it.

--8<---------------cut here---------------end--------------->8---


¹ http://lists.gnu.org/archive/html/help-gnu-emacs/2008-10/msg00120.html



--- End Message ---
--- Begin Message --- Subject: Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert' Date: Wed, 08 Oct 2008 03:42:03 -0400 User-agent: Gnus (www.gnus.org), GNU Emacs (www.gnu.org/software/emacs/)
Sven Joachim wrote:

> A recent message¹ on help-gnu-emacs showed eshell failing because
> `assert' is not defined.  

Unreproducible.

>There are no less than six eshell files which use this macro without
>requiring cl at compile time. The following patch takes care of this:

I don't think there is a real problem here, because cl is brought in
through other requires, nevertheless for the sake of clarity I have
installed your patch.


--- End Message ---

reply via email to

[Prev in Thread] Current Thread [Next in Thread]