all messages for Emacs-related lists mirrored at yhetil.org
 help / color / mirror / code / Atom feed
* loaddefs.el under cedet/
@ 2009-10-02 16:09 Eli Zaretskii
  2009-10-02 16:53 ` Andreas Schwab
  0 siblings, 1 reply; 5+ messages in thread
From: Eli Zaretskii @ 2009-10-02 16:09 UTC (permalink / raw)
  To: Chong Yidong; +Cc: emacs-devel

What target in lisp/Makefile.in is supposed to create/update the
loaddefs.el files under cedet/?  After removing them, they don't get
re-created, which I think is a bug.  But I don't see what target did
that during bootstrap, so I don't know what to fix.

What am I missing?




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

* Re: loaddefs.el under cedet/
  2009-10-02 16:09 loaddefs.el under cedet/ Eli Zaretskii
@ 2009-10-02 16:53 ` Andreas Schwab
  2009-10-02 18:05   ` Eli Zaretskii
  0 siblings, 1 reply; 5+ messages in thread
From: Andreas Schwab @ 2009-10-02 16:53 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: Chong Yidong, emacs-devel

Eli Zaretskii <eliz@gnu.org> writes:

> What target in lisp/Makefile.in is supposed to create/update the
> loaddefs.el files under cedet/?  After removing them, they don't get
> re-created, which I think is a bug.  But I don't see what target did
> that during bootstrap, so I don't know what to fix.

The secondary loaddefs files are updated as a side effect of updating
the primary loaddefs file.  If you remove lisp/loaddefs.el the other
loaddefs files are forced to be updated as well.

Andreas.

-- 
Andreas Schwab, schwab@linux-m68k.org
GPG Key fingerprint = 58CA 54C7 6D53 942B 1756  01D3 44D5 214B 8276 4ED5
"And now for something completely different."




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

* Re: loaddefs.el under cedet/
  2009-10-02 16:53 ` Andreas Schwab
@ 2009-10-02 18:05   ` Eli Zaretskii
  2009-10-02 18:09     ` Eli Zaretskii
  2009-10-02 18:32     ` Glenn Morris
  0 siblings, 2 replies; 5+ messages in thread
From: Eli Zaretskii @ 2009-10-02 18:05 UTC (permalink / raw)
  To: Andreas Schwab; +Cc: cyd, emacs-devel

> From: Andreas Schwab <schwab@linux-m68k.org>
> Cc: Chong Yidong <cyd@stupidchicken.com>, emacs-devel@gnu.org
> Date: Fri, 02 Oct 2009 18:53:07 +0200
> 
> Eli Zaretskii <eliz@gnu.org> writes:
> 
> > What target in lisp/Makefile.in is supposed to create/update the
> > loaddefs.el files under cedet/?  After removing them, they don't get
> > re-created, which I think is a bug.  But I don't see what target did
> > that during bootstrap, so I don't know what to fix.
> 
> The secondary loaddefs files are updated as a side effect of updating
> the primary loaddefs file.  If you remove lisp/loaddefs.el the other
> loaddefs files are forced to be updated as well.

Thanks, that worked.

But I think we should add these 3 files to LOADDEFS, so that removing
them causes them to be re-created.  WDYT?




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

* Re: loaddefs.el under cedet/
  2009-10-02 18:05   ` Eli Zaretskii
@ 2009-10-02 18:09     ` Eli Zaretskii
  2009-10-02 18:32     ` Glenn Morris
  1 sibling, 0 replies; 5+ messages in thread
From: Eli Zaretskii @ 2009-10-02 18:09 UTC (permalink / raw)
  To: schwab, cyd, emacs-devel

> Date: Fri, 02 Oct 2009 20:05:49 +0200
> From: Eli Zaretskii <eliz@gnu.org>
> Cc: cyd@stupidchicken.com, emacs-devel@gnu.org
> 
> > From: Andreas Schwab <schwab@linux-m68k.org>
> > Cc: Chong Yidong <cyd@stupidchicken.com>, emacs-devel@gnu.org
> > Date: Fri, 02 Oct 2009 18:53:07 +0200
> > 
> > Eli Zaretskii <eliz@gnu.org> writes:
> > 
> > > What target in lisp/Makefile.in is supposed to create/update the
> > > loaddefs.el files under cedet/?  After removing them, they don't get
> > > re-created, which I think is a bug.  But I don't see what target did
> > > that during bootstrap, so I don't know what to fix.
> > 
> > The secondary loaddefs files are updated as a side effect of updating
> > the primary loaddefs file.  If you remove lisp/loaddefs.el the other
> > loaddefs files are forced to be updated as well.
> 
> Thanks, that worked.

Btw, "make autoloads" leaves loaddefs.el~ in the subdirectories of
lisp/cedet.




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

* Re: loaddefs.el under cedet/
  2009-10-02 18:05   ` Eli Zaretskii
  2009-10-02 18:09     ` Eli Zaretskii
@ 2009-10-02 18:32     ` Glenn Morris
  1 sibling, 0 replies; 5+ messages in thread
From: Glenn Morris @ 2009-10-02 18:32 UTC (permalink / raw)
  To: Eli Zaretskii; +Cc: cyd, Andreas Schwab, emacs-devel

Eli Zaretskii wrote:

> But I think we should add these 3 files to LOADDEFS, so that removing
> them causes them to be re-created.  WDYT?

That's not appropriate, as the comment above LOADDEFS indicates:

  # Note this includes only those files that need special rules to
  # build; ie it does not need to include things created via
  # generated-autoload-file (eg calc/calc-loaddefs.el).

Doing what you suggest would have no effect anyway. The `autoloads'
rule contains a `doit' dependency, which means it always runs. Simply
running the rule does not necessarily recreate the cedet loaddefs.

You cannot get the result you want without changing the way the Emacs
autoloads mechanism currently works. Your point seems to be, that if
you delete one of the cedet loaddefs files in isolation, and then run
`make autoloads', it will not be recreated. The last time this
question came up, the answer was "If it hurts, don't do that."

http://lists.gnu.org/archive/html/emacs-devel/2008-04/msg00886.html




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

end of thread, other threads:[~2009-10-02 18:32 UTC | newest]

Thread overview: 5+ messages (download: mbox.gz follow: Atom feed
-- links below jump to the message on this page --
2009-10-02 16:09 loaddefs.el under cedet/ Eli Zaretskii
2009-10-02 16:53 ` Andreas Schwab
2009-10-02 18:05   ` Eli Zaretskii
2009-10-02 18:09     ` Eli Zaretskii
2009-10-02 18:32     ` Glenn Morris

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.