* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to [not found] <E1Uj5Lc-0007VJ-NY@vcs.savannah.gnu.org> @ 2013-06-02 18:26 ` Glenn Morris 2013-06-02 18:28 ` Glenn Morris 2013-06-03 6:37 ` Tassilo Horn 0 siblings, 2 replies; 11+ messages in thread From: Glenn Morris @ 2013-06-02 18:26 UTC (permalink / raw) To: Tassilo Horn; +Cc: Emacs developers Tassilo Horn wrote: > ------------------------------------------------------------ > + (intersection args em-term.el:321:1:Warning: the function `intersection' is not known to be defined. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-02 18:26 ` /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to Glenn Morris @ 2013-06-02 18:28 ` Glenn Morris 2013-06-03 6:38 ` Tassilo Horn 2013-06-03 6:37 ` Tassilo Horn 1 sibling, 1 reply; 11+ messages in thread From: Glenn Morris @ 2013-06-02 18:28 UTC (permalink / raw) To: Tassilo Horn; +Cc: Emacs developers PS please add a NEWS entry. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-02 18:28 ` Glenn Morris @ 2013-06-03 6:38 ` Tassilo Horn 0 siblings, 0 replies; 11+ messages in thread From: Tassilo Horn @ 2013-06-03 6:38 UTC (permalink / raw) To: Glenn Morris; +Cc: Emacs developers Glenn Morris <rgm@gnu.org> writes: > PS please add a NEWS entry. Done. Bye, Tassilo ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-02 18:26 ` /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to Glenn Morris 2013-06-02 18:28 ` Glenn Morris @ 2013-06-03 6:37 ` Tassilo Horn 2013-06-03 7:31 ` Glenn Morris 1 sibling, 1 reply; 11+ messages in thread From: Tassilo Horn @ 2013-06-03 6:37 UTC (permalink / raw) To: Glenn Morris; +Cc: Emacs developers Glenn Morris <rgm@gnu.org> writes: Hi Glenn, >> ------------------------------------------------------------ >> + (intersection args > > em-term.el:321:1:Warning: the function `intersection' is not known to be > defined. Thanks for the heads up. I've changed it to `cl-intersection'. And thanks for adding the missing :version tags to the new defcustoms. Bye, Tassilo ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 6:37 ` Tassilo Horn @ 2013-06-03 7:31 ` Glenn Morris 2013-06-03 8:37 ` Tassilo Horn 0 siblings, 1 reply; 11+ messages in thread From: Glenn Morris @ 2013-06-03 7:31 UTC (permalink / raw) To: Emacs developers Tassilo Horn wrote: > Thanks for the heads up. I've changed it to `cl-intersection'. You still need to actually require cl-lib. ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 7:31 ` Glenn Morris @ 2013-06-03 8:37 ` Tassilo Horn 2013-06-03 15:12 ` Glenn Morris 0 siblings, 1 reply; 11+ messages in thread From: Tassilo Horn @ 2013-06-03 8:37 UTC (permalink / raw) To: Glenn Morris; +Cc: Emacs developers Glenn Morris <rgm@gnu.org> writes: > Tassilo Horn wrote: > >> Thanks for the heads up. I've changed it to `cl-intersection'. > > You still need to actually require cl-lib. Really? It says cl-intersection is an *autoloaded* compiled Lisp function in `cl-seq.el'. Bye, Tassilo ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 8:37 ` Tassilo Horn @ 2013-06-03 15:12 ` Glenn Morris 2013-06-03 18:22 ` Tassilo Horn 0 siblings, 1 reply; 11+ messages in thread From: Glenn Morris @ 2013-06-03 15:12 UTC (permalink / raw) To: Emacs developers Tassilo Horn wrote: > Really? It says > > cl-intersection is an *autoloaded* compiled Lisp function in > `cl-seq.el'. Yes, really. Is it so hard to run emacs -Q? It is autoloaded from cl-seq once you load cl-lib. (I suppose this is an instance where the "is an autoloaded function" blurb is not that useful.) ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 15:12 ` Glenn Morris @ 2013-06-03 18:22 ` Tassilo Horn 2013-06-03 19:42 ` Glenn Morris 0 siblings, 1 reply; 11+ messages in thread From: Tassilo Horn @ 2013-06-03 18:22 UTC (permalink / raw) To: Glenn Morris; +Cc: Emacs developers Glenn Morris <rgm@gnu.org> writes: >> Really? It says >> >> cl-intersection is an *autoloaded* compiled Lisp function in >> `cl-seq.el'. > > Yes, really. Ok, I added a require. > Is it so hard to run emacs -Q? Rest assured I did that. em-term.el requires esh-ext which already requires cl-lib. So neither do you get an error at runtime nor a warning when compiling... Bye, Tassilo ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 18:22 ` Tassilo Horn @ 2013-06-03 19:42 ` Glenn Morris 2013-06-03 20:32 ` Glenn Morris 2013-06-04 6:33 ` Tassilo Horn 0 siblings, 2 replies; 11+ messages in thread From: Glenn Morris @ 2013-06-03 19:42 UTC (permalink / raw) To: Emacs developers Tassilo Horn wrote: > em-term.el requires esh-ext which already requires cl-lib. Only at compile time, not at runtime. > So neither do you get an error at runtime nor a warning when > compiling... Previously, the only thing in eshell that required cl-lib at run-time was em-ls. So if you remove eshell-ls from the list of eshell modules: emacs -Q --eval "(setq eshell-modules-list '(eshell-alias eshell-banner eshell-basic eshell-cmpl eshell-dirs eshell-glob eshell-hist eshell-pred eshell-prompt eshell-script eshell-term eshell-unix))" -f eshell there was indeed an error at runtime. I don't know why there is no warning during bootstrap, but there is a warning from a built emacs: emacs -Q -batch -f batch-byte-compile em-term.el In end of data: em-term.el:321:1:Warning: the function `cl-intersection' is not known to be defined. Hmm, looks like you get a warning only if esh-ext.el is already compiled when you compiled em-term.el. I think this might be a (recent) compiler bug, maybe my 2013-05-22 change? Pedantically yours, ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 19:42 ` Glenn Morris @ 2013-06-03 20:32 ` Glenn Morris 2013-06-04 6:33 ` Tassilo Horn 1 sibling, 0 replies; 11+ messages in thread From: Glenn Morris @ 2013-06-03 20:32 UTC (permalink / raw) To: Emacs developers Glenn Morris wrote: > Hmm, looks like you get a warning only if esh-ext.el is already compiled > when you compiled em-term.el. I think this might be a (recent) compiler > bug, maybe my 2013-05-22 change? No, looks like it always been like that. I filed http://debbugs.gnu.org/14551 but it's probably unfixable (?). ^ permalink raw reply [flat|nested] 11+ messages in thread
* Re: /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to 2013-06-03 19:42 ` Glenn Morris 2013-06-03 20:32 ` Glenn Morris @ 2013-06-04 6:33 ` Tassilo Horn 1 sibling, 0 replies; 11+ messages in thread From: Tassilo Horn @ 2013-06-04 6:33 UTC (permalink / raw) To: Glenn Morris; +Cc: Emacs developers Glenn Morris <rgm@gnu.org> writes: >> em-term.el requires esh-ext which already requires cl-lib. > > Only at compile time, not at runtime. Oh, right. I confused the "equivalent to progn in interpreted code" in the docstring of `eval-when-compile' with "at runtime", which is of course wrong. >> So neither do you get an error at runtime nor a warning when >> compiling... > > Previously, the only thing in eshell that required cl-lib at run-time > was em-ls. So if you remove eshell-ls from the list of eshell modules: > > emacs -Q --eval "(setq eshell-modules-list '(eshell-alias eshell-banner > eshell-basic eshell-cmpl eshell-dirs eshell-glob eshell-hist eshell-pred > eshell-prompt eshell-script eshell-term eshell-unix))" -f eshell > > there was indeed an error at runtime. > > I don't know why there is no warning during bootstrap, but there is > a warning from a built emacs: > > emacs -Q -batch -f batch-byte-compile em-term.el > > In end of data: > em-term.el:321:1:Warning: the function `cl-intersection' is not known to > be defined. > > Hmm, looks like you get a warning only if esh-ext.el is already compiled > when you compiled em-term.el. I think this might be a (recent) compiler > bug, maybe my 2013-05-22 change? > > Pedantically yours, Thanks for the explanation. Bye, Tassilo ^ permalink raw reply [flat|nested] 11+ messages in thread
end of thread, other threads:[~2013-06-04 6:33 UTC | newest] Thread overview: 11+ messages (download: mbox.gz follow: Atom feed -- links below jump to the message on this page -- [not found] <E1Uj5Lc-0007VJ-NY@vcs.savannah.gnu.org> 2013-06-02 18:26 ` /srv/bzr/emacs/trunk r112822: * eshell/esh-ext.el (eshell-external-command): Pass args to Glenn Morris 2013-06-02 18:28 ` Glenn Morris 2013-06-03 6:38 ` Tassilo Horn 2013-06-03 6:37 ` Tassilo Horn 2013-06-03 7:31 ` Glenn Morris 2013-06-03 8:37 ` Tassilo Horn 2013-06-03 15:12 ` Glenn Morris 2013-06-03 18:22 ` Tassilo Horn 2013-06-03 19:42 ` Glenn Morris 2013-06-03 20:32 ` Glenn Morris 2013-06-04 6:33 ` Tassilo Horn
Code repositories for project(s) associated with this public inbox https://git.savannah.gnu.org/cgit/emacs.git This is a public inbox, see mirroring instructions for how to clone and mirror all data and code used for this inbox; as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).