unofficial mirror of emacs-devel@gnu.org 
 help / color / mirror / code / Atom feed
* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
       [not found] <E1RSVcR-0007q0-22@vcs.savannah.gnu.org>
@ 2011-11-22  8:50 ` Juanma Barranquero
  2011-11-22 16:43   ` Stefan Monnier
  0 siblings, 1 reply; 9+ messages in thread
From: Juanma Barranquero @ 2011-11-22  8:50 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

> revno: 106462
> committer: Stefan Monnier <monnier@iro.umontreal.ca>
> branch nick: trunk
> timestamp: Mon 2011-11-21 09:22:02 -0500
> message:
>  * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.


"../src/oo/i386/emacs.exe" -batch --no-site-file --no-site-lisp -l autoload \
                --eval "(setq find-file-hook nil
find-file-suppress-same-file-warnings t)" \
                -f w32-batch-update-autoloads
"C:/emacs/trunk/lisp/loaddefs.el" make . calc calendar emacs-lisp
emulation erc eshell gnus i
nternational language mail mh-e net nxml org play progmodes textmodes
url vc cedet cedet/ede cedet/semantic cedet/srecode
cedet/semantic/analyze cedet
/semantic/bovine cedet/semantic/decorate cedet/semantic/symref
cedet/semantic/wisent
Wrong type argument: listp, "6341f93b6ab75d19df0532a6baba3250"
make: *** [autoloads] Error -1

It does not fail afer reverting your change.

  Juanma



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-22  8:50 ` [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify Juanma Barranquero
@ 2011-11-22 16:43   ` Stefan Monnier
  2011-11-23  0:42     ` Juanma Barranquero
  0 siblings, 1 reply; 9+ messages in thread
From: Stefan Monnier @ 2011-11-22 16:43 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: emacs-devel

>> revno: 106462
>> committer: Stefan Monnier <monnier@iro.umontreal.ca>
>> branch nick: trunk
>> timestamp: Mon 2011-11-21 09:22:02 -0500
>> message:
>>  * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.


> "../src/oo/i386/emacs.exe" -batch --no-site-file --no-site-lisp -l autoload \
>                 --eval "(setq find-file-hook nil
> find-file-suppress-same-file-warnings t)" \
>                 -f w32-batch-update-autoloads
> "C:/emacs/trunk/lisp/loaddefs.el" make . calc calendar emacs-lisp
> emulation erc eshell gnus i
> nternational language mail mh-e net nxml org play progmodes textmodes
> url vc cedet cedet/ede cedet/semantic cedet/srecode
> cedet/semantic/analyze cedet
> /semantic/bovine cedet/semantic/decorate cedet/semantic/symref
> cedet/semantic/wisent
> Wrong type argument: listp, "6341f93b6ab75d19df0532a6baba3250"
> make: *** [autoloads] Error -1

> It does not fail afer reverting your change.

Hmm... I can't reproduce it here.  Can you show a backtrace, as well as
some context (does it fail for bootstrap or only for subsequent
refreshes)?


        Stefan



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-22 16:43   ` Stefan Monnier
@ 2011-11-23  0:42     ` Juanma Barranquero
  2011-11-23 13:47       ` Andy Moreton
  2011-11-23 18:42       ` [Emacs-diffs] " Eli Zaretskii
  0 siblings, 2 replies; 9+ messages in thread
From: Juanma Barranquero @ 2011-11-23  0:42 UTC (permalink / raw)
  To: Stefan Monnier; +Cc: emacs-devel

On Tue, Nov 22, 2011 at 17:43, Stefan Monnier <monnier@iro.umontreal.ca> wrote:

> Can you show a backtrace

Debugger entered--Lisp error: (wrong-type-argument listp
"7a7dc38245a868ec15a09346dc1ecc35")
  time-less-p("7a7dc38245a868ec15a09346dc1ecc35" (20168 31602))
  update-directory-autoloads("." "calc" "calendar" "emacs-lisp"
"emulation" "erc" "eshell" "gnus" "international" "language" "mail"
"mh-e" "net" "nxml" "org" "play" "progmodes" "textmodes" "url" "vc"
"cedet" "cedet/ede" "cedet/semantic" "cedet/srecode"
"cedet/semantic/analyze" "cedet/semantic/bovine"
"cedet/semantic/decorate" "cedet/semantic/symref"
"cedet/semantic/wisent")
  apply(update-directory-autoloads ("." "calc" "calendar" "emacs-lisp"
"emulation" "erc" "eshell" "gnus" "international" "language" "mail"
"mh-e" "net" "nxml" "org" "play" "progmodes" "textmodes" "url" "vc"
"cedet" "cedet/ede" "cedet/semantic" "cedet/srecode"
"cedet/semantic/analyze" "cedet/semantic/bovine"
"cedet/semantic/decorate" "cedet/semantic/symref"
"cedet/semantic/wisent"))
  batch-update-autoloads()
  w32-batch-update-autoloads()
  command-line-1(("--eval" "(toggle-debug-on-error)" "--eval" "(setq
find-file-hook nil find-file-suppress-same-file-warnings t)" "-f"
"w32-batch-update-autoloads" "C:/emacs/debug/lisp/loaddefs.el" "make"
"." "calc" "calendar" "emacs-lisp" "emulation" "erc" "eshell" "gnus"
"international" "language" "mail" "mh-e" "net" "nxml" "org" "play"
"progmodes" "textmodes" "url" "vc" "cedet" "cedet/ede"
"cedet/semantic" "cedet/srecode" "cedet/semantic/analyze"
"cedet/semantic/bovine" "cedet/semantic/decorate"
"cedet/semantic/symref" "cedet/semantic/wisent"))
  command-line()
  normal-top-level()

> as well as some context (does it fail for bootstrap

No.

> or only for subsequent refreshes)?

Yes. Simply

  cd lisp
  make autoloads

    Juanma



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23  0:42     ` Juanma Barranquero
@ 2011-11-23 13:47       ` Andy Moreton
  2011-11-23 17:58         ` Eli Zaretskii
  2011-11-23 18:42       ` [Emacs-diffs] " Eli Zaretskii
  1 sibling, 1 reply; 9+ messages in thread
From: Andy Moreton @ 2011-11-23 13:47 UTC (permalink / raw)
  To: emacs-devel

On Wed 23 Nov 2011, Juanma Barranquero wrote:

> On Tue, Nov 22, 2011 at 17:43, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
>
>> or only for subsequent refreshes)?
>
> Yes. Simply
>
>   cd lisp
>   make autoloads

I see the same problems, with a similar backtrace. 

Stefan, could you please revert your change to allow emacs to build
properly ? A full analysis can then be condusted at leasure without
breaking the build for Windows users.

Thanks,

    AndyM




^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23 13:47       ` Andy Moreton
@ 2011-11-23 17:58         ` Eli Zaretskii
  2011-11-23 23:41           ` Andy Moreton
  0 siblings, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2011-11-23 17:58 UTC (permalink / raw)
  To: Andy Moreton; +Cc: emacs-devel

> From: Andy Moreton <andrewjmoreton@gmail.com>
> Date: Wed, 23 Nov 2011 13:47:03 +0000
> 
> Stefan, could you please revert your change to allow emacs to build
> properly ?

What prevents you from reverting it in your branch?

Btw, at least for me, reverting that particular change in autoload.el
didn't fix the problem.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23  0:42     ` Juanma Barranquero
  2011-11-23 13:47       ` Andy Moreton
@ 2011-11-23 18:42       ` Eli Zaretskii
  2011-11-23 19:21         ` martin rudalics
  1 sibling, 1 reply; 9+ messages in thread
From: Eli Zaretskii @ 2011-11-23 18:42 UTC (permalink / raw)
  To: Juanma Barranquero; +Cc: monnier, emacs-devel

> From: Juanma Barranquero <lekktu@gmail.com>
> Date: Wed, 23 Nov 2011 01:42:57 +0100
> Cc: emacs-devel@gnu.org
> 
> On Tue, Nov 22, 2011 at 17:43, Stefan Monnier <monnier@iro.umontreal.ca> wrote:
> 
> > Can you show a backtrace
> 
> Debugger entered--Lisp error: (wrong-type-argument listp
> "7a7dc38245a868ec15a09346dc1ecc35")
>   time-less-p("7a7dc38245a868ec15a09346dc1ecc35" (20168 31602))
>   update-directory-autoloads("." "calc" "calendar" "emacs-lisp"

The problem was that on Windows, "make autoloads" produced in
loaddefs.el MD5 checksums, but Emacs expects to see time stamps in
that file (compare with ldefs-boot.el).

And the reason "make autoloads" produced MD5 checksums is that
autoload-generate-file-autoloads compared file names case-sensitively,
so d:/foo/bar/lisp/loaddefs.el and D:/foo/bar/lisp/loaddefs.el would
not compare equal!

Should be fixed now.  You may need to copy ldefs-boot.el to
loaddefs.el, before you run "make autoloads" after you "bzr up" and
recompile autoload.el.



^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23 18:42       ` [Emacs-diffs] " Eli Zaretskii
@ 2011-11-23 19:21         ` martin rudalics
  0 siblings, 0 replies; 9+ messages in thread
From: martin rudalics @ 2011-11-23 19:21 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Juanma Barranquero, monnier, emacs-devel

> Should be fixed now.

Many thanks.

martin




^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23 17:58         ` Eli Zaretskii
@ 2011-11-23 23:41           ` Andy Moreton
  2011-11-24  3:54             ` Eli Zaretskii
  0 siblings, 1 reply; 9+ messages in thread
From: Andy Moreton @ 2011-11-23 23:41 UTC (permalink / raw)
  To: emacs-devel

On Wed 23 Nov 2011, Eli Zaretskii wrote:

>> From: Andy Moreton <andrewjmoreton@gmail.com>
>> Date: Wed, 23 Nov 2011 13:47:03 +0000
>> 
>> Stefan, could you please revert your change to allow emacs to build
>> properly ?
>
> What prevents you from reverting it in your branch?
>
> Btw, at least for me, reverting that particular change in autoload.el
> didn't fix the problem.

Nothing of course. However it is a great deal easier to test other
changes with a clean tree rather than carrying local patches.

    AndyM




^ permalink raw reply	[flat|nested] 9+ messages in thread

* Re: /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify.
  2011-11-23 23:41           ` Andy Moreton
@ 2011-11-24  3:54             ` Eli Zaretskii
  0 siblings, 0 replies; 9+ messages in thread
From: Eli Zaretskii @ 2011-11-24  3:54 UTC (permalink / raw)
  To: Andy Moreton; +Cc: emacs-devel

> From: Andy Moreton <andrewjmoreton@gmail.com>
> Date: Wed, 23 Nov 2011 23:41:44 +0000
> 
> On Wed 23 Nov 2011, Eli Zaretskii wrote:
> 
> >> From: Andy Moreton <andrewjmoreton@gmail.com>
> >> Date: Wed, 23 Nov 2011 13:47:03 +0000
> >> 
> >> Stefan, could you please revert your change to allow emacs to build
> >> properly ?
> >
> > What prevents you from reverting it in your branch?
> 
> Nothing of course. However it is a great deal easier to test other
> changes with a clean tree rather than carrying local patches.

As a general rule, I agree.  But given the apparent annoyance that it
caused you, so much so as to ask Stefan to revert a change, reverting
locally sounds appropriate.



^ permalink raw reply	[flat|nested] 9+ messages in thread

end of thread, other threads:[~2011-11-24  3:54 UTC | newest]

Thread overview: 9+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
     [not found] <E1RSVcR-0007q0-22@vcs.savannah.gnu.org>
2011-11-22  8:50 ` [Emacs-diffs] /srv/bzr/emacs/trunk r106462: * lisp/emacs-lisp/autoload.el (autoload-generate-file-autoloads): Simplify Juanma Barranquero
2011-11-22 16:43   ` Stefan Monnier
2011-11-23  0:42     ` Juanma Barranquero
2011-11-23 13:47       ` Andy Moreton
2011-11-23 17:58         ` Eli Zaretskii
2011-11-23 23:41           ` Andy Moreton
2011-11-24  3:54             ` Eli Zaretskii
2011-11-23 18:42       ` [Emacs-diffs] " Eli Zaretskii
2011-11-23 19:21         ` martin rudalics

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).