* bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert' @ 2008-10-06 9:01 ` Sven Joachim 2008-10-08 7:50 ` bug#1099: marked as done (23.0.60; Various eshell files need cl at compile time for `assert' ) Emacs bug Tracking System [not found] ` <handler.1099.D1099.122345187014602.notifdone@emacsbugs.donarmstrong.com> 0 siblings, 2 replies; 5+ messages in thread From: Sven Joachim @ 2008-10-06 9:01 UTC (permalink / raw) To: emacs-pretest-bug 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 ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#1099: marked as done (23.0.60; Various eshell files need cl at compile time for `assert' ) 2008-10-06 9:01 ` bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert' Sven Joachim @ 2008-10-08 7:50 ` Emacs bug Tracking System [not found] ` <handler.1099.D1099.122345187014602.notifdone@emacsbugs.donarmstrong.com> 1 sibling, 0 replies; 5+ messages in thread From: Emacs bug Tracking System @ 2008-10-08 7:50 UTC (permalink / raw) To: Glenn Morris [-- Attachment #1: Type: text/plain, Size: 911 bytes --] 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 [-- Attachment #2: Type: message/rfc822, Size: 7107 bytes --] From: Sven Joachim <svenjoac@gmx.de> To: emacs-pretest-bug@gnu.org Subject: 23.0.60; Various eshell files need cl at compile time for `assert' Date: Mon, 06 Oct 2008 11:01:22 +0200 Message-ID: <87bpxy2j31.fsf@gmx.de> 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 [-- Attachment #3: Type: message/rfc822, Size: 1960 bytes --] From: Glenn Morris <rgm@gnu.org> To: 1099-done@emacsbugs.donarmstrong.com 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 Message-ID: <qmljwz8rec.fsf@fencepost.gnu.org> 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. ^ permalink raw reply [flat|nested] 5+ messages in thread
[parent not found: <handler.1099.D1099.122345187014602.notifdone@emacsbugs.donarmstrong.com>]
* bug#1099: closed by Glenn Morris <rgm@gnu.org> (Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert') [not found] ` <handler.1099.D1099.122345187014602.notifdone@emacsbugs.donarmstrong.com> @ 2008-10-08 8:28 ` Sven Joachim 2008-10-08 16:37 ` Glenn Morris 0 siblings, 1 reply; 5+ messages in thread From: Sven Joachim @ 2008-10-08 8:28 UTC (permalink / raw) To: Glenn Morris, 1099 > From: Glenn Morris <rgm@gnu.org> > Subject: Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert' > To: 1099-done@emacsbugs.donarmstrong.com > 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. I had the same problem, though. It went away after bootstrapping. >>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, It isn't. When I byte-compiled the affected files manually, I got a warning that `assert' is not known to be defined. Maybe you have cl loaded in your environment? > nevertheless for the sake of clarity I have > installed your patch. Thanks. Sven ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#1099: closed by Glenn Morris <rgm@gnu.org> (Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert') 2008-10-08 8:28 ` bug#1099: closed by Glenn Morris <rgm@gnu.org> (Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert') Sven Joachim @ 2008-10-08 16:37 ` Glenn Morris 2008-10-08 17:25 ` Sven Joachim 0 siblings, 1 reply; 5+ messages in thread From: Glenn Morris @ 2008-10-08 16:37 UTC (permalink / raw) To: Sven Joachim; +Cc: 1099 I think you might have seen such a problem if eshell.el was compiled first, which is not something that happens during a normal serial bootstrap, due to the way the files happen to sort. ^ permalink raw reply [flat|nested] 5+ messages in thread
* bug#1099: closed by Glenn Morris <rgm@gnu.org> (Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert') 2008-10-08 16:37 ` Glenn Morris @ 2008-10-08 17:25 ` Sven Joachim 0 siblings, 0 replies; 5+ messages in thread From: Sven Joachim @ 2008-10-08 17:25 UTC (permalink / raw) To: Glenn Morris; +Cc: 1099 On 2008-10-08 18:37 +0200, Glenn Morris wrote: > I think you might have seen such a problem if eshell.el was compiled > first, which is not something that happens during a normal serial > bootstrap, due to the way the files happen to sort. Exactly. It must have happened when one of the files was recompiled after `cvs up'. ^ permalink raw reply [flat|nested] 5+ messages in thread
end of thread, other threads:[~2008-10-08 17:25 UTC | newest] Thread overview: 5+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <qmljwz8rec.fsf@fencepost.gnu.org> 2008-10-06 9:01 ` bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert' Sven Joachim 2008-10-08 7:50 ` bug#1099: marked as done (23.0.60; Various eshell files need cl at compile time for `assert' ) Emacs bug Tracking System [not found] ` <handler.1099.D1099.122345187014602.notifdone@emacsbugs.donarmstrong.com> 2008-10-08 8:28 ` bug#1099: closed by Glenn Morris <rgm@gnu.org> (Re: bug#1099: 23.0.60; Various eshell files need cl at compile time for `assert') Sven Joachim 2008-10-08 16:37 ` Glenn Morris 2008-10-08 17:25 ` Sven Joachim
Code repositories for project(s) associated with this external index https://git.savannah.gnu.org/cgit/emacs.git https://git.savannah.gnu.org/cgit/emacs/org-mode.git This is an external index of several public inboxes, see mirroring instructions on how to clone and mirror all data and code used by this external index.